3874 2013-02-11 16:35:03 +0000 Overrides are evaluated for primary and multilib configuration, causing potential conflicts 2013-02-14 10:15:17 +0000 1 1 7 Build System, Metadata & Runtime OE-Core core 1.4 x86 Multiple RESOLVED FIXED Medium+ major 1.4 M4 1 mark.hatle richard.purdie meta.mr.watcher meta.watcher --- --- --- oldest_to_newest 29714 0 mark.hatle 2013-02-11 16:35:03 +0000 Reproducer: cd meta/recipes-devtools/valgrind touch valgrind-3.8.1/powerpc.patch touch valgrind-3.8.1/powerpc64.patch edit valgrind_3.8.1.bb +SRC_URI_append_powerpc = " file://powerpc.patch" +SRC_URI_append_powerpc64 = " file://powerpc64.patch" cd meta/conf/machine cp qemuppc.conf qemuppc64.conf edit qemuppc64.conf: -require conf/machine/include/tune-ppc603e.inc +require conf/machine/include/tune-ppce5500.inc (Note, this will not result in functional machine, but it's enough to reproduce the problem!) modify your conf/local.conf MACHINE = "qemuppc64.conf" require conf/multilib.conf MULTILIBS = "multilib:lib64" DEFAULTTUNE_virtclass-multilib-lib64 = "ppc64e5500" To see the problem: bitbake -e valgrind | grep "^SRC_URI=" SRC_URI="http://www.valgrind.org/downloads/valgrind-3.8.1.tar.bz2 file://fix_issue_caused_by_ccache.patch file://fix_unsupporting_kernel_3.patch file://fixed-perl-path.patch file://Added-support-for-PPC-instructions-mfatbu-mfatbl.patch file://configure-with-newer-glibc.patch file://powerpc.patch" (that is correct) bitbake -e lib64-valgrind | grep "^SRC_URI=" SRC_URI="http://www.valgrind.org/downloads/valgrind-3.8.1.tar.bz2 file://fix_issue_caused_by_ccache.patch file://fix_unsupporting_kernel_3.patch file://fixed-perl-path.patch file://Added-support-for-PPC-instructions-mfatbu-mfatbl.patch file://configure-with-newer-glibc.patch file://powerpc.patch file://powerpc64.patch" (that is incorrect, both powerpc and powerpc64 are listed!) The 'OVERRIDES=' values for both are correct... What appears to be happening is SRC_URI is evaluated twice. 29719 1 mark.hatle 2013-02-11 19:59:33 +0000 After some investigation. It appears that the problem is related to the order in which bitbake processes various anonymous python chunks, specifically in: bitbake/lib/bb/parse/ast.py: def finalize(fn, d, variant = None): all_handlers = {} for var in d.getVar('__BBHANDLERS') or []: ... bb.event.fire(bb.event.RecipePreFinalise(fn), d) bb.data.expandKeys(d) bb.data.update_data(d) code = [] for funcname in d.getVar("__BBANONFUNCS") or []: code.append("%s(d)" % funcname) bb.utils.better_exec("\n".join(code), {"d": d}) bb.data.update_data(d) ... The first update_data sets the values (and appends) using the default tune, while the second sets the values based on the updated OVERRIDES settings from the anonfuncs. 29723 2 mark.hatle 2013-02-11 20:03:41 +0000 Possible workaround (fix?) from Richard Purdie: diff --git a/meta/classes/multilib.bbclass b/meta/classes/multilib.bbclass index 5454b4c..d52f721 100644 --- a/meta/classes/multilib.bbclass +++ b/meta/classes/multilib.bbclass @@ -51,6 +51,11 @@ python multilib_virtclass_handler () { e.data.setVar("PN", variant + "-" + e.data.getVar("PN", False)) e.data.setVar("SHLIBSDIR_virtclass-multilib-" + variant ,e.data.getVar("SHL e.data.setVar("OVERRIDES", e.data.getVar("OVERRIDES", False) + override) + + # Workaround for issue in YP Bugzilla #3874 + newtune = e.data.getVar("DEFAULTTUNE_" + "virtclass-multilib-" + variant) + if newtune: + e.data.setVar("DEFAULTTUNE", newtune) } addhandler multilib_virtclass_handler 29732 3 richard.purdie 2013-02-11 22:43:12 +0000 The issue is that OVERRIDES has virtclass-multilib-xxxx added to it, this eventually changed DEFAULTTUNE which then changes TRANSLATED_TARGET_ARCH which is in OVERRIDES meaning we then need to reevaulate the overides and the TTA gets applied twice since once you apply an override, it doesn't get undone. Expanding DEFAULTTUNE to the correct value in advance avoids the issue and means only the correct overrides get applied. 29789 4 richard.purdie 2013-02-14 10:15:17 +0000 Fixed in master, http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=e3f5d7dee070464c636c4cb38844e098ec9d75f8