diff options
| author |   Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-09-13 08:50:26 +0100 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2014-09-13 08:51:49 +0100 |
| commit | 72a1ca4a104ccab73d6abcbd44db9c2636a58572 (patch) | |
| tree | bf9a5789dcce755892db7f39b3928ab88972a979 | |
| parent | e24095f54c52a547c0462836586a5d716249036e (diff) | |
| download | bitbake-72a1ca4a104ccab73d6abcbd44db9c2636a58572.tar.gz | |
data_smart: Fix remove operator and its interaction with data expansion
If you have:
FOO = "${bindir}/X Y"
FOO_remove = "${bindir}/X"
the expected result is "Y". Currently this doesn't work since the removed
expressions are not expanded first. This patch adjusts things so the
expressions are expanded before being processed for removal.
Also add a test to ensure this case continues to work.
[YOCTO #6624]
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | lib/bb/data_smart.py | 3 | ||||
| -rw-r--r-- | lib/bb/tests/data.py | 7 |
2 files changed, 9 insertions, 1 deletions
diff --git a/lib/bb/data_smart.py b/lib/bb/data_smart.py index 3d773b1d6..cf8919c64 100644 --- a/lib/bb/data_smart.py +++ b/lib/bb/data_smart.py @@ -616,7 +616,8 @@ class DataSmart(MutableMapping): cachename = var + "[" + flag + "]" value = self.expand(value, cachename) if value and flag == "_content" and local_var is not None and "_removeactive" in local_var: - filtered = filter(lambda v: v not in local_var["_removeactive"], + removes = [self.expand(r) for r in local_var["_removeactive"]] + filtered = filter(lambda v: v not in removes, value.split(" ")) value = " ".join(filtered) if expand: diff --git a/lib/bb/tests/data.py b/lib/bb/tests/data.py index 944a90665..9b09ff4c6 100644 --- a/lib/bb/tests/data.py +++ b/lib/bb/tests/data.py @@ -259,6 +259,13 @@ class TestConcatOverride(unittest.TestCase): bb.data.update_data(self.d) self.assertEqual(self.d.getVar("TEST", True), "") + def test_remove_expansion(self): + self.d.setVar("BAR", "Z") + self.d.setVar("TEST", "${BAR}/X Y") + self.d.setVar("TEST_remove", "${BAR}/X") + bb.data.update_data(self.d) + self.assertEqual(self.d.getVar("TEST", True), "Y") + class TestOverrides(unittest.TestCase): def setUp(self): self.d = bb.data.init() |