From: Richard Purdie <richard.purdie@linuxfoundation.org>
Date: Tue, 26 May 2015 10:41:02 +0000 (+0100)
Subject: bitbake: data: Clean up expandKeys None value handling
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=30cba374ccf4db59d5d1933a53057941c8d02e10;p=thirdparty%2Fopenembedded%2Fopenembedded-core-contrib.git

bitbake: data: Clean up expandKeys None value handling

This comparison is interesting even in the case of empty vales. Enabling
this warning actually found a bug in the metadata in avahi. Make the
code handle None specifically and also remove the dead code path in the
second if statement.

(Bitbake rev: a4cd4c56284812efb2a2bc0c8667ddad073f8e94)

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---

diff --git a/bitbake/lib/bb/data.py b/bitbake/lib/bb/data.py
index 218998a0af0..f8dc976a1c8 100644
--- a/bitbake/lib/bb/data.py
+++ b/bitbake/lib/bb/data.py
@@ -157,9 +157,9 @@ def expandKeys(alterdata, readdata = None):
             continue
         todolist[key] = ekey
         newval = alterdata.getVar(ekey, 0)
-        if newval:
+        if newval is not None:
             val = alterdata.getVar(key, 0)
-            if val is not None and newval is not None:
+            if val is not None:
                 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