Re: [PATCH] [cygwin|mingw] fix dlpreopen with --disable-static (take 8?)

2010-06-27 Thread Ralf Wildenhues
* Charles Wilson wrote on Sun, Jun 27, 2010 at 02:51:21PM CEST: > > In that case, Ralf's suggested libfile_$(transliterated implib name) > > is used, because we have the .la file available which allows that > > shortcut. The only time we need `dlltool --identify' is when > > dlpreopening a non-lib

Re: [PATCH] Add func_append_quoted and do inline func_append substitutions.

2010-06-27 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Mon, Jun 28, 2010 at 01:24:39AM CEST: > Looking through the XSI substitutions, or more correctly the bash/ksh > func_append usage, there's room here to consistently use func_append > everywhere to make for easier maintenance... but rather than take the > additio

Re: [PATCH] [cygwin|mingw] fix dlpreopen with --disable-static (take 8?)

2010-06-27 Thread Ralf Wildenhues
Hello Charles, * Charles Wilson wrote on Mon, Jun 28, 2010 at 12:05:40AM CEST: > On 6/27/2010 4:43 PM, Ralf Wildenhues wrote: > > * Charles Wilson wrote on Sun, Jun 27, 2010 at 02:51:21PM CEST: > >> So...can I get a verdict? Is -dlpreopen not-an-la-file supposed to work

Re: [PATCH] Add an XSI replacement for func_split_short_opt.

2010-06-28 Thread Ralf Wildenhues
Hi Eric, thanks for the suggestion. I had considered the idea for a second, but failed to see the nontrivial half. * Eric Blake wrote on Mon, Jun 28, 2010 at 02:49:40PM CEST: > tmp=${1#?} > patt= > i=2 > while test $i -lt ${#1}; do > patt="?$patt" If the parameter may be expected to be very l

Re: [PATCH 3/4] Uniform const'ness of symlist variable lt_preloaded_symbols.

2010-06-28 Thread Ralf Wildenhues
Hello Vincent, * Ralf Wildenhues wrote on Sat, Jun 19, 2010 at 12:08:29PM CEST: > I would like to commit the patch quoted below, on the way of support for > LTO with GCC. For that, I need to know from somebody who is willing to > help out with the Windows CE port of Libtool whether the

Re: [PATCH] Add an XSI replacement for func_split_short_opt.

2010-06-29 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Tue, Jun 29, 2010 at 05:09:29PM CEST: > On 29 Jun 2010, at 21:03, Eric Blake wrote: > > Ultimately, I'd like to fix m4sh to make it easier to probe/require XSI > > support, but that will have to wait until after autoconf 2.66. > > While that might turn out to b

Re: checking for header/library mismatch, rpath problem?

2010-06-29 Thread Ralf Wildenhues
[ please elide autoconf@ from followups, thanks ] Hi Peter, * Peter Breitenlohner wrote on Tue, Jun 29, 2010 at 10:36:32AM CEST: > Here the macro we are using in TeX Live for such tests (C and C++). Our > purpose is to test properties of libraries that can be either > (1) uninstalled libtool lib

Re: [PATCH] Add an XSI replacement for func_split_short_opt.

2010-06-29 Thread Ralf Wildenhues
* Gary V. Vaughan wrote on Tue, Jun 29, 2010 at 08:30:43PM CEST: > On 30 Jun 2010, at 01:22, Ralf Wildenhues wrote: > > I think m4sh can simply use code like > > > > if ( eval '$smart_works' ) >/dev/null 2>&1; then > >func_foo () { smart c

Re: [SCM] GNU Libtool branch, master, updated. v2.2.10-49-gc13532a

2010-07-04 Thread Ralf Wildenhues
[ moving from -commit to -patches, I think the former was for commits only; maybe it should have reply-to set? ] Hi Gary, Peter, * Gary V. Vaughan wrote on Sat, Jul 03, 2010 at 10:23:12AM CEST: > On 3 Jul 2010, at 15:15, Peter Rosin wrote: > >Fix typo in "Add func_append_quoted and do..." >

Re: [PATCH] [cygwin|mingw] fix dlpreopen with --disable-static (take 7)

2010-07-04 Thread Ralf Wildenhues
* Charles Wilson wrote on Sun, Jul 04, 2010 at 06:15:06AM CEST: > On 6/26/2010 2:51 PM, Ralf Wildenhues wrote: > > OK. Here's my take on this: if you fix all nits I noted inline below, > > post the updated and tested patch (you decide what testing is needed), > > then

Re: [PATCH] [cygwin|mingw] fix dlpreopen with --disable-static (take 8?)

2010-07-04 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Sat, Jul 03, 2010 at 06:10:57PM CEST: > On 6/28/2010 2:10 AM, Ralf Wildenhues wrote: > > * Charles Wilson wrote on Mon, Jun 28, 2010 at 12:05:40AM CEST: > >> It obviously isn't SUPPOSED to be dead -- or it wouldn't be there. >

Re: MSVC: Fix ccache test for MSVC.

2010-07-04 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Wed, Jun 30, 2010 at 09:10:02PM CEST: > Can you please cherry-pick this commit of yours into master? > "Fix ccache test for MSVC." > 563c2406deb4d5ed0ebf5faf5c8b798d52ee6ac2 Done, thanks for the reminder. > I could do it myself but I don't know what to do about t

*_cmds file name expansion bug (was: MSVC: For MSVC, embed the manifest as a resource in the executable.)

2010-07-04 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Thu, Jul 01, 2010 at 10:30:56PM CEST: > I.e. *.exe get expanded. That doesn't happen with "*.[eE][xX][eE])": > [...] > This all seems awfully fragile. I don't know how to quote > this expression in libtool.m4 and need help. Someone please? Ouch. Our expansion co

Re: MSVC: Preloading in ltdl doesn't heed libname_spec.

2010-07-05 Thread Ralf Wildenhues
Hi Peter, any chance you could post your patches inline? Thanks. * Peter Rosin wrote on Mon, Jul 05, 2010 at 02:45:28PM CEST: > http://lists.gnu.org/archive/html/libtool-patches/2009-01/msg00152.html This is ok with nits below addressed, thanks. > Make preloading heed libname_spec. >

Re: MSVC: Support for response files with $NM.

2010-07-05 Thread Ralf Wildenhues
e debian/gcc case actually uses the @FILE code path in the testsuite (should be in the low_cmd_len tests)? > 2010-07-01 Ralf Wildenhues > Peter Rosin > > Support for response files with $NM. > * libltdl/m4/libtool.m4 (_LT_CMD_GLOBAL_SYMBOLS) > : New

Re: MSVC: Preloading in ltdl doesn't heed libname_spec.

2010-07-05 Thread Ralf Wildenhues
Hi Bob, * Bob Friesenhahn wrote on Tue, Jul 06, 2010 at 06:48:56AM CEST: > On Mon, 5 Jul 2010, Bob Friesenhahn wrote: > >Yesterday's libtool was doing quite good with the tests but I am > >seeing plenty of failures now for all Unixish targets. Even Linux > >blows failures all over the place. Not

Re: MSVC: Support for response files with $NM.

2010-07-06 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Tue, Jul 06, 2010 at 11:27:31AM CEST: > Den 2010-07-06 08:07 skrev Ralf Wildenhues: > >Did you confirm that the debian/gcc case actually uses the @FILE code > >path in the testsuite (should be in the low_cmd_len tests)? > > No I didn

Re: [PATCH RFC] Add sysroot support.

2010-07-07 Thread Ralf Wildenhues
Hello Thierry, thanks for your patch and explanations. * Thierry Reding wrote on Wed, Jul 07, 2010 at 11:51:58AM CEST: > the following patch adds support for gcc's --sysroot argument to libtool. > I'll post the patch first and will reply to it, giving more details where > needed. > > I've been b

verbose test output (was: MSVC: Preloading in ltdl doesn't heed libname_spec.)

2010-07-08 Thread Ralf Wildenhues
Hello Bob, * Bob Friesenhahn wrote on Thu, Jul 08, 2010 at 06:34:46PM CEST: > On Tue, 6 Jul 2010, Ralf Wildenhues wrote: > >> > >>Perhaps things are not all that bad. While 35 of 110 tests fail > >>under Debian Linux, only these two extra are failing under

Re: [SCM] GNU Libtool branch, master, updated. v2.2.10-59-g8c4dae1

2010-07-08 Thread Ralf Wildenhues
[ moving from -commit ] Hi Gary, * Gary V. Vaughan wrote on Thu, Jul 08, 2010 at 03:37:23PM CEST: > commit 8c4dae1232958c24989f31ab5b5768d00be2ef03 > Author: Gary V. Vaughan > Date: Thu Jul 8 18:47:59 2010 +0700 > > Fix a spurious trailing space and a botched merge. > > * libltdl

Re: MSVC: Always dllimport the variable for MSVC in link-order.at

2010-07-11 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Sat, Jul 10, 2010 at 10:15:09AM CEST: > Next in to MSVC queue... > > This is a variation of > 95bdbc456f7c5d6d669121cf20d62a9c28bec2bf > "Always dllimport the variable for MSVC in link-order.at" > > Patch previously discussed here: > http://lists.gnu.org/archive/

Re: MSVC: Always dllimport the variable for MSVC in link-order.at

2010-07-15 Thread Ralf Wildenhues
Hi Peter, Charles, * Peter Rosin wrote on Thu, Jul 15, 2010 at 10:39:42PM CEST: > Den 2010-07-12 07:46 skrev Ralf Wildenhues: > >* Peter Rosin wrote on Sat, Jul 10, 2010 at 10:15:09AM CEST: > >the patch looks fairly innocent at first sight, but your description > >makes me

Re: MSVC: Always dllimport the variable for MSVC in link-order.at

2010-07-19 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Sat, Jul 17, 2010 at 04:33:07AM CEST: > Den 2010-07-16 05:36 skrev Ralf Wildenhues: > >>The manual already says that it is not portable to export data items from > >>a libtool library. At least I think it does. > > > >The "Li

Re: MSVC: stresstest.at fixes

2010-07-19 Thread Ralf Wildenhues
Hello Peter, * Peter Rosin wrote on Sun, Jul 18, 2010 at 06:45:53PM CEST: > Next in the MSVC queue. It's getting short quickly! > > These two patches are commits > ee8c181987478752a9432a903cf7febbf761d034 > dllimport when the lib is shared and not linking with -static > > and > > 5ff2e6e4e5a1f0

Re: MSVC: Support for response files with $NM.

2010-07-19 Thread Ralf Wildenhues
Hi Peter, a little while ago: * Peter Rosin wrote on Wed, Jul 07, 2010 at 10:15:53PM CEST: > No problem, but I was initially very confused (and still am) by the > fact that > > make check-local TESTSUITEFLAGS='-v -d -k max_cmd_len' \ > INNER_TESTSUITEFLAGS=',export -v -d export' | grep cre

Re: [PATCH RFC] Add sysroot support.

2010-07-21 Thread Ralf Wildenhues
Hello, * Paolo Bonzini wrote on Wed, Jul 21, 2010 at 06:04:43PM CEST: > On 07/21/2010 03:58 PM, Charles Wilson wrote: > >On 7/8/2010 1:02 PM, Paolo Bonzini wrote: > > > >>I have another sysroot patch that seems (from a first cursory view) > >>pretty different. I'll rebase and send. > > > >Ping? >

Re: [RFT PATCH v3 2/9] handle sysroot flags

2010-08-01 Thread Ralf Wildenhues
Hi Paolo, patch 1/9 is OK right away. Thanks! * Paolo Bonzini wrote on Thu, Jul 29, 2010 at 01:23:15AM CEST: > * libltdl/config/ltmain.m4sh (func_mode_link): Recognize --sysroot > option. This patch 2/9 is OK but please commit together with the rest. Thanks, Ralf

Re: [RFT PATCH v3 3/9] add --with-sysroot

2010-08-01 Thread Ralf Wildenhues
Next one: 3/9. This is a bit of a preliminary review, because I haven't read all patches and all of the discussion yet. * Paolo Bonzini wrote on Thu, Jul 29, 2010 at 01:23:16AM CEST: > * libltdl/m4/libtool.m4 (_LT_HOST_NONCANONICAL, _LT_WITH_SYSROOT): New. > (LT_SETUP): Require _LT_WITH_SYSROOT.

Re: using ld for linking non-C code

2010-08-01 Thread Ralf Wildenhues
* Charles Wilson wrote on Sun, Aug 01, 2010 at 02:10:36PM CEST: > On 8/1/2010 6:46 AM, Ralf Wildenhues wrote: > > Because that allows it to set run paths to the compiler libs? > > (I don't know either.) > > Hmm. If so, what use case does that capability have? You want t

using ld for linking non-C code (was: [REBASED PATCH 0/9] Paolo's sysroot patches, rebased plus some fixes)

2010-08-01 Thread Ralf Wildenhues
* Charles Wilson wrote on Mon, Jul 26, 2010 at 01:57:57PM CEST: > As an aside, WHY is it that libtool tries to figure out what the language > driver > does, and use ld directly, rather than just trust the language driver and use > it > to link instead? It seems that the current libtool behavior

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-01 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Sun, Aug 01, 2010 at 04:47:26PM CEST: > This should, and I say "should" :) pass all tests. Thanks for the repost. Did anything change beside the discussion attached to the last patch series post? This passes for me on GNU/Linux with a non sysrooted GCC. How come there i

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-01 Thread Ralf Wildenhues
* Charles Wilson wrote on Sun, Aug 01, 2010 at 08:03:11PM CEST: > On 8/1/2010 10:47 AM, Paolo Bonzini wrote: > >> 112: override pic_flag at configure time FAILED (pic_flag.at:48) > >> > >> 112 has been discussed before and should be > >> skipped on non-ELF. > > > > Since I wasn't around, why is

Re: [RFT PATCH v4 6/8] emit sysrooted paths when installing .la files

2010-08-01 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Sun, Aug 01, 2010 at 04:47:32PM CEST: > * libltdl/config/ltmain.m4sh (func_replace_sysroot): New. > (func_mode_link): Prepend paths in $libdir with the sysroot. Further, > replace the sysroot with = (using func_replace_sysroot) whenever > such a path is written in a .la fi

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-01 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Sun, Aug 01, 2010 at 04:47:26PM CEST: > handle sysroot flags > add --with-sysroot > teach libtool -L= and -R= > handle sysrooted paths when reading dependencies to la files > process postdeps to include sysrooted paths > emit sysrooted paths when installing .la

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-02 Thread Ralf Wildenhues
* Charles Wilson wrote on Mon, Aug 02, 2010 at 06:15:29AM CEST: > > On 8/1/2010 2:43 PM, Ralf Wildenhues wrote: > >> I can't really give a good review of the rest without a sysrooted > >> compiler, and I don't have time to build one any more this weekend, > &

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-02 Thread Ralf Wildenhues
* Charles Wilson wrote on Sun, Aug 01, 2010 at 10:04:16PM CEST: > support, use --prefix=/mingw + DESTDIR=/the-sysroot. What about the w32 users that use --prefix=C:/mingw and then cannot use DESTDIR because that will not concatenate? Also, in 'ln -sf' in the testsuite addition, the -f is non-port

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-02 Thread Ralf Wildenhues
* Charles Wilson wrote on Mon, Aug 02, 2010 at 08:00:47PM CEST: > On 8/2/2010 1:55 PM, Ralf Wildenhues wrote: > >What about the w32 users that use --prefix=C:/mingw and then cannot use > >DESTDIR because that will not concatenate? > > According to the GNU Standards, you can

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-02 Thread Ralf Wildenhues
More issues: - the part of ltmain starting with # Collect and forward deplibs of preopened libtool libs will source .la files but looks like it mistreats = in $dependency_libs - is it intended that the = remain in the .la files after these are moved to their final location on the host sy

fix pic_flag test (was: [RFT PATCH v4 0/8] Sysroot series)

2010-08-02 Thread Ralf Wildenhues
* Charles Wilson wrote on Mon, Aug 02, 2010 at 05:23:28AM CEST: > The "unexpected" failure was > 112: override pic_flag at configure time FAILED (pic_flag.at:48) > which isn't a regression on cygwin. I'm pushing this patch to fix two issues with the test case. Really should've known better than t

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-02 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Mon, Aug 02, 2010 at 10:51:30PM CEST: > On Mon, Aug 2, 2010 at 22:13, Ralf Wildenhues wrote: > > > - is it intended that the = remain in the .la files after these are > > moved to their final location on the host system?  Or should > > --mode=fi

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-02 Thread Ralf Wildenhues
* Charles Wilson wrote on Tue, Aug 03, 2010 at 07:30:28AM CEST: > On 8/2/2010 4:51 PM, Paolo Bonzini wrote: > > On Mon, Aug 2, 2010 at 22:13, Ralf Wildenhues wrote: > >> - is it intended that the = remain in the .la files after these are > >> moved to their final locat

Re: [RFT PATCH v4 6/8] emit sysrooted paths when installing .la files

2010-08-03 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Tue, Aug 03, 2010 at 09:52:54AM CEST: > On 08/01/2010 08:31 PM, Ralf Wildenhues wrote: > >Although it is fairly unlikely, I prefer not relying on :+ support, so > >please replace with 'if test -z "$lt_sysroot"...'. Two instances. >

Re: [RFT PATCH v3 3/9] add --with-sysroot

2010-08-03 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Tue, Aug 03, 2010 at 10:04:08AM CEST: > On 08/01/2010 12:48 PM, Ralf Wildenhues wrote: > [ other packages that already use --with-sysroot ] > >I'm not sure how to address these (not even sure > >they use Libtool) except for a big sign in NEWS, and

Re: [PATCH] Add func_append_quoted and do inline func_append substitutions.

2010-08-03 Thread Ralf Wildenhues
* Peter Rosin wrote on Thu, Jul 01, 2010 at 07:38:45PM CEST: > Den 2010-06-28 01:24 skrev Gary V. Vaughan: > >@@ -704,15 +710,14 @@ func_mode_compile () > > save_ifs="$IFS"; IFS=',' > > for arg in $args; do > > IFS="$save_ifs" > >-func_quote_for_eval "$arg" > >-l

Fix testsuite errors due to shell quoted parameter expansion issue.

2010-08-03 Thread Ralf Wildenhues
Interesting shell unportability: $ bash -c 'f=" val" e=; echo "$e"$f' val $ ksh -c 'f=" val" e=; echo "$e"$f' val ksh93, dash, zsh all do it like ksh. Is that a bug in bash? Anyway, the patch below seems to fix it, but I may want to fix the log entry to pinpoing the culprit correctly. Thanks,

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-04 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Tue, Aug 03, 2010 at 07:22:46AM CEST: > For instance, suppose I have a cross compiler for mingw whose sysroot is > /usr/i686-pc-mingw32/sys-root. Suppose I want to build the gtk2 stack, > for win32. Well, that means I need to build gettext, and libiconv, and

Re: Fix testsuite errors due to shell quoted parameter expansion issue.

2010-08-04 Thread Ralf Wildenhues
* Eric Blake wrote on Tue, Aug 03, 2010 at 10:55:36PM CEST: > On 08/03/2010 02:32 PM, Ralf Wildenhues wrote: > > Interesting shell unportability: > > > > $ bash -c 'f=" val" e=; echo "$e"$f' > > val > > $ ksh -c 'f=" val&quo

Re: libtool-2.2.11a on AIX 5.3 / 5.3 current git master - tests/nomode.test

2010-08-04 Thread Ralf Wildenhues
* Rainer Tammer wrote on Wed, Aug 04, 2010 at 08:41:26AM CEST: > FAIL: tests/nomode.test > > This failure is due to the following error: > > $ ./libtool > ./libtool[1591]: shift: bad number > > -> code > # save first non-option argument > nonopt="$opt"; shift > > But there is nothing in $1,

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-05 Thread Ralf Wildenhues
Hi Charles, thanks for the elaborate explanations. * Charles Wilson wrote on Thu, Aug 05, 2010 at 08:10:10AM CEST: > On 8/4/2010 2:15 PM, Ralf Wildenhues wrote: > > * Charles Wilson wrote on Tue, Aug 03, 2010 at 07:22:46AM CEST: [...] > >> Thanks to a quirk in window'

Re: Next Libtool Point Release Pending

2010-08-05 Thread Ralf Wildenhues
Hello Gary, * Gary V. Vaughan wrote on Fri, Aug 06, 2010 at 06:01:37AM CEST: > I promised to roll a 2.2.12 release this month. But looking at the > huge progress we've made so far with Windows since 2.2.10, I'm > wondering if we would be better to roll 2.2.12 sooner rather than > later? I am sti

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-05 Thread Ralf Wildenhues
* Charles Wilson wrote on Fri, Aug 06, 2010 at 03:26:30AM CEST: > But a STEP in that direction is to reduce the current pain experienced > by cross compiles for $host=mingw even with --prefix=/unixish; pain > caused by the current lack of sysroot support in libtool (and some other > utilities). >

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-06 Thread Ralf Wildenhues
* Charles Wilson wrote on Fri, Aug 06, 2010 at 02:47:59PM CEST: > On 8/6/2010 1:19 AM, Ralf Wildenhues wrote: > > Wrt. lib and lib32, lib64, can we find out all files GCC needs by asking > > it several -print-* questions? Plus maybe optimistically searching for > > libc and

Re: Next Libtool Point Release Pending

2010-08-06 Thread Ralf Wildenhues
Hello, * Gary V. Vaughan wrote on Fri, Aug 06, 2010 at 08:53:15PM CEST: > Another viable compromise might be to call the next release > 2.3.0? Maybe it's early enough to finalize a decision on this when we know a bit better which patches are in, and which features can be advertised. There is als

Fix ltdl-libdir test for C++ compilers.

2010-08-06 Thread Ralf Wildenhues
Pushed as obvious to let 'configure CC=g++' pass the testsuite again. Cheers, Ralf Fix ltdl-libdir test for C++ compilers. * tests/ltdl-libdir.at (libdir of installed modules): Declare module function as extern "C". diff --git a/tests/ltdl-libdir.at b/tests/ltdl-libdir.at index

Fix AC_NO_EXECUTABLES test failure on AIX. (was: libtool-2.2.8 on AIX 6.1)

2010-08-06 Thread Ralf Wildenhues
> On 02.08.2010 20:40, Ralf Wildenhues wrote: > > * Rainer Tammer wrote on Mon, Aug 02, 2010 at 05:40:13PM CEST: > >> I just have tested libtool 2.2.8 on AIX 6.1. > >> 95: AC_NO_EXECUTABLES FAILED > >> (no-executables.at:76)

versioning test on AIX (was: libtool-2.2.8 on AIX 6.1)

2010-08-06 Thread Ralf Wildenhues
* Rainer Tammer wrote on Tue, Aug 03, 2010 at 09:40:23AM CEST: > On 02.08.2010 20:40, Ralf Wildenhues wrote: > > * Rainer Tammer wrote on Mon, Aug 02, 2010 at 05:40:13PM CEST: > >> 56: versioning FAILED > >> (versioning.at:169) &g

revert unwanted enabling of libtool tracing (was: libtool-2.2.11a on AIX 5.3 current git master / followup on the old test suite and failures in the new testsuit)

2010-08-06 Thread Ralf Wildenhues
Rainer, I'm going through things in some random order, and it will take me at least a little while to get through them all. * Rainer Tammer wrote on Thu, Aug 05, 2010 at 09:25:14AM CEST: > 38. link-order2.at:46: testing Link order of deplibs ... > libtool: compile: cc_r -qlanglvl=extc89 -g -c a0.

Do not call toplevel configure in non-VPATH build. (was: libtool-2.2.11a on AIX 5.3 current git master / followup on the old test suite and failures in the new testsuit)

2010-08-08 Thread Ralf Wildenhues
* Rainer Tammer wrote on Thu, Aug 05, 2010 at 09:25:14AM CEST: > 112. pic_flag.at:24: testing override pic_flag at configure time ... > ./pic_flag.at:36: $CC $CPPFLAGS $CFLAGS $C_pic_flag -c foo.c || exit 77 > stderr: > stdout: > ./pic_flag.at:39: grep fpic stdout stderr && exit 77 > ./pic_flag.at:

Allow running pic_flag test with different shell.

2010-08-08 Thread Ralf Wildenhues
I'd like to ensure that $CONFIG_SHELL and $configure_options are used consistently throughout the testsuite. So I'm fixing the pic_flag test to also use LT_AT_CONFIGURE, after making that macro accept a second argument to specify another configure script location. Hope this is trivial enough to j

Avoid unwanted error message in configure output.

2010-08-08 Thread Ralf Wildenhues
There might be no 'print' program available, and some shells (like FreeBSD sh) redirect that stderr only when you call the program from a subshell. I'm installing this to beautify configure output. The second 'print' instance should be fine, and 'printf' can be assumed to at least be present ever

Re: libtool-2.2.11a on AIX 5.3 current git master / followup on the old test suite and failures in the new testsuit

2010-08-08 Thread Ralf Wildenhues
* Rainer Tammer wrote on Thu, Aug 05, 2010 at 09:25:14AM CEST: > I just refreshed my source from git and rerun the test suit on AIX 5.3. > 75: lt_dlopenext error messages FAILED > (lt_dlopenext.at:202) > 76: libdir of installed modules FAILED > (ltdl-libdi

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-08 Thread Ralf Wildenhues
FWIW, I didn't find any more objectionable things in the rest of this patch series, so whenever you have something that addresses the known issues, I think a public branch would be fine. If merely additional testsuite items are missing, that shouldn't hold up creating a branch, only merging it. A

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-08 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Sat, Aug 07, 2010 at 06:06:14PM CEST: > configure:2917: checking whether the C compiler works > configure:2939: gcc --sysroot=/tmp/sysroot conftest.c >&5 > /usr/bin/ld: this linker was not configured to use sysroots > collect2: ld returned 1 exit status >

Re: [PATCH] Add func_append_quoted and do inline func_append substitutions.

2010-08-08 Thread Ralf Wildenhues
Hi Gary, * Ralf Wildenhues wrote on Mon, Jun 28, 2010 at 07:34:35AM CEST: > * Gary V. Vaughan wrote on Mon, Jun 28, 2010 at 01:24:39AM CEST: > > Looking through the XSI substitutions, or more correctly the bash/ksh > > func_append usage, there's room here to consiste

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-08 Thread Ralf Wildenhues
* Charles Wilson wrote on Sun, Aug 08, 2010 at 10:40:56PM CEST: > So, you suggesting *possibly* a test where: > > if not cross-compiling > if (the native) ld supports --sysroot > 1. create a fake sysroot (contents TBD?) > 2. execute sysroot.at contents as a subtest, with >native

Re: [RFT PATCH v4 0/8] Sysroot series

2010-08-08 Thread Ralf Wildenhues
* Charles Wilson wrote on Mon, Aug 09, 2010 at 12:14:37AM CEST: > On 8/7/2010 2:36 AM, Ralf Wildenhues wrote: > > FWIW, I didn't find any more objectionable things in the rest of this > > patch series, so whenever you have something that addresses the known > > issue

finish mode (was: [RFT PATCH v4 0/8] Sysroot series)

2010-08-08 Thread Ralf Wildenhues
* Charles Wilson wrote on Mon, Aug 09, 2010 at 12:14:37AM CEST: > I think --mode=finish needs to accept additional arguments that are not > directories, somehow. Maybe mode=finish needs to iterate through its > arguments, and split them into "directories" and ".la files", and then > execute two di

Re: [PATCH] Add func_append_quoted and do inline func_append substitutions.

2010-08-08 Thread Ralf Wildenhues
* Gary V. Vaughan wrote on Sun, Aug 08, 2010 at 08:50:12AM CEST: > On 7 Aug 2010, at 18:15, Ralf Wildenhues wrote: > > <http://lists.gnu.org/archive/html/libtool-patches/2010-06/msg00190.html> > I don't see that the +=/func_append substitution is causing any additiona

Re: Next Libtool Point Release Pending

2010-08-09 Thread Ralf Wildenhues
Hello, * Peter O'Gorman wrote on Mon, Aug 09, 2010 at 05:38:44PM CEST: > I don't think having a stable and development branch worked well > with 1.5.x and 2.x. It added more work for us, and did not serve our > users well - they had to wait years for the development branch's > features to make it

autobuild logs for Libtool (was: Next Libtool Point Release Pending)

2010-08-09 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Mon, Aug 09, 2010 at 08:42:24PM CEST: > I do think however that it's time that our testing efforts become a bit > more coordinated, to make regression tracking easier and more apparent. > Will reply to this post with more details. I think Libtool should

another func_append inlining regression (was: libtool-2.2.11a on AIX 5.3 current git master / followup on the old test suite and failures in the new testsuit)

2010-08-09 Thread Ralf Wildenhues
* Rainer Tammer wrote on Thu, Aug 05, 2010 at 09:25:14AM CEST: > The new test suite still yields many failures in comparison to 2.2.8. I assume most if not all of the regressions are due to a single issue: > # grep FAILED check.log > 38: Link order of deplibs FAILED > (

Re: another func_append inlining regression

2010-08-09 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Mon, Aug 09, 2010 at 10:55:25PM CEST: > Note the double colon after -blibpath, which is due to a regression in > v2.2.10-40-g0da7d80 fixed with the patch below. I wish I'd never > approved that patch, this one took me so long to find ... I did a re-review

Re: How to maintain compatibility with Automake.

2010-08-10 Thread Ralf Wildenhues
* Gary V. Vaughan wrote on Tue, Aug 10, 2010 at 11:09:33AM CEST: > On 10 Aug 2010, at 15:19, Peter Rosin wrote: > > m4_defun([AM_PROG_AR], > > [AC_CHECK_TOOLS(AR, [ar], false) > > : ${AR=ar} > > AC_SUBST([AR])dnl > > ])# AM_PROG_AR > > ]) > > Just like that! NAK. I don't have time to explain now

Re: How to maintain compatibility with Automake.

2010-08-10 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Tue, Aug 10, 2010 at 11:15:21AM CEST: > * Gary V. Vaughan wrote on Tue, Aug 10, 2010 at 11:09:33AM CEST: > > On 10 Aug 2010, at 15:19, Peter Rosin wrote: > > > m4_defun([AM_PROG_AR], > > > [AC_CHECK_TOOLS(AR, [ar], false) > > >

Re: [PATCH] Allow the use of a listing file if the archiver supports it.

2010-08-12 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Thu, Aug 12, 2010 at 03:28:57PM CEST: > This is a patch that makes use of @FILE support in the > archiver, if the archiver supports it. That makes linking > succeed for long command lines when using MSVC, as MSVC > can't do piecewise linking (-r -o) which is the cu

Re: [PATCH 0/2] sysroot patches followup

2010-08-12 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Thu, Aug 12, 2010 at 06:12:15AM CEST: > Hi, I pushed the first 8 sysroot patches to refs/heads/sysroot on > Savannah. Thanks. So far, with a merge of sysroot to master, I see one regression on AIX with runtimelinking (configure LDFLAGS=-Wl,-brtl) and a different one on So

Re: [PATCH v2 1/4] fix sysroot tests to pass on Fedora 13

2010-08-12 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Fri, Aug 13, 2010 at 03:23:03AM CEST: > * tests/sysroot.at: Link $prefix/include into the sysroot as well, as > required by newer GCC. Obviously. Thanks! Ralf

Re: [PATCH v2 2/4] fix sysroot handling for deplibs of preopened libtool libs

2010-08-12 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Fri, Aug 13, 2010 at 03:23:04AM CEST: > * libltdl/config/ltmain.m4sh: Pass $dependency_libs of preopened libtool > through func_resolve_sysroot. This looks ok, but is it covered in the testsuite? (If not, I can try to do it later, but at least would like to be sure we make

Re: [PATCH v2 3/4] reorganize parsing of --mode=finish arguments

2010-08-12 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Fri, Aug 13, 2010 at 03:23:05AM CEST: > * libltdl/config/ltmain.m4sh (func_mode_finish): Split arguments for > directories and *.la files. Add .la files for all directories. OK if you fix the log entry. IIUC the last sentence is wrong. Thanks, Ralf

Re: [PATCH v2 3/4] reorganize parsing of --mode=finish arguments

2010-08-12 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Fri, Aug 13, 2010 at 03:23:05AM CEST: > * libltdl/config/ltmain.m4sh (func_mode_finish): Split arguments for > directories and *.la files. Add .la files for all directories. Ah, there's another nit. > --- a/libltdl/config/ltmain.m4sh > +++ b/libltdl/config/ltmain.m4sh >

Re: [PATCH v2 4/4] add libtool --mode=finish mode for sysroot

2010-08-12 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Fri, Aug 13, 2010 at 03:23:06AM CEST: > --- Erm, you should be mentioning the files you change in your log entries, and Libtool also doesn't auto-generate its ChangeLog (yet), so you need that, too, also for all the other patches in the sysroot branch. Sorry for not notic

Re: MSVC status

2010-08-13 Thread Ralf Wildenhues
* Christopher Hulbert wrote on Fri, Aug 13, 2010 at 01:45:37PM CEST: > Is there interest (besides mine) in libtool being able to use native > Windows compilers from cygwin. I just find cygwin easier to > use/maintain than mingw/msys. > > In the branch I had been working on, I defined the libtool v

Re: [PATCH 0/2] sysroot patches followup

2010-08-14 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Thu, Aug 12, 2010 at 11:28:31PM CEST: > Thanks. So far, with a merge of sysroot to master, I see one regression > on AIX with runtimelinking (configure LDFLAGS=-Wl,-brtl) OK so this one really is a regression with the patch series. bisect converges at the

Re: [PATCH 0/2] sysroot patches followup

2010-08-14 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Thu, Aug 12, 2010 at 11:28:31PM CEST: > Thanks. So far, with a merge of sysroot to master, I see one regression > on AIX with runtimelinking (configure LDFLAGS=-Wl,-brtl) and a different > one on Solaris, The Solaris issue is shown below and is actual

Re: [PATCH] Factor the sed command used to make a regex from a literal.

2010-08-16 Thread Ralf Wildenhues
eneral.m4sh (sed_make_literal_regex): New. > * libltdl/config/ltmain.m4sh (func_generate_dlsyms, FWIW, I think GCS style is * file-name (function-name) (function-name, ...): i.e., close and reopen parens at EOL. > func_cygming_dll_for_implib_fallback_core): Use it. Suggested > by Ralf Wildenhues.

Re: [PATCH] improve code for sysroot --mode=finish

2010-08-16 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Sun, Aug 15, 2010 at 06:57:18PM CEST: > Here are the improvements you suggested. I'll send a patch for > sed_make_literal_regex soon. > > Ok? Yes, with the nit addressed. Thanks! Ralf > * libltdl/config/ltmain.m4sh (func_mode_finish): Change sysroot_regex > and sysroot

Re: [PATCH v2 4/4] add libtool --mode=finish mode for sysroot

2010-08-16 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Sun, Aug 15, 2010 at 03:46:06PM CEST: > On 08/13/2010 12:59 AM, Ralf Wildenhues wrote: > >Erm, you should be mentioning the files you change in your log entries, > >and Libtool also doesn't auto-generate its ChangeLog (yet), so you need > >that, t

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-16 Thread Ralf Wildenhues
Hi Charles, Paolo, * Charles Wilson wrote on Mon, Aug 16, 2010 at 05:12:03PM CEST: > On 8/12/2010 9:23 PM, Paolo Bonzini wrote: > > These cover bullets 3/5 of Charles' list: > > > > 3) issue with '# Collect and forward deplibs of preopened libtool libs' > >In progress? Anything? > > 4,5) > >

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-16 Thread Ralf Wildenhues
* Charles Wilson wrote on Mon, Aug 16, 2010 at 10:27:46PM CEST: > Taking all of this together, I wonder if a good approach might be the > following: libltdl replaces the '=' with the value of the environment > variable LIBLTDL_SYSROOT. So usually -- with this variable empty -- '=' > is just stripp

Re: [PATCH v2 0/3] sysroot followup patches

2010-08-16 Thread Ralf Wildenhues
* Charles Wilson wrote on Mon, Aug 16, 2010 at 10:50:21PM CEST: > On 8/16/2010 4:42 PM, Ralf Wildenhues wrote: > > > > Can you run > > VERBOSE=yes $srcdir/tests/fcdemo-conf.test; echo $? > > > > and post the output of that? VERBOSE=debug is even more info

Re: [RFT PATCH v4 3/8] teach libtool -L= and -R=

2010-08-18 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Sun, Aug 01, 2010 at 04:47:29PM CEST: > * libltdl/config/ltmain.m4sh (func_resolve_sysroot): New. > (func_mode_link): Always pass result of stripping -L and -R to > func_resolve_sysroot before using it. Remove absolute path expansion > when func_resolve_sysroot subsumes it

Re: [RFT PATCH v4 3/8] teach libtool -L= and -R=

2010-08-18 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Thu, Aug 19, 2010 at 06:58:31AM CEST: > * Paolo Bonzini wrote on Sun, Aug 01, 2010 at 04:47:29PM CEST: > > +# func_resolve_sysroot PATH > > +# Change PATH to an absolute path or replace a leading = with a > > +# sysroot. Store the result into func_r

[PATCH] Do not absolutize paths eagerly.

2010-08-19 Thread Ralf Wildenhues
AM, Ralf Wildenhues wrote: > >Why does this function absolutize paths that have nothing to do with > >sysroot, even in-tree deplib paths? > >Is there a deeper reason in this? If yes, there should probably be a > >test which is fixed by it. > > It's a bug, I think

Don't hide the table of contents in the manual.

2010-08-19 Thread Ralf Wildenhues
This moves the ToC to the beginning in the PDF output, and IMHO looks so much nicer. Will push soon unless somebody complains. Thanks Bruno! Ralf Don't hide the table of contents in the manual. * doc/libtool.texi: Move the table of contents to the beginning. Prompted by patch fr

Remove superfluous dependency in Makefile.am.

2010-08-19 Thread Ralf Wildenhues
Pushed as obvious: there is nothing that would generate demo-conf.test and I couldn't find any reason for the test result to depend on the libtool script in particular. Cheers, Ralf Remove superfluous dependency in Makefile.am. * Makefile.am (tests/demo-conf.test): Remove dependency

Bump copyright years in manual.

2010-08-19 Thread Ralf Wildenhues
Pushed as obvious. Bump copyright years in manual. * doc/libtool.texi: Bump copyright years. diff --git a/doc/libtool.texi b/doc/libtool.texi index f508015..d0eaec4 100644 --- a/doc/libtool.texi +++ b/doc/libtool.texi @@ -26,7 +26,7 @@ @ifnottex This file documents GNU Libtool @val

using git Libtool with Autoconf 2.59, Automake 1.9.6

2010-08-20 Thread Ralf Wildenhues
I tried running the Libtool test suites with old Autoconf 2.59 and Automake 1.9.6 (on GNU/Linux only) after 'autoreconf -fvi'-ing all but the toplevel directory with those tools. I was surprised to see that it still works more or less! :-) This patch fixes the fall-out. OK to push if the use of

Re: [PATCH] Do not absolutize paths eagerly.

2010-08-20 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Fri, Aug 20, 2010 at 06:34:58AM CEST: > * libltdl/config/ltmain.m4sh (func_resolve_sysroot): Partly > revert v2.2.10-83-gc45a288: Do not absolutize path here, only do > sysroot replacement. > (func_mode_link): Adjust. > Fixes AIX testsuite regression.

Re: versioning test on AIX

2010-08-21 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Sat, Aug 07, 2010 at 12:30:01AM CEST: > The test tries to ensure that with an incompatible library upgrade, > programs previously linked against the old library still work. This > usually works on systems where different major versions of a library > have di

Re: libtool-2.2.11a on AIX 5.3 current git master / followup on the old test suite and failures in the new testsuit

2010-08-21 Thread Ralf Wildenhues
* Rainer Tammer wrote on Sat, Aug 07, 2010 at 09:35:56AM CEST: > I have added one additional patch (as I did for the last run): > > # diff /daten/source/libtool-2.2.11a/tests/defs.in defs.in [...] > < # On AIX, shared libraries remain loaded in memory after use if they > < # are world-readable, un

  1   2   3   4   5   6   7   8   9   10   >