[Buildroot] Analysis of bug #5030: busybox built fails if we use an override src dir BUSYBOX_OVERRIDE_SRCDIR and that dir does not contain .config

[Peter Korsgaard <jacmet@uclibc.org>]

>>>>> "Thomas" == Thomas De Schampheleire <patrickdepinguin@gmail.com> writes:

Hi,

 >>> > I also accept the principle of this patch.
 >>> >
 >>> > As a side note, this behavior of busybox.mk was also problematic when
 >>> > trying to implement out of tree build for packages, because .config is
 >>> > inherently part of the *build* directory, but the build directory
 >>> > doesn't exist yet during the extract step: it is only created at the
 >>> > beginning of the configure step. So my out-of-tree patch set contains:
 >>> >
 >>> > -# We do this here to avoid busting a modified .config in configure
 >>> > -BUSYBOX_POST_EXTRACT_HOOKS += BUSYBOX_COPY_CONFIG
 >>> > -
 >>> >  define BUSYBOX_CONFIGURE_CMDS
 >>> > +       $(BUSYBOX_COPY_CONFIG)
 >>> >
 >>> 
 >>> Then maybe we should apply exactly this change, and not the proposed
 >>> one that uses a hook?
 >>> Additionally, we should then also make this change in uclibc, right?
 >> 
 >> Yes. But I would like to hear Peter's opinion on this.

 > Peter, what do you think?

I don't feel strongly about it, but if changing it to "the linux way"
solves problems, then lets do it.

-- 
Bye, Peter Korsgaard