[oe] `PR = "r0"`: Add or not to add?

Anders Darander anders at chargestorm.se
Wed Aug 31 12:43:40 UTC 2011


* Koen Kooi <koen at dominion.thruhere.net> [110831 14:36]:
> Op 31 aug. 2011, om 14:27 heeft Paul Eggleton het volgende geschreven:

> > On Wednesday 31 August 2011 13:16:50 Koen Kooi wrote:
> >> Op 31-08-11 13:55, Anders Darander schreef:
> >>> To sad. It's a lot easier to remember to bump the PR, when PR actually is
> >>> in the recipe. Thus, including PR=0 will often remove one issue with
> >>> patches.
> >> 
> >> That's what review is for, no?
> > 
> > Surely you'd rather people have a better chance of getting it right the first 
> > time rather than you having to remind them for every patch? The almost 
> > insignificant burden of a PR = "r0" in each recipe seems worthwhile to me if it 
> > even helps a single person remember.

> I've found that in .dev having PR = r0 in recipes is a bigger burden
> than not having them, especially when using shared .inc files like
> xorg. And there have been enough patches where people don't get PR
> bumps right even if there's a PR already in the recipe. So I'm
> unconvinced that it makes a real difference in "getting it right".

> Even in the extreme case that every packaging patch is missing a PR
> bump that, for me,  is less annoying than issues I faced in .dev with
> PR = r0 sprinkled throughout the metadata.

Just curious (and it might convince me and others) what issues have PR =
"r0" caused in .dev?

/Anders

-- 
Anders Darander
ChargeStorm AB




More information about the Openembedded-devel mailing list