sorting

packtivity/statecontexts/posixfs_context.py

@@ -18,15 +18,16 @@ def __init__(self,readwrite = None, readonly = None, dependencies = None, identi
         except AssertionError:
             raise TypeError('readwrite and readonly must be None or a list {} {}'.format(type(readonly)))
         self._identifier = identifier
-        self.readwrite = list(map(os.path.realpath,readwrite) if readwrite else  [])
 
         readonlies = list(map(os.path.realpath,readonly) if readonly else  [])
         for d in dependencies or []:
             if d.readwrite:
                 readonlies += d.readwrite # if dep has readwrite add those
             else:
                 readonlies += d.readonly # else add the readonlies
-        self.readonly = readonlies
+
+        self.readwrite = sorted(list(map(os.path.realpath,readwrite) if readwrite else  []))
+        self.readonly  = sorted(list(map(os.path.realpath,readonlies)))
         self.datamodel = None
 
     def __repr__(self):
@@ -66,7 +67,7 @@ def state_hash(self):
         return: SHA1 hash
         '''
         #hash the upstream / input state
-        dep_checksums = [checksumdir.dirhash(d) for d in self.readonly if os.path.isdir(d)] 
+        dep_checksums = [checksumdir.dirhash(d) for d in self.readonly if os.path.isdir(d)]
 
         #hash out writing state
         state_checksums = [checksumdir.dirhash(d) for d in self.readwrite if os.path.isdir(d)]
@@ -102,7 +103,7 @@ def json(self):
     @classmethod
     def fromJSON(cls,jsondata):
         return cls(
-            readwrite    = jsondata['readwrite'],
-            readonly     = jsondata['readonly'],
             identifier   = jsondata['identifier'],
+            readwrite    = sorted(jsondata['readwrite']),
+            readonly     = sorted(jsondata['readonly']),
         )