[oe-commits] [bitbake] branch master-next updated: bitbake: fix version comparison when one of the versions ends in .

[git at git.openembedded.org]

This is an automated email from the git hooks/post-receive script.

rpurdie pushed a commit to branch master-next
in repository bitbake.

The following commit(s) were added to refs/heads/master-next by this push:
     new fef56d2  bitbake: fix version comparison when one of the versions ends in .
fef56d2 is described below

commit fef56d28c3efec4876c379898cbc4d4c65303aee
Author: Alexander Kanavin <alex.kanavin at gmail.com>
AuthorDate: Sun Feb 24 21:07:28 2019 +0100

    bitbake: fix version comparison when one of the versions ends in .
    
    Previously, this would happen:
    
    ======================================================================
    ERROR: test_vercmpstring (bb.tests.utils.VerCmpString)
    ----------------------------------------------------------------------
    Traceback (most recent call last):
      File "/home/alexander/development/poky/bitbake/lib/bb/tests/utils.py", line 45, in test_vercmpstring
        result = bb.utils.vercmp_string('1.', '1.1')
      File "/home/alexander/development/poky/bitbake/lib/bb/utils.py", line 143, in vercmp_string
        return vercmp(ta, tb)
      File "/home/alexander/development/poky/bitbake/lib/bb/utils.py", line 135, in vercmp
        r = vercmp_part(va, vb)
      File "/home/alexander/development/poky/bitbake/lib/bb/utils.py", line 124, in vercmp_part
        elif ca < cb:
    TypeError: '<' not supported between instances of 'NoneType' and 'int'
    
    ----------------------------------------------------------------------
    
    Signed-off-by: Alexander Kanavin <alex.kanavin at gmail.com>
    Signed-off-by: Richard Purdie <richard.purdie at linuxfoundation.org>
---
 lib/bb/tests/utils.py | 4 ++++
 lib/bb/utils.py       | 4 ++++
 2 files changed, 8 insertions(+)

diff --git a/lib/bb/tests/utils.py b/lib/bb/tests/utils.py
index 2f4ccf3..f1cd83a 100644
--- a/lib/bb/tests/utils.py
+++ b/lib/bb/tests/utils.py
@@ -42,6 +42,10 @@ class VerCmpString(unittest.TestCase):
         self.assertTrue(result < 0)
         result = bb.utils.vercmp_string('1.1', '1.0+1.1-beta1')
         self.assertTrue(result > 0)
+        result = bb.utils.vercmp_string('1.', '1.1')
+        self.assertTrue(result < 0)
+        result = bb.utils.vercmp_string('1.1', '1.')
+        self.assertTrue(result > 0)
 
     def test_explode_dep_versions(self):
         correctresult = {"foo" : ["= 1.10"]}
diff --git a/lib/bb/utils.py b/lib/bb/utils.py
index 9cb702d..b652a68 100644
--- a/lib/bb/utils.py
+++ b/lib/bb/utils.py
@@ -121,6 +121,10 @@ def vercmp_part(a, b):
             return -1
         elif oa > ob:
             return 1
+        elif ca is None:
+            return -1
+        elif cb is None:
+            return 1
         elif ca < cb:
             return -1
         elif ca > cb:

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.