From: Arnout Vandecappelle <arnout@mind.be>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 00/13] Add support for a project directory
Date: Sun, 14 Oct 2012 15:57:44 +0200 [thread overview]
Message-ID: <507AC4D8.7020209@mind.be> (raw)
In-Reply-To: <20121014145539.4f456877@skate>
On 14/10/12 14:55, Thomas Petazzoni wrote:
> Arnout,
>
> On Sun, 14 Oct 2012 12:43:14 +0200, Arnout Vandecappelle wrote:
>
>>> All what this stuff is adding can already be done with the current
>>> Buildroot, by putting the configuration files in a board/something/foo
>>> directory, and adjusting the Buildroot configuration.
>>
>> It doesn't even have to be in a board/something/foo directory, it can
>> easily be out of tree. The only problem is that you have to write down
>> a fairly long path for each of the relevant config options.
>
> Come on, it's just something like:
>
> $(TOPDIR)/../myproject/linux.config
Except that I want the flexibility to have the buildroot tree in a different
place (e.g. a shared directory). That said, it's still easy:
$(MYPROJECT_DIR)/linux.config
The issue is that it has to be filled in in many places. I'm just trying
to make it easier. And I'm also trying to establish a default policy,
so it's easier for users to understand how to use buildroot.
[snip]
> I still don't get what this patch set adds that you can't do with the
> existing Buildroot infrastructure.
As I wrote several times already, all of this can already be done with the
current infrastructure (with one exception: 'make savedefconfig' saving
to a different file than 'defconfig'). It just provides simpler defaults.
[snip]
> I really would prefer to _document_ how to properly use the existing
> Buildroot infrastructure to cleanly separate custom stuff from
> Buildroot, rather than introducing more mechanisms on top of it.
Well, that documentation would be:
- Define all the custom paths to $(PROJECT_DIR)/foo.config
- Write a Makefile that does 'make -C <buildroot_dir> PROJECT_DIR=<project_dir>'
And people will wonder why all those paths aren't already
$(PROJECT_DIR)/foo.config by default.
[snip]
> It's not a question of diffstat. It's a question of "is this mechanism
> easy to grasp for newcomers and can it be immediately understood" and
> "does this mechanism adds any value over what Buildroot already
> provides"?
IMHO, it is.
I could do the following: I write a section in the documentation about
how to save your customization. This section will have two alternatives:
in-tree configuration or out-of-tree configuration. Then I can post a
second patch that removes the parts of the out-of-tree documentation that
become unnecessary when this patch series is implemented.
>>> the most annoying problems are
>>> not solved. So I really do prefer to keep things as it is: people have
>>> to use version control systems to keep their changes cleanly separated
>>> from the base Buildroot version.
>>
>> Version control doesn't really solve it, in my experience.
>
> Why so?
Mainly because customers for some obscure reason want to use svn.
[snip]
Regards,
Arnout
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286540
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
next prev parent reply other threads:[~2012-10-14 13:57 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-13 23:13 [Buildroot] [PATCH 00/13] Add support for a project directory Arnout Vandecappelle
2012-10-13 23:13 ` [Buildroot] [PATCH 01/13] Add BR2_PROJECT_DIR config option Arnout Vandecappelle
2012-10-13 23:13 ` [Buildroot] [PATCH 02/13] Set default BR2_PACKAGE_OVERRIDE_FILE based on BR2_PROJECT_DIR Arnout Vandecappelle
2012-10-13 23:14 ` [Buildroot] [PATCH 03/13] linux: get default paths from BR2_PROJECT_DIR Arnout Vandecappelle
2012-10-13 23:14 ` [Buildroot] [PATCH 04/13] busybox: " Arnout Vandecappelle
2012-10-13 23:14 ` [Buildroot] [PATCH 05/13] target/generic: " Arnout Vandecappelle
2012-10-13 23:14 ` [Buildroot] [PATCH 06/13] toolchain-crosstool-ng: " Arnout Vandecappelle
2012-10-13 23:14 ` [Buildroot] [PATCH 07/13] uClibc: " Arnout Vandecappelle
2012-10-13 23:14 ` [Buildroot] [PATCH 08/13] Store BR2_DEFCONFIG in .config, and use it to update the original input Arnout Vandecappelle
2012-10-14 18:37 ` Thomas De Schampheleire
2012-10-13 23:14 ` [Buildroot] [PATCH 09/13] Skip menuconfig if BR2_DEFCONFIG or BR2_PROJECT_DIR is given Arnout Vandecappelle
2012-10-13 23:14 ` [Buildroot] [PATCH 10/13] Add update-all-config target Arnout Vandecappelle
2012-10-14 18:45 ` Thomas De Schampheleire
2012-10-20 16:47 ` Arnout Vandecappelle
2012-10-20 16:52 ` Arnout Vandecappelle
2012-12-03 14:18 ` Stephan Hoffmann
2012-12-03 16:41 ` Thomas Petazzoni
2012-10-13 23:14 ` [Buildroot] [PATCH 11/13] Add target to create a project directory Arnout Vandecappelle
2012-10-13 23:21 ` [Buildroot] [PATCH v2] " Arnout Vandecappelle
2012-10-13 23:35 ` Valentine Barshak
2012-10-14 12:50 ` Arnout Vandecappelle
2012-10-16 17:36 ` Valentine Barshak
2012-10-13 23:14 ` [Buildroot] [PATCH 12/13] target/generic: add filesystem overlay option Arnout Vandecappelle
2012-10-14 0:39 ` Danomi Manchego
2012-10-14 12:53 ` Arnout Vandecappelle
2012-10-14 16:12 ` Danomi Manchego
2012-10-14 18:50 ` Thomas De Schampheleire
2012-10-20 16:15 ` Arnout Vandecappelle
2012-10-13 23:14 ` [Buildroot] [PATCH 13/13] Document BR2_PROJECT_DIR in the manual Arnout Vandecappelle
2012-10-14 8:35 ` [Buildroot] [PATCH 00/13] Add support for a project directory Thomas Petazzoni
2012-10-14 8:46 ` Thomas Petazzoni
2012-10-14 10:43 ` Arnout Vandecappelle
2012-10-14 12:55 ` Thomas Petazzoni
2012-10-14 13:57 ` Arnout Vandecappelle [this message]
2012-10-16 20:03 ` Arnout Vandecappelle
2012-10-17 17:26 ` Thomas Petazzoni
2012-10-17 18:42 ` Sagaert Johan
2012-10-14 18:56 ` Thomas De Schampheleire
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=507AC4D8.7020209@mind.be \
--to=arnout@mind.be \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox