On 5/28/13 9:06 AM, Richard Purdie wrote:
On Tue, 2013-05-28 at 15:46 +0200, Martin Jansa wrote:
On Tue, May 28, 2013 at 01:28:41PM +0000, [email protected] wrote:
Module: openembedded-core.git
Branch: master-next
Commit: e1cf564ebc8e7b4fa626a645356f6a4d7f5ba064
URL:    
http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=e1cf564ebc8e7b4fa626a645356f6a4d7f5ba064

Author: Mark Hatle <[email protected]>
Date:   Tue May 21 13:29:03 2013 -0500

base.bbclass: Deprecate the PRINC logic

The PRINC logic is now deprecated, the PR server should be used to handle
the automatic incrementing of the PR (package release) field.  The default
setting of '0' has been removed, and a warning message has been added.

How are people supposed to remove existing PRINC without causing version
going backwards?

Do I have to choose between seeing 100 warnings about deprecated PRINC
or 100 ERRORs from buildhistory about versions going backwards?

Sorry, this is coming out a bit backwards.

At the TSC meeting, we discussed ways of progressing with removal of
PRINC as it is causing pain and we shouldn't need it any more. We were
wondering if we could have the system warn on usage of PRINC, then
accept PR bumps to the main recipe at the same time that the usage of
PRINC was removed (taking PR bumps and removing PRINC seems to be the
only way to proceed). Initially I wondered if we could make it a hard
error, which would then force the PR bump to be in sync with the
removal. People are justifiably concerned at the idea of hard errors
though.

Mark sent me a patch, I thought it was an RFC on the list and applied it
to master-next to experiment with. It wasn't send to the list, I'm not
sure what Mark's intention was, its possible I was supposed to raise the
subject, then post the patch.

Ya, this was a result of the TSC discussion. I sent it to Richard to get feedback on the approach, before it went to the list.

Anyhow, as you point out, the patch has a couple of issues. We need to
put back the default value into local.conf for now at the very least so
this warns, rather than gives an obtuse error. Its not going into master
until there is more discussion.

So lets reset here, Mark will post the RFC patch and we can discuss
whether there is a way we can get rid of the PRINC usage without causing
people too many problems. Sorry for the confusion caused :/.

I'll be happy to do so.

--Mark

Cheers,

Richard



_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to