[bitbake-devel] [PATCH] data: Move warning code to the first loop for performance
Bernhard Reutner-Fischer
rep.dot.nop at gmail.com
Tue May 26 09:56:10 UTC 2015
On May 25, 2015 9:53:52 AM GMT+02:00, Richard Purdie <richard.purdie at linuxfoundation.org> wrote:
>By doing this we can take advantage of the expansion cache before
>starting write operations on the data store.
>
>Signed-off-by: Richard Purdie <richard.purdie at linuxfoundation.org>
>
>diff --git a/bitbake/lib/bb/data.py b/bitbake/lib/bb/data.py
>index 82eefef..218998a 100644
>--- a/bitbake/lib/bb/data.py
>+++ b/bitbake/lib/bb/data.py
>@@ -156,17 +156,17 @@ def expandKeys(alterdata, readdata = None):
> if key == ekey:
> continue
> todolist[key] = ekey
>+ newval = alterdata.getVar(ekey, 0)
>+ if newval:
>+ val = alterdata.getVar(key, 0)
>+ if val is not None and newval is not None:
newval cannot ever be None in this if.
Thanks,
>+ bb.warn("Variable key %s (%s) replaces original key %s
>(%s)." % (key, val, ekey, newval))
>
> # These two for loops are split for performance to maximise the
> # usefulness of the expand cache
>
> for key in todolist:
> ekey = todolist[key]
>- newval = alterdata.getVar(ekey, 0)
>- if newval:
>- val = alterdata.getVar(key, 0)
>- if val is not None and newval is not None:
>- bb.warn("Variable key %s (%s) replaces original key %s
>(%s)." % (key, val, ekey, newval))
> alterdata.renameVar(key, ekey)
>
> def inheritFromOS(d, savedenv, permitted):
More information about the bitbake-devel
mailing list