Re: support for SunPRO C/C++ on Linux

[Ralf Wildenhues]

Hi Bruno,

* Bruno Haible wrote on Wed, May 10, 2006 at 02:01:31PM CEST:
> 
> Here is a revised patch. I changed the recognition of the Sun compilers,
> and the whole_archive_flag_spec and postdeps, so that now all 112 tests PASS.

Cool.

> > > With this patch, the FAILs are turned into PASS; all tests PASS or SKIP.
> >
> > Which ones skip?
> 
> Good question. I had many SKIPs, but this was either because I had forgotten
> to copy a recent config.guess, or because I did
>       ./configure
>       make
>       make check
> - not knowing that after modifying libtool.m4, a simple "make" does not
> update the aclocal.m4 and configure files in the subdirectories;

Yes.  This issue has been fixed in CVS HEAD.  I won't backport it though.

Some notes:

> *** 3353,3358 ****
> --- 3353,3379 ----
>       # dependencies.
>       output_verbose_link_cmd='templist=`$CC -shared $CFLAGS -v 
> conftest.$objext 2>&1 | grep "ld"`; templist=`echo $templist | $SED 
> "s/\(^.*ld.*\)\( .*ld .*$\)/\1/"`; list=""; for z in $templist; do case $z in 
> conftest.$objext) list="$list $z";; *.$objext);; *) list="$list $z";;esac; 
> done; echo $list'
>       ;;
> +       *)
> +     if LC_ALL=C $CC -V 2>&1 | sed 1q | grep "Sun C" > /dev/null; then


If that LC_ALL=C was really necessary, then that is a bug.  Autoconf
resets the locale, and many configure tests depend on this.  Any reason
not to simplify this to something like this?
        case `$CC -V 2>&1 >/dev/null` in
        *Sun\ C*)

(several instances)


> +         _LT_AC_TAGVAR(whole_archive_flag_spec, 
> $1)='${wl}--whole-archive`new_convenience=; for conv in
> +$convenience\"\"; do test -z \"$conv\" || 
> new_convenience=\"$new_convenience,$conv\"; done; $echo 
> \"$new_convenience\"`+${wl}--no-whole-archive'

Are you sure the compiler driver won't reorder arguments here?  There
has been a significant fix for this on Solaris post-1.5.22 (on
2006-02-03, after several tries in the past), and only the CVS HEAD
Libtool testsuite exposes the known failures fully.  Related question:
are you volunteering for the forward-port of the patch?  (If not, I can
do it, but it'll take longer then.)

Rest looks good, except there will be issues mixing C++ libraries
compiled with different compilers (as expected).

Do you happen to know whether Sun changed their minds and offered this
compiler suite for free (as in beer) now?  So that I could integrate it
into testing..

Cheers,
Ralf