* [PATCH] data: Make expandKeys deterministic
@ 2015-05-27 16:37 Richard Purdie
0 siblings, 0 replies; only message in thread
From: Richard Purdie @ 2015-05-27 16:37 UTC (permalink / raw)
To: bitbake-devel
At least in theory, the order the keys are expanded in can make
a difference, particularly if there is key overlap.
We also want to ensure that any underlying base key is processed
before any overridden version of that variable (FOO before FOO_x)
which helps the update_data removal code I've been testing.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
diff --git a/bitbake/lib/bb/data.py b/bitbake/lib/bb/data.py
index 439c365..84f5355 100644
--- a/bitbake/lib/bb/data.py
+++ b/bitbake/lib/bb/data.py
@@ -159,7 +159,7 @@ def expandKeys(alterdata, readdata = None):
# These two for loops are split for performance to maximise the
# usefulness of the expand cache
- for key in todolist:
+ for key in sorted(todolist):
ekey = todolist[key]
newval = alterdata.getVar(ekey, 0)
if newval is not None:
--
cgit v0.10.2
^ permalink raw reply related [flat|nested] only message in thread
only message in thread, other threads:[~2015-05-27 16:37 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-05-27 16:37 [PATCH] data: Make expandKeys deterministic Richard Purdie
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox