From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 19052 invoked by alias); 11 Apr 2007 07:36:43 -0000 Received: (qmail 19011 invoked by uid 22791); 11 Apr 2007 07:36:40 -0000 X-Spam-Check-By: sourceware.org Received: from server.usilu.net (HELO mail.usilu.net) (195.176.178.200) by sourceware.org (qpsmtpd/0.31) with ESMTP; Wed, 11 Apr 2007 08:36:35 +0100 Received: from [1.240.182.245] ([213.140.22.65] RDNS failed) by mail.usilu.net over TLS secured channel with Microsoft SMTPSVC(6.0.3790.3959); Wed, 11 Apr 2007 09:37:16 +0200 Message-ID: <461C8FE6.9010800@lu.unisi.ch> Date: Wed, 11 Apr 2007 07:36:00 -0000 From: Paolo Bonzini Reply-To: bonzini@gnu.org User-Agent: Thunderbird 1.5.0.10 (Macintosh/20070221) MIME-Version: 1.0 To: Dave Korn CC: 'Steve Ellcey' , binutils@sourceware.org, gcc-patches@gcc.gnu.org, gdb-patches@gcc.gnu.org, newlib@sourceware.org, Ralf.Wildenhues@gmx.de, aoliva@redhat.com, libtool@cwilson.fastmail.fm, schwab@suse.de Subject: Re: Final(?) patch to update libtool in GCC and src trees References: <006501c77bab$1753a410$2e08a8c0@CAM.ARTIMI.COM> <200704102006.NAA21177@hpsje.cup.hp.com> <008301c77bd6$7fa44bc0$2e08a8c0@CAM.ARTIMI.COM> In-Reply-To: <008301c77bd6$7fa44bc0$2e08a8c0@CAM.ARTIMI.COM> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2007-04/txt/msg00132.txt.bz2 Dave Korn wrote: > On 10 April 2007 21:07, Steve Ellcey wrote: > >>> I appreciate the problem, but Cygwin can be a bit 'special' when it >>> comes to libtool; if it hasn't been done yet, can you possibly bear to >>> hold off a bit while I give it some hasty testing? (By 'a bit', I mean no >>> more than say 'overnight'). Sorry to be a pain :-/ >>> >>> >>> cheers, >>> DaveK >> I can wait, especially since no one has given me the go ahead yet. :-) >> I know of no cygwin testing that has been done. > > Mixed results so far. Patching a binutils checkout and building it got me > this: > > > === ld tests === > @@ -297,14 +297,8 @@ Running /usr/build/src-binutils/ld/tests > Running /usr/build/src-binutils/ld/testsuite/ld-arm/arm-elf.exp ... > Running /usr/build/src-binutils/ld/testsuite/ld-auto-import/auto-import.exp > ... > Running /usr/build/src-binutils/ld/testsuite/ld-bootstrap/bootstrap.exp ... > -FAIL: bootstrap > -FAIL: bootstrap with strip > FAIL: bootstrap with --static > -FAIL: bootstrap with --traditional-format > -FAIL: bootstrap with --no-keep-memory > -FAIL: bootstrap with --relax > Running /usr/build/src-binutils/ld/testsuite/ld-cdtest/cdtest.exp ... > -FAIL: cdtest > FAIL: cdtest with -Ur > Running /usr/build/src-binutils/ld/testsuite/ld-checks/checks.exp ... > Running /usr/build/src-binutils/ld/testsuite/ld-cris/cris.exp ... > @@ -396,40 +390,40 @@ Running /usr/build/src-binutils/ld/tests > > === ld Summary === > > -# of expected passes 53 > -# of unexpected failures 9 > +# of expected passes 59 > +# of unexpected failures 3 > # of unexpected successes 1 > # of expected failures 3 > # of unsupported tests 1 > -/usr/build/obj-binutils/ld/ld-new 2.17.50.20070410 > +/usr/build/obj-binutils-patched/ld/ld-new 2.17.50.20070410 > > > ..which is entirely good :-D Then I tried a winsup checkout; it failed to > configure in newlib: > > checking for i686-pc-cygwin-ranlib... ranlib > checking for i686-pc-cygwin-readelf... no > checking for readelf... readelf > checking for a BSD-compatible install... /usr/bin/install -c > checking whether to enable maintainer-specific portions of Makefiles... no > checking for .preinit_array/.init_array/.fini_array support... readelf: Error: > Input file 'conftest' is not readable. > no > checking for array aliasing support... yes > configure: updating cache ./config.cache > configure: error: conditional "am__fastdepCXX" was never defined. > Usually this means the macro was only invoked conditionally. > Makefile:8649: *** [configure-target-newlib] Error 1 > > #0 configure-target-newlib at /usr/build/obj-winsup.new/Makefile:8649 > #1 all-target-newlib at /usr/build/obj-winsup.new/Makefile:8690 > #2 maybe-all-target-newlib at /usr/build/obj-winsup.new/Makefile:8687 > #3 all-target (.PHONY target) > make[1]: Leaving directory `/usr/build/obj-winsup.new' > > > This is what config.log showed: > > configure:3737: checking for a BSD-compatible install > configure:3793: result: /usr/bin/install -c > configure:3808: checking whether to enable maintainer-specific portions of > Makefiles > configure:3817: result: no > configure:21892: checking for .preinit_array/.init_array/.fini_array support > configure:21905: gcc -L/usr/build/obj-winsup.new/i686-pc-cygwin/winsup > -L/usr/build/obj-winsup.new/i686-pc-cygwin/winsup/cygwin > -L/usr/build/obj-winsup.new/i686-pc-cygwin/winsup/w32api/lib -isystem > /usr/build/src-winsup/winsup/include -isystem > /usr/build/src-winsup/winsup/cygwin/include -isystem > /usr/build/src-winsup/winsup/w32api/include > -B/usr/build/obj-winsup.new/i686-pc-cygwin/newlib/ -isystem > /usr/build/obj-winsup.new/i686-pc-cygwin/newlib/targ-include -isystem > /usr/build/src-winsup/newlib/libc/include > -I/usr/build/src-winsup/winsup/cygwin/include -o conftest conftest.c > -static -nostartfiles -nostdlib 1>&5 > configure:21908: $? = 0 > configure:21921: result: no > configure:21931: checking for array aliasing support > configure:21942: gcc -L/usr/build/obj-winsup.new/i686-pc-cygwin/winsup > -L/usr/build/obj-winsup.new/i686-pc-cygwin/winsup/cygwin > -L/usr/build/obj-winsup.new/i686-pc-cygwin/winsup/w32api/lib -isystem > /usr/build/src-winsup/winsup/include -isystem > /usr/build/src-winsup/winsup/cygwin/include -isystem > /usr/build/src-winsup/winsup/w32api/include > -B/usr/build/obj-winsup.new/i686-pc-cygwin/newlib/ -isystem > /usr/build/obj-winsup.new/i686-pc-cygwin/newlib/targ-include -isystem > /usr/build/src-winsup/newlib/libc/include > -I/usr/build/src-winsup/winsup/cygwin/include -c conftest.c > 1>&5 > conftest.c:1: warning: excess elements in scalar initializer > conftest.c:1: warning: (near initialization for `x3') > conftest.c:1: warning: excess elements in scalar initializer > conftest.c:1: warning: (near initialization for `x3') > configure:21945: $? = 0 > configure:21954: result: yes > configure:22030: updating cache ./config.cache > configure:22141: error: conditional "am__fastdepCXX" was never defined. > Usually this means the macro was only invoked conditionally. > > > Note the huge jump: > > configure:3817: result: no > configure:21892: checking for .preinit_array/.init_array/.fini_array support Right. Inside those two lines we have if test "${use_libtool}" = "yes"; then AC_LIBTOOL_WIN32_DLL AM_PROG_LIBTOOL AC_PROG_AWK fi I would rewrite them at the very least as AC_PROG_AWK AC_PROG_CXX if test "${use_libtool}" = "yes"; then AC_LIBTOOL_WIN32_DLL AM_PROG_LIBTOOL fi There might be more macros to put above the "if" to avoid that they be expanded inside an "if". Paolo