utils: drop exported var env caching for oe_popen
authorChris Larson <chris_larson@mentor.com>
Mon, 14 Feb 2011 17:41:50 +0000 (12:41 -0500)
committerChris Larson <chris_larson@mentor.com>
Mon, 14 Feb 2011 17:41:52 +0000 (12:41 -0500)
This was dangerous, as an oe_run executed from the configuration metadata
would result in caching the config metadata information, rather than the final
metadata information.  We could also cache it by the datasmart instance, but I
don't think its worth the trouble at this time, particularly given the next
bitbake release exports all exported vars into the process environment
automatically, avoiding the need for any of this.

Signed-off-by: Chris Larson <chris_larson@mentor.com>
classes/utils.bbclass

index eea084a..7591d47 100644 (file)
@@ -62,13 +62,10 @@ def is_machine_specific(d):
             return True
 
 def oe_popen_env(d):
-    env = d.getVar("__oe_popen_env", False)
-    if env is None:
-        env = {}
-        for v in d.keys():
-            if d.getVarFlag(v, "export"):
-                env[v] = d.getVar(v, True) or ""
-        d.setVar("__oe_popen_env", env)
+    env = {}
+    for v in d.keys():
+        if d.getVarFlag(v, "export"):
+            env[v] = d.getVar(v, True) or ""
     return env
 
 def oe_run(d, cmd, **kwargs):