Re: [REBASED PATCH 0/9] Paolo's sysroot patches, rebased plus some fixes

[Paolo Bonzini]

On 07/28/2010 07:35 PM, Charles Wilson wrote:
> $ diff good-041/inst/lib/liba.la bad-041/inst/lib/liba.la
> 20c20
> < dependency_libs='
> -R/usr/src/packages/libtool/git/build-bisect/tests/testsuite.dir/041/foobar'
>
> ---
>  > dependency_libs=''
>
> $ diff good-041/inst/lib/libb.la bad-041/inst/lib/libb.la
> 20c20
> < dependency_libs='
> -R/usr/src/packages/libtool/git/build-bisect/tests/testsuite.dir/041/foobar
> /usr/src/packages/libtool/git/build-bisect/tests/testsuite.dir/041/inst/lib/liba.la'
>
> ---
>  > dependency_libs='
> /usr/src/packages/libtool/git/build-bisect/tests/testsuite.dir/041/inst/lib/liba.la
> /usr/src/packages/libtool/git/build-bisect/tests/testsuite.dir/041/inst/lib/liba.la'

There you go.  Two problems here

+ func_append newdependency_libs " ${lt_sysroot:+=}$libdir/$name"
+ newdependency_libs="$newdependency_libs ${lt_sysroot:+=}$libdir/$name"

Double line. Actually not covered by new:41.

+ ;;
+      -L*)
+ func_stripname -L '' "$i"
+ func_replace_sysroot "$func_stripname_result"
+ func_append newdeplibs " -L$func_replace_sysroot_result"

s/newdeplibs/newdependency_libs/ -- this was in my patchset too

+ ;;
+      -R*)
+ func_stripname -R '' "$i"
+ func_replace_sysroot "$func_stripname_result"
+ func_append newdeplibs " -R$func_replace_sysroot_result"

s/newdeplibs/newdependency_libs/ -- also in my patchset.

This should fix it.  I'll send an updated patchset soon.

Paolo

0002-fix.patch
Description: Text document