[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: support for SunPRO C/C++ on Linux
From: |
Ralf Wildenhues |
Subject: |
Re: support for SunPRO C/C++ on Linux |
Date: |
Wed, 10 May 2006 18:17:03 +0200 |
User-agent: |
Mutt/1.5.11 |
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
- support for SunPRO C/C++ on Linux, Bruno Haible, 2006/05/08
- Re: support for SunPRO C/C++ on Linux, Ralf Wildenhues, 2006/05/08
- Re: support for SunPRO C/C++ on Linux, Bruno Haible, 2006/05/08
- Re: support for SunPRO C/C++ on Linux, Bruno Haible, 2006/05/10
- Re: support for SunPRO C/C++ on Linux, Bruno Haible, 2006/05/15
- Re: support for SunPRO C/C++ on Linux, Bruno Haible, 2006/05/15
- Re: support for SunPRO C/C++ on Linux, Ralf Wildenhues, 2006/05/15