]> code.ossystems Code Review - openembedded-core.git/commit
cross-canadian/meta-environment: Allow modification of TARGET_OS to be optional
authorRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 15 Jan 2015 10:03:50 +0000 (10:03 +0000)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Fri, 16 Jan 2015 09:13:44 +0000 (09:13 +0000)
commit85ff3d6491c54aa712ed238c561742cda4f4ba07
treea7284642bb029729c66dcd99514c3c231c972698
parent16562034a2c28cbfc6c90f9324c42c08e0655b7d
cross-canadian/meta-environment: Allow modification of TARGET_OS to be optional

There are some cases we want the manipulation cross-canadian performance
on TARGET_OS, there are also cases like meta-environment where we do not
want this manipulation.

We did try and use immediate expansion to avoid this problem and it
works in the non multilib case. If we have a multilib that used an
extension, like for example:

require conf/multilib.conf
MULTILIBS = "multilib:lib32 multilib:lib64"
DEFAULTTUNE = "mips32r2"
DEFAULTTUNE_virtclass-multilib-lib32 = "mips64-n32"
DEFAULTTUNE_virtclass-multilib-lib64 = "mips64"

then the n32 extension case will be misconfigured.

It turns out saving an unexpanded variable is hard. The best I could
come up with was:

SAVEDTOS := "${@d.getVar('TARGET_OS', False).replace("{", "*")}"

and then

localdata.setVar("TARGET_OS", d.getVar("SAVEDOS", False).replace('*','{'))

which is rather evil, I'd challenge someone to come up with a nicer way
of making it work though!

Rather than the above madness, we modify cross-canadian to make the
problamtic code conditional.

This fixes the original issue (where a linux-gnuspe target was seeing
'linux') of
http://cgit.openembedded.org/openembedded-core/commit/?id=0038634ee6e2b6035c023a2702547f20f67c103a
but also fixes the multilib one.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
meta/classes/cross-canadian.bbclass
meta/recipes-core/meta/meta-environment.bb