Openembedded Core Discussions
 help / color / mirror / Atom feed
* sstate: Fix SSTATE_DUPWHITELIST variable usage
@ 2012-09-26 11:45 Richard Purdie
  0 siblings, 0 replies; only message in thread
From: Richard Purdie @ 2012-09-26 11:45 UTC (permalink / raw)
  To: openembedded-core

We need to split this variable before using it. Otherwise a single "/"
character in the list whitelists every overlapping sysroot file which
was not the intention making the whole thing useless.

We'll start seeing warnings about overlapping files now this is working
correctly after this patch.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
index 6ccaf6d..0037ce5 100644
--- a/meta/classes/sstate.bbclass
+++ b/meta/classes/sstate.bbclass
@@ -144,7 +144,7 @@ def sstate_install(ss, d):
 
     # Check the file list for conflicts against the master manifest
     mastermanifest = d.getVar("SSTATE_MASTERMANIFEST", True)
-    whitelist = d.getVar("SSTATE_DUPWHITELIST", True)
+    whitelist = (d.getVar("SSTATE_DUPWHITELIST", True) or "").split()
     lock = bb.utils.lockfile(mastermanifest + ".lock")
     if not os.path.exists(mastermanifest):
         open(mastermanifest, "w").close()





^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2012-09-26 11:58 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-26 11:45 sstate: Fix SSTATE_DUPWHITELIST variable usage Richard Purdie

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox