From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Sat, 27 Aug 2016 21:41:26 +0200 Subject: [Buildroot] [PATCH 05/16 v3] core: move rule to create basic directories In-Reply-To: <35c825b427c5e89497aafde9b787986eeeac1ad7.1468750623.git.yann.morin.1998@free.fr> References: <35c825b427c5e89497aafde9b787986eeeac1ad7.1468750623.git.yann.morin.1998@free.fr> Message-ID: <20160827214126.05cf6760@free-electrons.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Hello, On Sun, 17 Jul 2016 12:34:25 +0200, Yann E. MORIN wrote: > Some of those directories will be needed even during configuration, like > BUILD_DIR, where we'll store the generated kconfig snippet. > > So, move the rule to create them outside the HAVE_CONFIG block. > > Signed-off-by: "Yann E. MORIN" > Cc: Thomas Petazzoni > Cc: Peter Korsgaard I've applied this patch to next, after fixing the typo in the commit log noticed by Romain Naour. One question below, though. > +################################################################################ > +# > +# staging and target directories do NOT list these as > +# dependencies anywhere else > +# > +################################################################################ Do you understand what this comment means? It seems to date back from the "major rework" commit 08782ae7d8812933ad3f932132955ea1c0fb1e52 in 2002, and I don't really understand the meaning of this comment. Thanks, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com