Bug 12473

The recipe cache (activated by CACHE option) is not working if there are nested DataSmart objects. For example BB_ORIGENV is itself a DataSmart and also an entry in the main DataSmart object. The get_hash function calculates a hash based on the values of the DataSmart dict. For nested objects the value may not be persistent. In the case of the BB_ORIGENV the value contains the memory address of the object which is different each bitbake invocation. This causes the hash to change in each invocation. The hash is used in the file names of the cache files. Therefore the cache can never be used.

This problem exists in this project: https://github.com/openembedded/bitbake.git
Selectable versions in bugzilla don't match available tags in git so I didn't specify the version. The problem exists at least in the recent versions and on master.

Below patch seems to fix it. The idea is to make the get_hash function recursive in case of nested DataSmart objects and use their hash in the main hash. But I'm not sure if this is the best way to solve it. Someone more experienced with the bitbake implementation should take a look.

diff --git a/lib/bb/data_smart.py b/lib/bb/data_smart.py
index 7b09af5..c0e4228 100644
--- a/lib/bb/data_smart.py
+++ b/lib/bb/data_smart.py
@@ -1014,7 +1014,10 @@ class DataSmart(MutableMapping):
                 continue

             value = d.getVar(key, False) or ""
-            data.update({key:value})
+            if type(value) is type(self):
+                data.update({key:value.get_hash()})
+            else:
+                data.update({key:value})

             varflags = d.getVarFlags(key, internalflags = True)
             if not varflags