[Buildroot] [PATCH v2 1/3] barebox: add custom version option

Fabio Porcedda fabio.porcedda at gmail.com
Fri May 3 06:49:04 UTC 2013


On Wed, May 1, 2013 at 11:10 PM, Thomas Petazzoni
<thomas.petazzoni at free-electrons.com> wrote:
> Dear Fabio Porcedda,

Hi Thomas,
thanks for reviewing.

> On Wed, 17 Apr 2013 09:41:41 +0200, Fabio Porcedda wrote:
>
>> -config BR2_TARGET_BAREBOX_2013_01
>> -     bool "2013.01.0"
>> -
>> -config BR2_TARGET_BAREBOX_2013_02
>> -     bool "2013.02.0"
>> -
>> -config BR2_TARGET_BAREBOX_2013_03
>> -     bool "2013.03.0"
>> -
>> -config BR2_TARGET_BAREBOX_2013_04
>> +config BR2_TARGET_BAREBOX_LATEST_VERSION
>>       bool "2013.04.0"
>>
>> +config BR2_TARGET_BAREBOX_CUSTOM_VERSION
>> +     bool "Custom version"
>> +     help
>> +       This option allows to use a specific official versions
>> +
>
> We need to have something that is consistent between U-Boot, Barebox
> and the kernel. Currently U-Boot has a version selection, the Linux
> kernel allows to chose the latest version (but with an option named
> after the version itself, so the name of the option changes when the
> version changes) or a custom version, and now for Barebox you're
> proposing yet another solution: select the latest version (with an
> option name that never changes) or a custom version.

I think it's easier to have a option name that doesn't change.
If there isn't a specific reason to have a option name that changes then
I will send a patch to change the linux kernel option name.

BR2_LINUX_KERNEL_3_9 -> BR2_LINUX_KERNEL_LATEST_VERSION

Best regards
Fabio Porcedda

>
> So, I'm fine doing changes on this, but they should be consistent
> across packages.
>
> Best regards,
>
> Thomas
> --
> Thomas Petazzoni, Free Electrons
> Kernel, drivers, real-time and embedded Linux
> development, consulting, training and support.
> http://free-electrons.com



--
Fabio Porcedda



More information about the buildroot mailing list