Re: [PATCH] change order of -L options when relinking

[Charles Wilson]

But add_dir gets added to deplibs, and then deplibs is reversed... 
Here's the sequence of events WITH this patch (scroll down to **** for 
the important bits):


 /bin/bash ../libtool --mode=install /usr/bin/install -c  libtwo.la 
/usr/src/libtool/devel/DEMO/destdir-relinklib-demo-1.0.1/inst/usr/lib/libtwo.la

libtool: install: warning: relinking `libtwo.la'

(cd /usr/src/libtool/devel/DEMO/destdir-relinklib-demo-1.0.1/libtwo; 
/bin/bash ../libtool --mode=relink gcc -g -O2 -o libtwo.la -rpath 
/usr/lib -version-info 2:1:1 -no-undefined two.lo ../libone/libone.la 
-inst-prefix-dir 
/usr/src/libtool/devel/DEMO/destdir-relinklib-demo-1.0.1/inst)

gcc -shared  .libs/two.o 
-L/usr/src/libtool/devel/DEMO/destdir-relinklib-demo-1.0.1/inst/usr/lib 
-L/usr/lib -lone  -o .libs/cygtwo-1.dll -Wl,--image-base=0x10000000 
-Wl,--out-implib,.libs/libtwo.dll.a

Creating library file: .libs/libtwo.dll.a
Info: resolving _one_global_int_var by linking to 
__imp__one_global_int_var (auto-import)

/usr/bin/install -c .libs/libtwo.dll.aT 
/usr/src/libtool/devel/DEMO/destdir-relinklib-demo-1.0.1/inst/usr/lib/libtwo.dll.a
base_file=`basename ${file}`
 dlpath=`/bin/bash 2>&1 -c '. .libs/'${base_file}'i;echo $dlname'`

dldir=/usr/src/libtool/devel/DEMO/destdir-relinklib-demo-1.0.1/inst/usr/lib/`dirname 
$dlpath`
 test -d $dldir || mkdir -p $dldir
 /usr/bin/install -c .libs/cygtwo-1.dll $dldir/cygtwo-1.dll
/usr/bin/install -c .libs/libtwo.lai 
/usr/src/libtool/devel/DEMO/destdir-relinklib-demo-1.0.1/inst/usr/lib/libtwo.la
libtool: install: warning: remember to run `libtool --finish /usr/lib'

*********************
with patch:

gcc -shared  .libs/two.o 
-L/usr/src/libtool/devel/DEMO/destdir-relinklib-demo-1.0.1/inst/usr/lib 
-L/usr/lib -lone  -o .libs/cygtwo-1.dll -Wl,--image-base=0x10000000 
-Wl,--out-implib,.libs/libtwo.dll.a

**********************
without patch:

 gcc -shared  .libs/two.o  -L/usr/lib 
-L/usr/src/libtool/devel/DEMO/tmp/destdir-
relinklib-demo-1.0.1/inst/usr/lib -lone  -o .libs/cygtwo-1.dll 
-Wl,--image-base=
0x10000000 -Wl,--out-implib,.libs/libtwo.dll.a

--
Chuck


Howard Chu wrote:

> Surely the patch description below is in the wrong order; using
> "-L/usr/lib -L/tmp/usr/lib" is the current incorrect behavior. Yes? -L search
> paths are used left-to-right.