diff options
| author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-04-19 21:52:10 +0000 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-04-24 15:51:37 +0100 |
| commit | 1efa999ed02186c9c58cb48418e0c62ecdc82ac7 (patch) | |
| tree | 08c7ba2af736ea67107eb9336444b84af819615a /meta | |
| parent | 7298e0c48198587d41319d0068aa3511afa6d1c5 (diff) | |
| download | openembedded-core-1efa999ed02186c9c58cb48418e0c62ecdc82ac7.tar.gz openembedded-core-1efa999ed02186c9c58cb48418e0c62ecdc82ac7.tar.bz2 openembedded-core-1efa999ed02186c9c58cb48418e0c62ecdc82ac7.zip | |
utils.bbclass: Testing via env in create_wrapper is a nice idea but breaks things
For example, pseudo-native wants to set LD_LIBRBARY_PATH but setting this
into the environment here causes the existing pseudo (running during do_install)
to poke into paths in /opt and this breaks builds.
The simplest fix is simply not to do this. Comments tweaks to match the code.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta')
| -rw-r--r-- | meta/classes/utils.bbclass | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/meta/classes/utils.bbclass b/meta/classes/utils.bbclass index 3b5946308c..fde8f446ad 100644 --- a/meta/classes/utils.bbclass +++ b/meta/classes/utils.bbclass @@ -282,7 +282,6 @@ create_cmdline_wrapper () { cmd=$1 shift - # run echo via env to test syntactic validity of the variable arguments echo "Generating wrapper script for $cmd" mv $cmd $cmd.real @@ -306,8 +305,7 @@ create_wrapper () { cmd=$1 shift - # run echo via env to test syntactic validity of the variable arguments - env $@ echo "Generating wrapper script for $cmd" + echo "Generating wrapper script for $cmd" mv $cmd $cmd.real cmdname=`basename $cmd`.real |