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
_______________________________________________
Bug-libtool mailing list
Bug-libtool@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-libtool
