summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Maw <richard.maw@codethink.co.uk>2014-01-20 14:31:59 (GMT)
committerRichard Maw <richard.maw@codethink.co.uk>2014-01-22 11:30:09 (GMT)
commite97bcc8c98377e70e99ace9975e650092311983b (patch)
tree1784b0beb5919e056bb21310c71c04e14d9a3f2a
parent585d50f2e0d2e7a8eaee2272c06094cce1d6df00 (diff)
downloadmorph-e97bcc8c98377e70e99ace9975e650092311983b.tar.gz
cache-key: Don't reduntantly hash products field
For chunks the products field doesn't need to be hashed, since the split-rules field is used instead, and includes the default rule set, and for strata and systems it is handled by hashing the dependencies.
-rw-r--r--morphlib/cachekeycomputer.py8
1 files changed, 6 insertions, 2 deletions
diff --git a/morphlib/cachekeycomputer.py b/morphlib/cachekeycomputer.py
index 2312abc..1a20faf 100644
--- a/morphlib/cachekeycomputer.py
+++ b/morphlib/cachekeycomputer.py
@@ -97,8 +97,12 @@ class CacheKeyComputer(object):
le_dict = dict((k, morphology[k]) for k in morphology.keys())
# Disregard all fields of a morphology that aren't important
- ignored_fields = ('strata', 'build-depends', 'description',
- 'chunks')
+ ignored_fields = (
+ 'description', # purely cosmetic, doesn't change builds
+ # The following are used to determine dependencies,
+ # so are already handled by the 'kids' field.
+ 'strata', 'build-depends', 'chunks',
+ 'products')
for ignored_field in ignored_fields:
if ignored_field in le_dict:
del le_dict[ignored_field]