[oe] [PATCH 1/2] Rename url params patch=<ignored>/pnum=<n> to apply={yes, no}/striplevel=<n>

[Chris Larson]

On Fri, Jun 4, 2010 at 10:31 AM, Tom Rini <tom_rini at mentor.com> wrote:

> Chris Larson wrote:
>
>> On Fri, Jun 4, 2010 at 5:52 AM, Frans Meulenbroeks <
>> fransmeulenbroeks at gmail.com> wrote:
>>
>>  Apologies for not replying to the original message, but that one is
>>> already deleted.
>>>
>>> Just bumped into this after doing a pull.
>>>
>>> A few comments:
>>>
>>> I would suggest if doing something like this that it is announced
>>> somewhat more explictly so people are well aware of it. Since it was
>>> brought as a patch it escaped my attention, and in fact it is not only
>>> a patch but also a change of policy.
>>>
>>>
>> Personally, I'd think people should always read any patches to the main
>> classes, including patch.bbclass.  Any change to a core class or config
>> file
>> has the potential for large impact, and it would benefit everyone to be
>> aware of those things.
>>
>> That said, I'll be sure to send out an RFC to try to draw more attention
>> to
>> it next time around.
>>
>>
>>  Also I'd like to suggest to complete the change by also updating the
>>> manual, otherwise this could become confusing.
>>>
>>>
>> Yes, I know, I intended (and still do intend) to update the manual, but
>> haven't gotten to it yet.  I apologize for the delay.
>>
>
> Maybe we should start saying "no changing how stuff is documented WITHOUT
> changing the documentation".  It's not like we keep the docs in some hard to
> edit format and I think other projects work the same way.


That sounds like a good policy to adopt.
-- 
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics