From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Sat, 14 Sep 2013 07:29:56 +0200 Subject: [Buildroot] [PATCH 2/3] Add support for BR2_EXTERNAL In-Reply-To: <52337C1B.3090108@mind.be> References: <1378646129-4167-1-git-send-email-thomas.petazzoni@free-electrons.com> <1378646129-4167-3-git-send-email-thomas.petazzoni@free-electrons.com> <52322C8E.7010501@mind.be> <5232351D.3030703@mind.be> <523238B0.3050500@mind.be> <52337C1B.3090108@mind.be> Message-ID: <20130914072956.20018a94@skate> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Dear Arnout Vandecappelle, On Fri, 13 Sep 2013 22:56:59 +0200, Arnout Vandecappelle wrote: > > O= is stored in the generated makefile (as you know). However, it gets very > > annoying to type BR2_EXTERNAL=/path/to/external and I don't always want to > > export it as shell environment variable. Thus, adding that line to > > mkmakefile is handy since usually when you specify O= you are doing a > > specific build which BR2_EXTERNAL would definitely be a part of. > > My point is: it should (and can) be made part of the .config. It really > has to be part of the .config, because some options in the .config will > refer to the packages that are defined in the external directory. I believe it cannot be part of the .config, because there is a chicken-and-egg problem due to the fact that running 'make menuconfig' requires to already know the BR2_EXTERNAL value. Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com