]> 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>
Wed, 11 Feb 2015 17:39:48 +0000 (17:39 +0000)
commit78a2eeea4e2ef867437c315337b9188e1f3fa759
tree4749c405ead31ffded3a3ebe45eaec9f4b905bd7
parent72ff97a2ec225bafb83be56ca1b8c3c4e68a0c55
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.

(From OE-Core rev: 85ff3d6491c54aa712ed238c561742cda4f4ba07)

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