Bug 9197 - DEFAULTTUNE_virtclass-multilib-lib32 = "corei7-32" results in errors
Summary: DEFAULTTUNE_virtclass-multilib-lib32 = "corei7-32" results in errors
Status: RESOLVED FIXED
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: configuration (show other bugs)
Version: 2.0
Hardware: x86 Multiple
: Medium+ normal
Target Milestone: 2.1 M3
Assignee: Richard Purdie
QA Contact: Valentin Hangan
URL:
Whiteboard: Rebecca: Need info to backport to Jethro
Depends on:
Blocks:
 
Reported: 2016-03-02 05:48 UTC by Anuj Mittal
Modified: 2016-05-04 11:55 UTC (History)
9 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Anuj Mittal 2016-03-02 05:48:32 UTC
Error log when compiling:

ERROR: ExpansionError during parsing /home/ilab/test-build/yocto_build/meta-openembedded/meta-oe/recipes-core/meta/distro-feed-configs.bb: Failure expanding variable CONFFILES_lib32-distro-feed

Added following to local.conf:

require conf/multilib.conf
DEFAULTTUNE = "corei7-64"
MULTILIBS = "multilib:lib32"
DEFAULTTUNE_virtclass-multilib-lib32 = "corei7-32"

The problem is because there is no "core2" tune.

diff --git a/meta/conf/machine/include/tune-corei7.inc b/meta/conf/machine/include/tune-corei7.inc
index 42cf932..bea652f 100644
--- a/meta/conf/machine/include/tune-corei7.inc
+++ b/meta/conf/machine/include/tune-corei7.inc
@@ -20,7 +20,7 @@ AVAILTUNES += "corei7-32"
 TUNE_FEATURES_tune-corei7-32 = "${TUNE_FEATURES_tune-x86} corei7"
 BASE_LIB_tune-corei7-32 = "lib"
 TUNE_PKGARCH_tune-corei7-32 = "corei7-32"
-PACKAGE_EXTRA_ARCHS_tune-corei7-32 = "${PACKAGE_EXTRA_ARCHS_tune-core2} corei7-32"
+PACKAGE_EXTRA_ARCHS_tune-corei7-32 = "${PACKAGE_EXTRA_ARCHS_tune-core2-32} corei7-32"
Comment 1 Anuj Mittal 2016-03-02 09:03:11 UTC
Forgot to mention explicitly that this happens on Jethro only when meta-openembedded/meta-oe is included while expanding:

CONFFILES_${PN} += '${@ " ".join( [ ( "${sysconfdir}/opkg/%s-feed.conf" % feed ) for feed in "all ${PACKAGE_EXTRA_ARCHS} ${MACHINE_ARCH}".split() ] ) }'

in distro-feed-configs.bb

I tried it on master and it doesn't give an error anymore but it still doesn't seem correct and should be fixed.
Comment 2 Rebecca Chang 2016-03-04 03:03:44 UTC
I took the ownership of this issue. Our team will be responsible to make sure the changes go upstream.
We will notify Poky owner to merge once the patch is available in OE-Core.
Comment 3 Rebecca Chang 2016-03-08 00:48:27 UTC
Patch submitted to OE-Core mailing list and pending review.

http://lists.openembedded.org/pipermail/openembedded-core/2016-March/118403.html
Comment 4 Rebecca Chang 2016-03-14 06:12:03 UTC
Patch has been merged into OE-Core master branch.
http://git.openembedded.org/openembedded-core/commit/?id=0903d6f0098f112d4263812df109e0c44c166db8

Please share the procedure to get it backport to Jethro. The patch was sent with [master][jethro] tag at the Subject. However it is not merge into jethro.