All of lore.kernel.org
 help / color / mirror / Atom feed
* PR and .bbappend
@ 2011-02-02 15:03 Gary Thomas
  2011-02-02 15:13 ` Chris Larson
  0 siblings, 1 reply; 4+ messages in thread
From: Gary Thomas @ 2011-02-02 15:03 UTC (permalink / raw)
  To: Poky

Should .bbappend files have a PR in them?  If so, must it
match the one in the main .bb file?

-- 
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: PR and .bbappend
  2011-02-02 15:03 PR and .bbappend Gary Thomas
@ 2011-02-02 15:13 ` Chris Larson
  2011-02-02 15:17   ` Mark Hatle
  0 siblings, 1 reply; 4+ messages in thread
From: Chris Larson @ 2011-02-02 15:13 UTC (permalink / raw)
  To: Gary Thomas; +Cc: Poky

On Wed, Feb 2, 2011 at 8:03 AM, Gary Thomas <gary@mlbassoc.com> wrote:
> Should .bbappend files have a PR in them?  If so, must it
> match the one in the main .bb file?

All metadata in a .bbappend is essentially concatenated to the recipe.
 If you define PR in the .bbappend, it will override the one in the
recipe, which is unlikely what you want, as that loses information.
The better route is to do something like PR .= ".1" in the .bbappend,
and bump that when changing the bbappend file.
-- 
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: PR and .bbappend
  2011-02-02 15:13 ` Chris Larson
@ 2011-02-02 15:17   ` Mark Hatle
  2011-02-02 17:40     ` Darren Hart
  0 siblings, 1 reply; 4+ messages in thread
From: Mark Hatle @ 2011-02-02 15:17 UTC (permalink / raw)
  To: Chris Larson; +Cc: Poky

On 2/2/11 9:13 AM, Chris Larson wrote:
> On Wed, Feb 2, 2011 at 8:03 AM, Gary Thomas <gary@mlbassoc.com> wrote:
>> Should .bbappend files have a PR in them?  If so, must it
>> match the one in the main .bb file?
> 
> All metadata in a .bbappend is essentially concatenated to the recipe.
>  If you define PR in the .bbappend, it will override the one in the
> recipe, which is unlikely what you want, as that loses information.
> The better route is to do something like PR .= ".1" in the .bbappend,
> and bump that when changing the bbappend file.

I was going to suggest the same thing.  A good approach is to append a value to
the existing PR.  (I suggest this get added as a best practice in the
documentation on bbappend...  I'm not sure who the correct person is to point
this out to though.)

--Mark


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: PR and .bbappend
  2011-02-02 15:17   ` Mark Hatle
@ 2011-02-02 17:40     ` Darren Hart
  0 siblings, 0 replies; 4+ messages in thread
From: Darren Hart @ 2011-02-02 17:40 UTC (permalink / raw)
  To: Mark Hatle; +Cc: Poky, Chris Larson

On 02/02/2011 07:17 AM, Mark Hatle wrote:
> On 2/2/11 9:13 AM, Chris Larson wrote:
>> On Wed, Feb 2, 2011 at 8:03 AM, Gary Thomas<gary@mlbassoc.com>  wrote:
>>> Should .bbappend files have a PR in them?  If so, must it
>>> match the one in the main .bb file?
>>
>> All metadata in a .bbappend is essentially concatenated to the recipe.
>>   If you define PR in the .bbappend, it will override the one in the
>> recipe, which is unlikely what you want, as that loses information.
>> The better route is to do something like PR .= ".1" in the .bbappend,
>> and bump that when changing the bbappend file.
>
> I was going to suggest the same thing.  A good approach is to append a value to
> the existing PR.  (I suggest this get added as a best practice in the
> documentation on bbappend...  I'm not sure who the correct person is to point
> this out to though.)

That would be Scott R, on CC.

>
> --Mark
> _______________________________________________
> poky mailing list
> poky@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/poky


-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2011-02-02 17:40 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-02-02 15:03 PR and .bbappend Gary Thomas
2011-02-02 15:13 ` Chris Larson
2011-02-02 15:17   ` Mark Hatle
2011-02-02 17:40     ` Darren Hart

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.