137762 - Linker Error with Sun Forte Compile against gkgfx

Status: VERIFIED FIXED

(SeaMonkey :: Build Config, defect)

Description

[km]

I get

CC -I/usr/openwin/include -mt  -DNDEBUG -DTRIMMED -O  -G -Qoption ld -z,muldefs
-h libimggif.so -o libimggif.so  GIF2.o nsGIFDecoder2.o nsGIFModule.o   
-xildoff     -lgkgfx -L../../../../dist/bin -lxpcom -L../../../../dist/bin
-L/var/tmp/moz/mozilla/dist/lib -lplds4 -lplc4 -lnspr4 -lpthread -ldl -lrt   
-lsocket -ldl -lm    
ld: fatal: library -lgkgfx: not found
ld: fatal: File processing errors. No output written to libimggif.so


The problem is that Sun CC applies the -L paths only to -l libraries
listed later in the command line.  libgkgfx is in dist/bin but that
occurs only later in the command line.

This problem does not show with gcc, since gcc aggregates all the 
-L directories at the front of the command line it presents to the
Solaris linker.

This only broke in the last couple of weeks, so the link command line
must have changed.

A workaround is to set the LD_LIBRARY_PATH for the build to include dist/bin

Comment 1

[hacker formerly known as [e-mail]]

Out of curiosity, what version of Forte?  Our WS6U2 box is still green.

Comment 2

[km]

Good point. I'm now using

CC: Forte Developer 7 C++ 5.4 DEV 2002/02/13

and after checking I see that its behavior is different that WS6U2.
WS6U2 and previous Sun comilers, act like gcc. They collect all the -L's
and move them to the beginning of the ld command line. WS7 passes
them through to ld as is.

Don't know if this is a bug in WS7 or a new feature.

Comment 3

[hacker formerly known as [e-mail]]

Attachment: Use LIBS_DIR before EXTRA_DSO_LIBS

The bustage was caused by a fix I proposed for bug 53597.  This patch should
fix the link problem.

Comment 4

[Donnie Cranford]

This patched went through fine on my Solaris 8 machine running Forte 7 and fixed
the build bustage, 

thanks Dauphin

r=dcran

-dcran-

Comment 5

[hacker formerly known as [e-mail]]

Patch has been checked in.

Comment 6

[km]

The same issue has come up in a new directory

    extensions/typeaheadfind/src

The same additions need to be made to its Makefile.in.

Comment 7

[hacker formerly known as [e-mail]]

Don't reopen this bug everytime this happens.  Inform whomever landed the
changes that they need to be aware of this problem and open a new bug if
necessary.  Propagate the knowledge!

Comment 8

[hacker formerly known as [e-mail]]

.

Comment 9

[Tracy Walker [:tracy]]

verified