[gnome-continuous-yocto/gnomeostree-3.28-rocko: 257/8267] bitbake: fetch: Use OrderedDict for url parameters



commit ca824a976a131d0955f61cff3df9711efd10de63
Author: Richard Purdie <richard purdie linuxfoundation org>
Date:   Thu May 12 08:04:17 2016 +0100

    bitbake: fetch: Use OrderedDict for url parameters
    
    Without this, the dict can reorder causing sanity test failures.
    
    (Bitbake rev: ca8c91acc9396385834b266d4e8b84d917e5e298)
    
    Signed-off-by: Richard Purdie <richard purdie linuxfoundation org>

 bitbake/lib/bb/fetch2/__init__.py |    5 +++--
 bitbake/lib/bb/tests/fetch.py     |   11 ++++++-----
 2 files changed, 9 insertions(+), 7 deletions(-)
---
diff --git a/bitbake/lib/bb/fetch2/__init__.py b/bitbake/lib/bb/fetch2/__init__.py
index 600e216..7d2f350 100644
--- a/bitbake/lib/bb/fetch2/__init__.py
+++ b/bitbake/lib/bb/fetch2/__init__.py
@@ -32,6 +32,7 @@ import signal
 import logging
 import urllib
 import urlparse
+import collections
 import bb.persist_data, bb.utils
 import bb.checksum
 from bb import data
@@ -297,7 +298,7 @@ class URI(object):
             if self.query else '')
 
     def _param_str_split(self, string, elmdelim, kvdelim="="):
-        ret = {}
+        ret = collections.OrderedDict()
         for k, v in [x.split(kvdelim, 1) for x in string.split(elmdelim)]:
             ret[k] = v
         return ret
@@ -390,7 +391,7 @@ def decodeurl(url):
         user = ''
         pswd = ''
 
-    p = {}
+    p = collections.OrderedDict()
     if parm:
         for s in parm.split(';'):
             if s:
diff --git a/bitbake/lib/bb/tests/fetch.py b/bitbake/lib/bb/tests/fetch.py
index cba783f..5ff156c 100644
--- a/bitbake/lib/bb/tests/fetch.py
+++ b/bitbake/lib/bb/tests/fetch.py
@@ -22,6 +22,7 @@
 import unittest
 import tempfile
 import subprocess
+import collections
 import os
 from bb.fetch2 import URI
 from bb.fetch2 import FetchMethod
@@ -133,10 +134,10 @@ class URITest(unittest.TestCase):
             'userinfo': 'anoncvs:anonymous',
             'username': 'anoncvs',
             'password': 'anonymous',
-            'params': {
-                'tag': 'V0-99-81',
-                'module': 'familiar/dist/ipkg'
-            },
+            'params': collections.OrderedDict([
+                ('tag', 'V0-99-81'),
+                ('module', 'familiar/dist/ipkg')
+            ]),
             'query': {},
             'relative': False
         },
@@ -660,7 +661,7 @@ class URLHandle(unittest.TestCase):
     datatable = {
        "http://www.google.com/index.html"; : ('http', 'www.google.com', '/index.html', '', '', {}),
        "cvs://anoncvs cvs handhelds org/cvs;module=familiar/dist/ipkg" : ('cvs', 'cvs.handhelds.org', 
'/cvs', 'anoncvs', '', {'module': 'familiar/dist/ipkg'}),
-       "cvs://anoncvs:anonymous cvs handhelds org/cvs;tag=V0-99-81;module=familiar/dist/ipkg" : ('cvs', 
'cvs.handhelds.org', '/cvs', 'anoncvs', 'anonymous', {'tag': 'V0-99-81', 'module': 'familiar/dist/ipkg'}),
+       "cvs://anoncvs:anonymous cvs handhelds org/cvs;tag=V0-99-81;module=familiar/dist/ipkg" : ('cvs', 
'cvs.handhelds.org', '/cvs', 'anoncvs', 'anonymous', collections.OrderedDict([('tag', 'V0-99-81'), ('module', 
'familiar/dist/ipkg')])),
        "git://git.openembedded.org/bitbake;branch=@foo" : ('git', 'git.openembedded.org', '/bitbake', '', 
'', {'branch': '@foo'}),
        "file://somelocation;someparam=1": ('file', '', 'somelocation', '', '', {'someparam': '1'}),
     }


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]