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.

(From OE-Core rev: 1efa999ed02186c9c58cb48418e0c62ecdc82ac7)

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Richard Purdie 2012-04-19 21:52:10 +00:00
parent 0816a3f355
commit 7ad49364fd
1 changed files with 1 additions and 3 deletions

View File

@ -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