[oe] [PATCH] barebox: Add recipe for this new bootloader

[Frans Meulenbroeks]

2010/1/17 Paul Menzel <paulepanter at users.sourceforge.net>:
> Am Sonntag, den 17.01.2010, 22:02 +0100 schrieb Koen Kooi:
>> On 17-01-10 21:32, Eric Benard wrote:
>> > * barebox is "a bootloader that inherits the best of U-Boot and the
>> Linux kernel"
>> > * add recipe for v2009.12.0 (last stable)
>> > * add recipe for git repository
>> >
>> > Signed-off-by: Eric Benard <eric at eukrea.com>
>>
>> > +++ b/recipes/barebox/barebox_2009.12.bb
>> > @@ -0,0 +1,15 @@
>>
>>
>> > +PR = "r0"
>>
>> > +++ b/recipes/barebox/barebox_git.bb
>> > @@ -0,0 +1,10 @@
>>
>> > +PR = "r0"
>>
>> Please don't set variables to the current default value, but leave
>> them
>> out. We had tons of problems with vars getting copy/paste where the
>> default changed and we had to manually fix those (e.g. efl.bbclass
>> still
>> pokes needlessly at PACKAGES).
>>
>> Setting PR is relatively minor, but lets stay consistent :)
>
> Could someone please change this in the manual please [1].
>
> »It is good practice to always define PR in your recipes, even for the
> "r0" release, so that when editing the recipe it is clear that the PR
> number needs to be updated.«
>
Personally I'm in favour of the above good practice.
I suggest to keep PR = "r0" in recipes

Frans