diff options
author | Kyle Russell <bkylerussell@gmail.com> | 2019-12-06 17:10:00 -0500 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2019-12-12 20:02:51 +0000 |
commit | 9155e70299b3f8e25413a24ca3d1a60fe46f5c6f (patch) | |
tree | c513d2807077b0d326d2532c09e2196dac8df098 /bitbake | |
parent | 8cd3ee6e1a50ad9f40466bcadb236c619c42ef19 (diff) | |
download | poky-9155e70299b3f8e25413a24ca3d1a60fe46f5c6f.tar.gz |
bitbake: runqueue: fix multiconfig task dependency filtering
multiconfig dependencies should be excluded from BB_TASKDEPDATA.
However in thud, multiconfig filtering on task dependencies doesn't
happen until after deps has already been added to taskdepdata.
One manifestation of this results in multiconfig dependencies leaking
into staging processing.
File: 'exec_python_func() autogenerated', lineno: 2, function: <module>
0001:
*** 0002:extend_recipe_sysroot(d)
0003:
File: '/home/user/thud/meta/classes/staging.bbclass', lineno: 344, function: extend_recipe_sysroot
0340: #bb.note(" start is %s" % str(start))
0341:
0342: # Direct dependencies should be present and can be depended upon
0343: for dep in set(start):
*** 0344: if setscenedeps[dep][1] == "do_populate_sysroot":
0345: if dep not in configuredeps:
0346: configuredeps.append(dep)
0347: bb.note("Direct dependencies are %s" % str(configuredeps))
0348: #bb.note(" or %s" % str(start))
Exception: KeyError: 'multiconfig:musl:/home/user/thud/meta/recipes-kernel/linux/linux-yocto_4.18.bb:do_deploy'
This can be reproduced on thud by backporting the multiconfig.MultiConfig.test_multiconfig
test and mcextend bbclass from warrior.
d22b6e03a5504145abed7c2ca44cf12854df85da mcextend: Add helper class useful for multiconfig
d9018a3d9c828551c465b68b27920ec4681524ae selftest: Add multiconfig test
Flipping the ordering to match warrior's behavior fixes the test case.
(Bitbake rev: b690030efc87850951e8e3ecf4ae3c1dd1dc9b63)
Signed-off-by: Kyle Russell <bkylerussell@gmail.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'bitbake')
-rw-r--r-- | bitbake/lib/bb/runqueue.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/bitbake/lib/bb/runqueue.py b/bitbake/lib/bb/runqueue.py index 383c183235..0f2fdceed8 100644 --- a/bitbake/lib/bb/runqueue.py +++ b/bitbake/lib/bb/runqueue.py | |||
@@ -2109,8 +2109,8 @@ class RunQueueExecuteTasks(RunQueueExecute): | |||
2109 | deps = self.rqdata.runtaskentries[revdep].depends | 2109 | deps = self.rqdata.runtaskentries[revdep].depends |
2110 | provides = self.rqdata.dataCaches[mc].fn_provides[taskfn] | 2110 | provides = self.rqdata.dataCaches[mc].fn_provides[taskfn] |
2111 | taskhash = self.rqdata.runtaskentries[revdep].hash | 2111 | taskhash = self.rqdata.runtaskentries[revdep].hash |
2112 | taskdepdata[revdep] = [pn, taskname, fn, deps, provides, taskhash] | ||
2113 | deps = self.filtermcdeps(task, deps) | 2112 | deps = self.filtermcdeps(task, deps) |
2113 | taskdepdata[revdep] = [pn, taskname, fn, deps, provides, taskhash] | ||
2114 | for revdep2 in deps: | 2114 | for revdep2 in deps: |
2115 | if revdep2 not in taskdepdata: | 2115 | if revdep2 not in taskdepdata: |
2116 | additional.append(revdep2) | 2116 | additional.append(revdep2) |