Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Mark Hatle <mark.hatle@windriver.com>
To: <openembedded-core@lists.openembedded.org>
Subject: Re: [RFC PATCH] base.bbclass: Deprecate the PRINC logic
Date: Wed, 29 May 2013 16:29:44 -0500	[thread overview]
Message-ID: <51A67348.9020709@windriver.com> (raw)
In-Reply-To: <1369862668.14887.236.camel@ted>

On 5/29/13 4:24 PM, Richard Purdie wrote:
> On Wed, 2013-05-29 at 14:00 -0300, Otavio Salvador wrote:
>> On Wed, May 29, 2013 at 11:47 AM, Richard Purdie
>> <richard.purdie@linuxfoundation.org> wrote:
>>          On Wed, 2013-05-29 at 16:37 +0200, Martin Jansa wrote:
>>          > On Wed, May 29, 2013 at 08:51:36AM -0500, Mark Hatle wrote:
>>          > > Background:
>>          > >
>>          > > At the recent TSC meeting we were discussing ways of
>>          removing the PRINC
>>          > > in favor of the PR server, which should now be standard.
>>           The first step
>>          > > in this process is coming up with a simple patch that
>>          declared PRINC as
>>          > > deprecated.  If this type of patch is successful, the
>>          block of code could
>>          > > be replaced with a bb.error eventually.
>>          > >
>>          > > It is not expected that this patch will go in by itself,
>>          but instead
>>          > > should be coordinated with changes to the recipes in
>>          common layers such
>>          > > as openembedded-core, meta-openembedded/meta-* and other
>>          community layers.
>>          >
>>          > This doesn't say what's the process of getting all PR
>>          increments
>>          > applied.
>>          >
>>          > Should we send list of recipes and required PR increments
>>          per layer (and
>>          > someone will sum these increments and create actual PR bump
>>          from it). Or
>>          > will we take turns and send actual PR bump patches per layer
>>          and someone
>>          > will define order of layers to go in (so that we prevent
>>          many conflicts
>>          > while merging)?
>>
>>
>>          This is something we need to figure out. The realistic process
>>          is
>>          probably do this layer by layer. If we can batch some up
>>          together that
>>          would obviously be better...
>
>> If this is the case, to ensure we have the PR in sync we should have
>> it PRINC as a bb.error; this will cause some pain but will avoid
>> PRServer picking the layer PRINC and losing it in next build. Or does
>> PRServer handle this gracefully?
>
> I proposed this but other TSC members didn't like the approach and would
> prefer a grace/warning period, maybe spanning until after the next
> release.
>
> I can see the arguments both ways...

I prefer the warning for one release approach.  What I'm afraid will happen is 
oe-core is released in the Fall, and a group of users migrates to it from the 
last release and suddenly all of their layers break.  These are the people who 
do not keep up with day to day development.

With the warning, they won't be immediately blocked -- but will be sufficiently 
annoyed (and warned) that they need to fix something or it will break.

--Mark

> Cheers,
>
> Richard
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>



  reply	other threads:[~2013-05-29 21:29 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-29 13:51 [RFC PATCH] base.bbclass: Deprecate the PRINC logic Mark Hatle
2013-05-29 14:11 ` Richard Purdie
2013-05-29 14:37 ` Martin Jansa
2013-05-29 14:47   ` Richard Purdie
2013-05-29 17:00     ` Otavio Salvador
2013-05-29 21:24       ` Richard Purdie
2013-05-29 21:29         ` Mark Hatle [this message]
2013-05-29 22:57           ` Otavio Salvador

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51A67348.9020709@windriver.com \
    --to=mark.hatle@windriver.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox