Test results for libtool-2.2.6b on Solaris 10 x86

PASS: tests/link.test
PASS: tests/link-2.test
PASS: tests/nomode.test
PASS: tests/objectlist.test
PASS: tests/quote.test
PASS: tests/sh.test
PASS: tests/suffix.test
PASS: tests/tagtrace.test
PASS: tests/cdemo-static.test
PASS: tests/cdemo-make.test
PASS: tests/cdemo-exec.test
PASS: tests/demo-static.test
PASS: tests/demo-make.test
PASS: tests/demo-exec.test
PASS: tests/demo-inst.test
PASS: tests/demo-unst.test
PASS: tests/depdemo-static.test
PASS: tests/depdemo-make.test
PASS: tests/depdemo-exec.test
PASS: tests/depdemo-inst.test
PASS: tests/depdemo-unst.test
PASS: tests/mdemo-static.test
PASS: tests/mdemo-make.test
PASS: tests/mdemo-exec.test
PASS: tests/mdemo-inst.test
PASS: tests/mdemo-unst.test
PASS: tests/cdemo-conf.test
PASS: tests/cdemo-make.test
PASS: tests/cdemo-exec.test
PASS: tests/demo-conf.test

libtool.m4 patch for LT_CMD_MAX_LEN on IRIX

Hi,

on IRIX 6.5.30 (latest release) the macro LT_CMD_MAX_LEN incorrectly detects
8k as maximum length, while IRIX supports 20k. The patch for libtool.m4 is
quite simple and uses getconf(1) (specified in POSIX.2-1992), which probably
works for a lot of other systems, too. As I only tested it on IRIX, I only
enabled it for IRIX.

This patch is necessary to build PHP5 which wants to link a lot of files.
Please include the attached patch in the next release of libtool.

Best regards,
Michael

_______________________________________________
Bug-libtool mailing list
Bug-libtool <at> gnu.org
http://lists.gnu.org/mailman/listinfo/bug-libtool

Re: libtool.m4 patch for LT_CMD_MAX_LEN on IRIX

Hello Michael,

thanks for the report.

* Michael Stapelberg wrote on Sat, Apr 17, 2010 at 11:38:40PM CEST:
> on IRIX 6.5.30 (latest release) the macro LT_CMD_MAX_LEN incorrectly detects
> 8k as maximum length, while IRIX supports 20k. The patch for libtool.m4 is
> quite simple and uses getconf(1) (specified in POSIX.2-1992), which probably
> works for a lot of other systems, too. As I only tested it on IRIX, I only
> enabled it for IRIX.
> 
> This patch is necessary to build PHP5 which wants to link a lot of files.
> Please include the attached patch in the next release of libtool.

I'm not sure I see why this patch should make a difference.  Before the
patch, git master of Libtool had no special case for IRIX in this macro,
which means 'getconf ARG_MAX' is tried first.  So, your patch should
make exactly zero difference.

If that is not the case for you, can you try to debug the issue?
Probably easiest to edit the configure script in question, wrap the
expanded LT_CMD_MAX_LEN code in 'set -x'/'set +x' and remove the
2>/dev/null from the getconf command to see an eventual failure.

Hope that helps.

Cheers,
Ralf

Re: Test results for libtool-2.2.6b on Solaris 10 x86

Hello Ken,

thanks for the bug report

* ken mays wrote on Sat, Apr 17, 2010 at 01:04:29AM CEST:
> PASS: tests/mdemo-conf.test
> PASS: tests/mdemo-make.test
> PASS: tests/mdemo-exec.test
> PASS: tests/mdemo-inst.test
> PASS: tests/mdemo-unst.test
> FAIL: tests/mdemo-dryrun.test
> PASS: tests/mdemo2-conf.test
> SKIP: tests/mdemo2-make.test
> SKIP: tests/mdemo2-exec.test
> PASS: tests/pdemo-conf.test
> FAIL: tests/pdemo-make.test
> SKIP: tests/pdemo-exec.test
> SKIP: tests/pdemo-inst.test

Can you rerun the tests verbosely and also run the new testsuite?
It should work like this

env TESTS="tests/mdemo-conf.test tests/mdemo-make.test
           tests/mdemo-exec.test tests/mdemo-inst.test
           tests/mdemo-unst.test tests/mdemo-dryrun.test
           tests/mdemo2-conf.test tests/mdemo2-make.test
           tests/mdemo2-exec.test
           tests/pdemo-conf.test tests/pdemo-make.test
           tests/pdemo-exec.test tests/pdemo-inst.test" \
    VERBOSE=yes make -e -k check

Re: libtool.m4 patch for LT_CMD_MAX_LEN on IRIX

Hi Ralf,

Excerpts from Ralf Wildenhues's message of 2010-04-19 22:19:31 +0200:
> I'm not sure I see why this patch should make a difference.  Before the
> patch, git master of Libtool had no special case for IRIX in this macro,
> which means 'getconf ARG_MAX' is tried first.  So, your patch should
> make exactly zero difference.
You are right, newer versions already contain the getconf command. Turns
out that php 5.3.1 shipped a configure which used an older version of libtool,
while php 5.3.2 contains a configure which uses a recent enough version.

So, everything fine with the most recent version of php5.

Best regards,
Michael

mailing list handling trivia

Hello,

whoever helps me with approving moderated emails on this list: First,
thank you very much!  Then, if you allow somebody new through, please
also hit the button that says something like "add this address to the
white list" or similar, so that there is never a need to manually
moderate somebody twice.  The button is on the very same web page.

