[Buildroot] [PATCH v5 4/5] barebox: support 2nd config build
Pieter Smith
pieter at boesman.nl
Sun Apr 24 16:52:29 UTC 2016
Hi Thomas,
On Sun, Apr 24, 2016 at 05:58:03PM +0200, Thomas Petazzoni wrote:
> Hello,
>
> On Sun, 24 Apr 2016 11:18:46 +0200, Pieter Smith wrote:
>
> > boot/barebox/Config.in | 2 +
> > boot/barebox/barebox-2/Config.in | 72 +++++++++++++++++++++++++++++++++++
> > boot/barebox/barebox-2/barebox-2.hash | 1 +
> > boot/barebox/barebox-2/barebox-2.mk | 9 +++++
> > boot/barebox/barebox.mk | 3 ++
>
> Regarding this one, I am not sure, there are two things that bother me
> a bit:
>
> - The name "barebox-2", which I find a bit weird. What about
> "barebox-aux" or something like that, as opposed to "barebox-main" ?
>
> - The organization of the folders. Having the barebox "infra" + the
> main barebox package both defined in boot/barebox/barebox.mk seems
> weird. Ideally, I would have preferred something like:
>
> boot/barebox/barebox.mk <-- common infrastructure
> boot/barebox/barebox-main/ <-- the main Barebox
> boot/barebox/barebox-aux/ <-- the auxiliary Barebox
I had it set up this way, but changed it due to some of the review feedback
that Arnout provided.
> Of course, some tricks will be needed to make the barebox-main/
> package behave properly when its options will still be named
> BR2_TARGET_BAREBOX_<foo> and not BR2_TARGET_BAREBOX_MAIN_<foo>.
> Alternatively, if that really doesn't work, what about:
>
> boot/barebox/barebox.mk <-- common infrastructure
> boot/barebox/barebox/ <-- the main Barebox
> boot/barebox/barebox-aux/ <-- the auxiliary Barebox
This approach seems reasonable. I will set it up this way. I will also rename
all BAREBOX_2 KConfig options to BAREBOX_AUX. Is this acceptable?
> I've applied the first three patches of the series to ease your work,
> but I'd like to see if we can get something a bit better for this
> patch. I know Arnout has done a lot of review, so don't hesitate to let
> me know if all this has already been discussed and why the current
> organization has been chosen compared to something more similar to my
> proposal.
I don't think the organization was changed based on concrete arguments. Arnout
just liked it more this way. I do however like your 2nd proposed organization
more.
[snip]
- Pieter
More information about the buildroot
mailing list