Skip to contentSkip to conversation

bitbake.conf/base.bbclass: Drop PRINC support

RFC PATCH In review Richard Purdie 4 participants 5 messages Mar 2, 2014 – Mar 4, 2014

lore
Share & tools
lore
#0 lore raw

RFC PATCH 1/1 bitbake.conf/base.bbclass: Drop PRINC support

PRINC is no longer needed since we have the PR service instead which people should be using. Drop the PRINC code.

---

meta/classes/base.bbclass +0 -12
index 3e9051c..3cbf2cb 100644
@@ -455,18 +455,6 @@ python () {
         else:
             appendVar('EXTRA_OECONF', extraconf)
 
-    # If PRINC is set, try and increase the PR value by the amount specified
-    princ = d.getVar('PRINC', True)
-    if princ and princ != "0":
-        pr = d.getVar('PR', True)
-        pr_prefix = re.search("\D+",pr)
-        prval = re.search("\d+",pr)
-        if pr_prefix is None or prval is None:
-            bb.error("Unable to analyse format of PR variable: %s" % pr)
-        nval = int(prval.group(0)) + int(princ)
-        pr = pr_prefix.group(0) + str(nval) + pr[prval.end():]
-        d.setVar('PR', pr)
-
     pn = d.getVar('PN', True)
     license = d.getVar('LICENSE', True)
     if license == "INVALID":
meta/conf/bitbake.conf +0 -1
index 4879aa2..a4a7107 100644
@@ -188,7 +188,6 @@ ASSUME_PROVIDED = "\
 PN = "${@bb.parse.BBHandler.vars_from_file(d.getVar('FILE'),d)[0] or 'defaultpkgname'}"
 PV = "${@bb.parse.BBHandler.vars_from_file(d.getVar('FILE'),d)[1] or '1.0'}"
 PR = "${@bb.parse.BBHandler.vars_from_file(d.getVar('FILE'),d)[2] or 'r0'}"
-PRINC ?= "0"
 PF = "${PN}-${EXTENDPE}${PV}-${PR}"
 EXTENDPE = "${@['','${PE\x7d_'][int(d.getVar('PE',1) or 0) > 0]}"
 P = "${PN}-${PV}"
#1 lore raw
Quoted 2 lines · Richard Purdie

PRINC is no longer needed since we have the PR service instead which people should be using. Drop the PRINC code.

Many layers are still using it and there wasn't enough of PV bumps to remove them :(.

Quoted 43 lines

---

meta/classes/base.bbclass +0 -12
index 3e9051c..3cbf2cb 100644
@@ -455,18 +455,6 @@ python () {
         else:
             appendVar('EXTRA_OECONF', extraconf)
 
-    # If PRINC is set, try and increase the PR value by the amount specified
-    princ = d.getVar('PRINC', True)
-    if princ and princ != "0":
-        pr = d.getVar('PR', True)
-        pr_prefix = re.search("\D+",pr)
-        prval = re.search("\d+",pr)
-        if pr_prefix is None or prval is None:
-            bb.error("Unable to analyse format of PR variable: %s" % pr)
-        nval = int(prval.group(0)) + int(princ)
-        pr = pr_prefix.group(0) + str(nval) + pr[prval.end():]
-        d.setVar('PR', pr)
-
     pn = d.getVar('PN', True)
     license = d.getVar('LICENSE', True)
     if license == "INVALID":
meta/conf/bitbake.conf +0 -1
index 4879aa2..a4a7107 100644
@@ -188,7 +188,6 @@ ASSUME_PROVIDED = "\
 PN = "${@bb.parse.BBHandler.vars_from_file(d.getVar('FILE'),d)[0] or 'defaultpkgname'}"
 PV = "${@bb.parse.BBHandler.vars_from_file(d.getVar('FILE'),d)[1] or '1.0'}"
 PR = "${@bb.parse.BBHandler.vars_from_file(d.getVar('FILE'),d)[2] or 'r0'}"
-PRINC ?= "0"
 PF = "${PN}-${EXTENDPE}${PV}-${PR}"
 EXTENDPE = "${@['','${PE\x7d_'][int(d.getVar('PE',1) or 0) > 0]}"
 P = "${PN}-${PV}"

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

Signature · 1 line
Martin 'JaMa' Jansa     jabber: [email protected]
#2 lore raw
In reply to Richard Purdie
Quoted 2 lines · Richard Purdie

PRINC is no longer needed since we have the PR service instead which people should be using. Drop the PRINC code.

Will dropping this support break someone who is moving from a prior version of oe-core (i.e. will any of the PRs go backwards?)

--Mark

Quoted 44 lines

---

meta/classes/base.bbclass +0 -12
index 3e9051c..3cbf2cb 100644
@@ -455,18 +455,6 @@ python () {
          else:
              appendVar('EXTRA_OECONF', extraconf)
 
-    # If PRINC is set, try and increase the PR value by the amount specified
-    princ = d.getVar('PRINC', True)
-    if princ and princ != "0":
-        pr = d.getVar('PR', True)
-        pr_prefix = re.search("\D+",pr)
-        prval = re.search("\d+",pr)
-        if pr_prefix is None or prval is None:
-            bb.error("Unable to analyse format of PR variable: %s" % pr)
-        nval = int(prval.group(0)) + int(princ)
-        pr = pr_prefix.group(0) + str(nval) + pr[prval.end():]
-        d.setVar('PR', pr)
-
      pn = d.getVar('PN', True)
      license = d.getVar('LICENSE', True)
      if license == "INVALID":
meta/conf/bitbake.conf +0 -1
index 4879aa2..a4a7107 100644
@@ -188,7 +188,6 @@ ASSUME_PROVIDED = "\
  PN = "${@bb.parse.BBHandler.vars_from_file(d.getVar('FILE'),d)[0] or 'defaultpkgname'}"
  PV = "${@bb.parse.BBHandler.vars_from_file(d.getVar('FILE'),d)[1] or '1.0'}"
  PR = "${@bb.parse.BBHandler.vars_from_file(d.getVar('FILE'),d)[2] or 'r0'}"
-PRINC ?= "0"
  PF = "${PN}-${EXTENDPE}${PV}-${PR}"
  EXTENDPE = "${@['','${PE\x7d_'][int(d.getVar('PE',1) or 0) > 0]}"
  P = "${PN}-${PV}"

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

#3 lore raw
In reply to Richard Purdie
Quoted 4 lines · Richard Purdie

PRINC is no longer needed since we have the PR service instead which people should be using. Drop the PRINC code.

Nack! Layers still use it. Please, delay removal for 1.7 and add a WARNING for 1.6.

Signature · 3 lines
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750
#4 lore raw
Quoted 9 lines · Otavio Salvador
Quoted 4 lines · Richard Purdie

PRINC is no longer needed since we have the PR service instead which people should be using. Drop the PRINC code.

Nack! Layers still use it. Please, delay removal for 1.7 and add a WARNING for 1.6.

I sent this out to test the situation, I don't think we can remove it for 1.6, we should add warnings though...

This is a reminder for layer maintainers using this to stop though! :)

Cheers,

Richard