libtool-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [REBASED PATCH 6/9] teach libtool -L= and -R=


From: Paolo Bonzini
Subject: Re: [REBASED PATCH 6/9] teach libtool -L= and -R=
Date: Thu, 29 Jul 2010 01:21:28 +0200
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.10) Gecko/20100621 Fedora/3.0.5-1.fc13 Lightning/1.0b2pre Thunderbird/3.0.5

On 07/29/2010 01:20 AM, Charles Wilson wrote:
On 7/28/2010 6:25 PM, Paolo Bonzini wrote:
On 07/22/2010 08:36 PM, Charles Wilson wrote:
@@ -5777,11 +5812,13 @@ func_mode_link ()
for deplib in $dependency_libs; do
newdependency_libs="$deplib $newdependency_libs"
if $opt_preserve_dup_deps ; then
+ func_resolve_sysroot "$deplib"
+ deplib=$func_resolve_sysroot_result
case "$tmp_libs " in
*" $deplib "*) func_append specialdeplibs " $deplib" ;;
esac
+ func_append tmp_libs " $deplib"
fi
- func_append tmp_libs " $deplib"
done

if test "$link_all_deplibs" != no; then

Any reason why you moved the func_append within the if?

No, that's a mistake. The rebase was kinda difficult, because since last
July all the foo="$foo $other" stuff got replaced with func_append. So,
the before-and-after diffs were a little hard to interpret; I tried to
be pretty careful, but I'm not surprised that a mistake got thru.

Do you think this might be the reason for the new:041 failure?

No, but I was going to post the fixed version in a minute.

Paolo



reply via email to

[Prev in Thread] Current Thread [Next in Thread]