[Buildroot] [v3, 3/4] barebox: user selection of build output images

[Pieter Smith]

On Sat, Feb 27, 2016 at 12:43:40AM +0100, Arnout Vandecappelle wrote:
> On 01/20/16 23:43, Pieter Smith wrote:
> > A non-default image can be selected as the barebox build output in the images/
> > directory. This typically is needed to select the appropriate image generated
> > by multi-platform support defconfigs (E.g. am335x).

[snip]

>  If we deprecate barebox pre-2012.10, then we can simplify this a lot:
> 
> config BR2_TARGET_BAREBOX_IMAGE_FILE
> 	string "Image file path"
> 	default "barebox-flash-image"
> 	help
> 	  Name of the built barebox image file in the barebox images directory
> 
> 
>  Even if we don't want to deprecate barebox pre-2012.10, I'd prefer to keep this
> simple config and do the handling of legacy barebox.bin in the .mk file. For
> pre-2012.10, you'd have to explicitly set it to barebox.bin instead (which
> should be mentioned in the help text then).

ACK. Done. I solved this with the above default and help text pointing the user
to use 'barebox.bin' for older versions. I'm just waiting for Yegor, and then
I'll repost.

Right now a user-friendly warning is generated if an old config is used and
this config item has not yet been set. I can have the makefile also generate a
hint for older barebox versions if the image filename isn't found:
- Do you think the additional complexity is warranted?

>  We should find a good way to deal with the images/ part of the path, however.
> This could e.g. be done in the .mk file, by checking both the top dir and the
> images dir.

ACK. Done. In v4 I first try the build ./ directory, and then ./images.

>  Regards,
>  Arnout

[snip]

- Pieter