Ralf Wildenhues <ralf.wildenh...@gmx.de> writes:

>> the Automake manual can be read otherwise: ch. 8.3.7 `_LIBADD',
>> `_LDFLAGS', and `_LIBTOOLFLAGS' states:
>> 
>>   As shown in previous sections, the `LIBRARY_LIBADD' variable should be
>>   used to list extra libtool objects (`.lo' files) or libtool libraries
>>   (`.la') to add to LIBRARY.
>> 
>>      The `LIBRARY_LDFLAGS' variable is the place to list additional
>>   libtool linking flags, such as `-version-info', `-static', and a lot
>>   more.  *Note Link mode: (libtool)Link mode.
>> 
>> There's nothing about external libraries specified with just -l<lib> in
>> the first sentence about _LIBADD.
>
> Well, later, in `Program and Library Variables', there is:
>
>      `_LDADD' and `_LIBADD' are inappropriate for passing
>      program-specific linker flags (except for `-l', `-L', `-dlopen'
>      and `-dlpreopen').  Use the `_LDFLAGS' variable for this purpose.
>
> which hints at that -l is OK here.  But independently of what's in the
> Automake manual, it is impossible to solve the ordering issues for
> static linking at the autotools level if -l can not be passed to
> _LIBADD, so we should improve the Automake manual if that is not
> explicit enough.

I'd appreciate that.

>> As I said, I could be persuaded to change this, but right now my primary
>> interest is to get Solaris back to bootstrap land ASAP, and with less
>> than 13000 testsuite failures ;-(
>
> Does this have to do anything with the issue at hand?

The -lm in _LIBADD currently breaks the bootstrap.  Unless there are
strong objections, I'll shortly commit a patch to move it to _LDFLAGS
for the time being (and fix the various issues with the quadmath.map-sun
target at the same time).

The testsuite failures are caused by some change in gcc/testsuite/lib;
haven't yet looked which one.

        Rainer

-- 
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University

Reply via email to