[Buildroot] [PATCH/autobuild 4/8] autobuild-run: remove redundant 'make oldconfig'
Arnout Vandecappelle
arnout at mind.be
Mon Apr 10 08:46:54 UTC 2017
On 10-04-17 10:25, Thomas Petazzoni wrote:
> Hello,
>
> On Sun, 9 Apr 2017 22:51:24 +0200, Arnout Vandecappelle
> (Essensium/Mind) wrote:
>> After running 'make randpackageconfig', and additional 'make oldconfig'
>> is done. However, this is redundant, since randpackageconfig already
>> does an olddefconfig.
>>
>> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>
>
> Well, this oldconfig is not exactly right after randpackageconfig.
>
> We do this:
>
> - randpackageconfig
> - fixup_config()
> - oldconfig
>
> The fixup_config() function potentially does some changes to
> the .config file, and that's why we re-run oldconfig afterwards. This
> ensures that the build will not wait indefinitely while asking the user
> the value of a config option.
D'oh, I missed that. Fortunately somebody reviews my crap! :-)
So instead I'll replace it with olddefconfig and squash with the preceding
patch, OK?
Regards,
Arnout
>
> Therefore, I wouldn't qualify this oldconfig as "redundant".
>
> Best regards,
>
> Thomas
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
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: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
More information about the buildroot
mailing list