Thanks,
Ralf

[libtool 2.2.6b] testsuite: 28 48 74 failed

Ralf,

This is the info from libtool 2.2.6b test on Solaris 10u8 x86
using the Sun Studio 11 C/C++ compiler.

Ken

      

_______________________________________________
Bug-libtool mailing list
Bug-libtool <at> gnu.org
http://lists.gnu.org/mailman/listinfo/bug-libtool

libtool.m4 usage of diff when comparing files

Instead of using diff, we can use cmp which more common than diff.
Reason is that I encountered a (custom) system without diff installed,
but with gcc, and had ./configure output that it could not run diff.

2010-04-24  Alfred M. Szmidt  <ams <at> gnu.org>

	* libltdl/m4/libtool.m4: Use cmp instead of diff.

--- libltdl/m4/libtool.m4.~1~	2009-11-16 14:11:59.000000000 +0100
+++ libltdl/m4/libtool.m4	2010-04-24 19:29:23.235097505 +0200
 <at>  <at>  -1426,7 +1426,7  <at>  <at> 
      # So say no if there are warnings other than the usual output.
      $ECHO "X$_lt_compiler_boilerplate" | $Xsed -e '/^$/d' >conftest.exp
      $SED '/^$/d; /^ *+/d' conftest.err >conftest.er2
-     if test ! -s conftest.er2 || diff conftest.exp conftest.er2 >/dev/null; then
+     if test ! -s conftest.er2 || cmp conftest.exp conftest.er2 >/dev/null; then
        $2=yes
      fi
    fi

Re: libtool.m4 usage of diff when comparing files

Sorry, the previous patch was munged, here is the correct one.

diff -ur libltdl/m4/libtool.m4 libltdl/m4/libtool.m4.~1~
--- libltdl/m4/libtool.m4	2010-04-24 19:35:27.074101220 +0200
+++ libltdl/m4/libtool.m4.~1~	2009-11-16 14:11:59.000000000 +0100
 <at>  <at>  -1426,7 +1426,7  <at>  <at> 
      # So say no if there are warnings other than the usual output.
      $ECHO "X$_lt_compiler_boilerplate" | $Xsed -e '/^$/d' >conftest.exp
      $SED '/^$/d; /^ *+/d' conftest.err >conftest.er2
-     if test ! -s conftest.er2 || cmp conftest.exp conftest.er2 >/dev/null; then
+     if test ! -s conftest.er2 || diff conftest.exp conftest.er2 >/dev/null; then
        $2=yes
      fi
    fi
 <at>  <at>  -1466,7 +1466,7  <at>  <at> 
        cat conftest.err 1>&AS_MESSAGE_LOG_FD
        $ECHO "X$_lt_linker_boilerplate" | $Xsed -e '/^$/d' > conftest.exp
        $SED '/^$/d; /^ *+/d' conftest.err >conftest.er2
-       if cmp conftest.exp conftest.er2 >/dev/null; then
+       if diff conftest.exp conftest.er2 >/dev/null; then
          $2=yes
        fi
      else
 <at>  <at>  -1880,7 +1880,7  <at>  <at> 
      # So say no if there are warnings
      $ECHO "X$_lt_compiler_boilerplate" | $Xsed -e '/^$/d' > out/conftest.exp
      $SED '/^$/d; /^ *+/d' out/conftest.err >out/conftest.er2
-     if test ! -s out/conftest.er2 || cmp out/conftest.exp out/conftest.er2 >/dev/null; then
+     if test ! -s out/conftest.er2 || diff out/conftest.exp out/conftest.er2 >/dev/null; then
        _LT_TAGVAR(lt_cv_prog_compiler_c_o, $1)=yes

Re: libtool.m4 usage of diff when comparing files

On Sat, 24 Apr 2010, Alfred M. Szmidt wrote:

> Sorry, the previous patch was munged, here is the correct one.

I assume that this patch is reversed?

Bob

>
> diff -ur libltdl/m4/libtool.m4 libltdl/m4/libtool.m4.~1~
> --- libltdl/m4/libtool.m4	2010-04-24 19:35:27.074101220 +0200
> +++ libltdl/m4/libtool.m4.~1~	2009-11-16 14:11:59.000000000 +0100
>  <at>  <at>  -1426,7 +1426,7  <at>  <at> 
>      # So say no if there are warnings other than the usual output.
>      $ECHO "X$_lt_compiler_boilerplate" | $Xsed -e '/^$/d' >conftest.exp
>      $SED '/^$/d; /^ *+/d' conftest.err >conftest.er2
> -     if test ! -s conftest.er2 || cmp conftest.exp conftest.er2 >/dev/null; then
> +     if test ! -s conftest.er2 || diff conftest.exp conftest.er2 >/dev/null; then
>        $2=yes
>      fi
>    fi
>  <at>  <at>  -1466,7 +1466,7  <at>  <at> 
>        cat conftest.err 1>&AS_MESSAGE_LOG_FD
>        $ECHO "X$_lt_linker_boilerplate" | $Xsed -e '/^$/d' > conftest.exp
>        $SED '/^$/d; /^ *+/d' conftest.err >conftest.er2
> -       if cmp conftest.exp conftest.er2 >/dev/null; then
> +       if diff conftest.exp conftest.er2 >/dev/null; then
>          $2=yes
>        fi
>      else