[PATCH 0/1] Bugfix in PRINC increment

[PATCH 0/1] Bugfix in PRINC increment

[Khem Raj]

Pull URL: git://git.openembedded.org/openembedded-core-contrib
  Branch: kraj/princ
  Browse: http://git.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=kraj/princ

Thanks,
    Khem Raj <raj.khem@gmail.com>
---


Khem Raj (1):
  base.bbclass: Fix PR increment bug when PR number is a single digit

 meta/classes/base.bbclass |   19 ++++++-------------
 1 files changed, 6 insertions(+), 13 deletions(-)

-- 
1.7.4.1

[PATCH 1/1] base.bbclass: Fix PR increment bug when PR number is a single digit

[Khem Raj]

PRINC which should add to base PR value has a problem when
the PR is single digit e.g. r0 - r9. Current algorithm
needed atleasts 2 digits to successfully populate end and begin
markers.

We reimplement the incrementing algorithm using regular expressions
which addressed the above mentioned problem and
simplifies the logic a bit and gets rid of loops and conditionals

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 meta/classes/base.bbclass |   19 ++++++-------------
 1 files changed, 6 insertions(+), 13 deletions(-)

diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
index d9ed15f..23095ec 100644
--- a/meta/classes/base.bbclass
+++ b/meta/classes/base.bbclass
@@ -233,25 +233,18 @@ do_build () {
 }
 
 python () {
-    import exceptions, string
+    import exceptions, string, re
 
     # If PRINC is set, try and increase the PR value by the amount specified
     princ = bb.data.getVar('PRINC', d, True)
     if princ:
         pr = bb.data.getVar('PR', d, True)
-        start = -1
-        end = -1
-        for i in range(len(pr)):
-            if pr[i] in string.digits:
-                if start == -1:
-                    start = i
-                else:
-                    end = i
-        if start == -1 or end == -1:
+        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)
-        prval = pr[start:end+1]
-        prval = int(prval) + int(princ)
-        pr = pr[0:start] + str(prval) + pr[end:len(pr)-1]
+        nval = int(prval.group(0)) + int(princ)
+        pr = pr_prefix.group(0) + str(nval) + pr[prval.end():]
         bb.data.setVar('PR', pr, d)
 
     pn = bb.data.getVar('PN', d, 1)
-- 
1.7.4.1

Re: [PATCH 0/1] Bugfix in PRINC increment

[Saul Wold]

On 05/16/2011 11:26 AM, Khem Raj wrote:
>
> Pull URL: git://git.openembedded.org/openembedded-core-contrib
>    Branch: kraj/princ
>    Browse: http://git.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=kraj/princ
>
> Thanks,
>      Khem Raj<raj.khem@gmail.com>
> ---
>
>
> Khem Raj (1):
>    base.bbclass: Fix PR increment bug when PR number is a single digit
>
>   meta/classes/base.bbclass |   19 ++++++-------------
>   1 files changed, 6 insertions(+), 13 deletions(-)
>
Merged into oe-core

Thanks
	Sau!