Re: [PATCH] [cygwin|mingw] Fix compile warnings when -std=c89.

2009-01-21 Thread Ralf Wildenhues
* Charles Wilson wrote on Wed, Jan 21, 2009 at 10:10:08PM CET: > Ralf Wildenhues wrote: > > Part (1) is easy to review: it is obvious that regressions are very > > unlikely to be system-dependent. One does get the impression that it > > might just be more efficient to

Re: [PATCH] [cygwin|mingw] Fix compile warnings when -std=c89.

2009-01-21 Thread Ralf Wildenhues
* Charles Wilson wrote on Wed, Jan 21, 2009 at 08:47:42PM CET: > Ralf Wildenhues wrote: > > I am very sorry that reviewing takes so long. Mostly this is due to > > time constraints on my side. > > On the plus side, your reviews are usually insightful, and lead to ideas &g

Re: [PATCH] [cygwin|mingw] Fix compile warnings when -std=c89.

2009-01-21 Thread Ralf Wildenhues
Hello Charles, * Charles Wilson wrote on Wed, Jan 21, 2009 at 04:21:01PM CET: > Thanks, Peter; your validation on msvc is useful and encouraging. Now > I'm just waiting for a go/no-go from one of the four libtool > maintainers: Gary Vaughan, Peter O'Gorman, Ralf Wildenhues, o

Re: [PATCH] Minor cygwin cleanup

2009-01-20 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Tue, Jan 20, 2009 at 11:31:43PM CET: > libltdl/config/ltmain.m4sh (func_generate_dlsyms): Correct > case pattern for cygwin. > --- > Ok to push? Yes, thanks! Cheers, Ralf

Re: plugins on the command line are renamed

2009-01-19 Thread Ralf Wildenhues
Hello Paul, let's drop the libtool list. * Paul Biggar wrote on Tue, Jan 20, 2009 at 03:11:25AM CET: > On Mon, Jan 19, 2009 at 10:15 PM, Ralf Wildenhues wrote: > > Please try the patch below. > > I had trouble applying this with git apply. Is it against git trunk? git

Re: Unify line endings in localization test

2009-01-19 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Mon, Jan 19, 2009 at 09:52:50PM CET: > Den 2009-01-19 21:35 skrev Ralf Wildenhues: >> >> I think we shouldn't be constantly rebasing pr-msvc-support. I think >> instead when we can't decide soonish whether to merge it into mast

Re: [PATCH] [cygwin|mingw] fix dlpreopen with --disable-static take 2

2009-01-19 Thread Ralf Wildenhues
Hello Charles, I haven't looked at your patches in detail yet, but a couple of things caught my eye: * Charles Wilson wrote on Sat, Jan 03, 2009 at 02:39:15AM CET: > diff --git a/libltdl/config/general.m4sh b/libltdl/config/general.m4sh > index 4bc304c..c4de91a 100644 > --- a/libltdl/config/gener

Re: [PATCH] [cygwin|mingw] fix dlpreopen with --disable-static take 3

2009-01-19 Thread Ralf Wildenhues
ecute-mode.at +++ b/tests/execute-mode.at @@ -1,6 +1,6 @@ # execute-mode.at -- libtool --mode=execute -*- Autotest -*- # -# Copyright (C) 2008 Free Software Foundation, Inc. +# Copyright (C) 2008, 2009 Free Software Foundation, Inc. # Written by Ralf Wildenhues, 2008 # # T

Re: set_program_name behavior

2009-01-19 Thread Ralf Wildenhues
Hello Bruno, * Bruno Haible wrote on Wed, Jan 14, 2009 at 10:51:46PM CET: > > The removal of */.libs/lt- prefixes is done so that "make check" produces > reliable results, and because the precise location of the executable used > under the hood by libtool is a detail that the user should not know

Re: spaces

2009-01-19 Thread Ralf Wildenhues
Hi Akim, * Akim Demaille wrote on Mon, Jan 19, 2009 at 09:44:37AM CET: > Le 10 janv. 09 à 22:25, Ralf Wildenhues a écrit : > >> Maybe we should update HACKING about the style we decide about? >> (note this is really just bike shedding, and babbling) > > I definitely th

Re: [PATCH] GNU/kOpenSolaris port

2009-01-19 Thread Ralf Wildenhues
4 (LT_SYS_DLOPEN_DEPLIBS): Likewise. * NEWS, THANKS: Update. 2009-01-19 Ralf Wildenhues Another LT_AT_EXEC_CHECK fixup. diff --git a/NEWS b/NEWS index 00bd1bb..5116598 100644 --- a/NEWS +++ b/NEWS @@ -12,8 +12,12 @@ New in 2.2.8 2009-??-??: git version 2.2.7a, Libtool team: - New l

Re: plugins on the command line are renamed

2009-01-19 Thread Ralf Wildenhues
Hello Paul, * Paul Biggar wrote on Sun, Jan 11, 2009 at 11:04:05PM CET: > I hope its not too soon to ping on this? No, pinging is certainly ok, and, sigh, unfortunately necessary. :-/ > On Wed, Dec 31, 2008 at 1:22 PM, Paul Biggar wrote: > > On Wed, Dec 31, 2008 at 8:43 AM, Ral

Another LT_AT_EXEC_CHECK fixup.

2009-01-19 Thread Ralf Wildenhues
Hello, I've had this really small patch to fix a couple of omissions. Seems obvious, pushed. Cheers, Ralf Another LT_AT_EXEC_CHECK fixup. * tests/shlibpath.at (shlibpath_overrides_runpath): Use LT_AT_EXEC_CHECK instead of AT_CHECK. diff --git a/tests/shlibpath.at b/tests/shlibp

Re: use $EXEEXT consistently in new testsuite

2009-01-19 Thread Ralf Wildenhues
pyright (C) 2005, 2007, 2008 Free Software Foundation, Inc. +# Copyright (C) 2005, 2007, 2008, 2009 Free Software Foundation, Inc. # Written by Ralf Wildenhues, 2005 # # This file is part of GNU Libtool. @@ -263,7 +263,7 @@ EOF # libgcj.spec or unable to find it. Skip this test for them.

[PATCH] func_version copes with multi-line copyright headers.

2009-01-19 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Mon, Jan 19, 2009 at 09:40:20PM CET: > The long line in ltmain.m4sh is actually necessary in order for > libtool --version > > to work correctly. Will address in a followup patch. OK to push? Thanks, Ralf func_version copes with multi-line copyr

happy late new year

2009-01-19 Thread Ralf Wildenhues
Pushed to master, sorry for the long delay. The long line in ltmain.m4sh is actually necessary in order for libtool --version to work correctly. Will address in a followup patch. Cheers, Ralf Bump copyright years. * ChangeLog.2008: New, rotated from ... * ChangeLog: ... here

Re: Unify line endings in localization test

2009-01-19 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Sat, Jan 17, 2009 at 12:42:34AM CET: > In the localization test, the output of the compiler is compared > with the grepped output of the compiler when run through libtool. > > MSVC outputs its messages with \r\n, but the grep in the above > strips that down to just

Re: [SCM] GNU Libtool branch, pr-msvc-support, updated. v2.2.6-83-g8c17887

2009-01-14 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Wed, Jan 14, 2009 at 09:13:23AM CET: > > The branch, pr-msvc-support has been updated > >via 8c17887ee34e73a2aeb127b94f5b76f45dc34017 (commit) > >via 2817364bb6efd20192c46edecfe085cbb288 (commit) > >via 06cfce005204bb8ca212aadab38b38c

Re: Rebase of the pr-msvc-support branch?

2009-01-11 Thread Ralf Wildenhues
Hello Markus, * Duft Markus wrote on Mon, Jan 12, 2009 at 08:29:48AM CET: > > Maybe on windows, we should recommend using interix? You can't be serious, on a GNU mailing list, suggesting that we recommend a proprietary system over a free one.

Re: Provide a means to activate dynamically the debug traces

2009-01-10 Thread Ralf Wildenhues
* Akim Demaille wrote on Fri, Jan 09, 2009 at 08:56:20PM CET: > Le 9 janv. 09 à 19:40, Ralf Wildenhues a écrit : >> >> You can instead use >> libtool --mode=execute -dlopen module.la $program $args > > Are you saying that the behavior is expected? Why? Why not act

Re: Rebase of the pr-msvc-support branch?

2009-01-09 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Fri, Jan 09, 2009 at 06:37:18PM CET: > It will solve a lot of problems if autotools can be used to > build real Windows programs. Yes, but we need to address the efficiency issue. For example, dropped the

Re: Provide a means to activate dynamically the debug traces

2009-01-09 Thread Ralf Wildenhues
Hi Akim, * Akim Demaille wrote on Fri, Jan 09, 2009 at 04:37:32PM CET: > > when I dlopen a module.la which has a dependency.la library, I > expected ltdl to automatically find dependency.la which is not > installed. Yet it does not seem to use the module.la's deplibs to > dlopen them, it deleg

Re: Rebase of the pr-msvc-support branch?

2009-01-08 Thread Ralf Wildenhues
* Eric Blake wrote on Fri, Jan 09, 2009 at 03:58:36AM CET: > According to Peter Rosin on 1/8/2009 12:43 PM: > > Should I kill the branch on savannah, rebase my local tree and finally > > push a brand new pr-msvc-support branch? > Now that you've told us the branch will be rebased, it makes sense t

Re: Fix distcc/ccache interferences with the test suite

2009-01-05 Thread Ralf Wildenhues
Hi Akim, * Akim Demaille wrote on Mon, Jan 05, 2009 at 03:22:24PM CET: > Hi all, Are those the only distcc/ccache-induced failures? Thanks, Ralf > From 02a3220b061f232d30baf1d1d3bc497d7e9417b8 Mon Sep 17 00:00:00 2001 > From: Akim Demaille > Date: Mon, 5 Jan 2009 15:10:55 +0100 > Subject: [PAT

Re: [PATCH] GNU/kOpenSolaris port

2008-12-29 Thread Ralf Wildenhues
* Robert Millan wrote on Tue, Dec 30, 2008 at 01:22:15AM CET: > On Mon, Dec 29, 2008 at 11:40:55PM +0100, Ralf Wildenhues wrote: > > > > Secondly, at a glance I don't see support for your system's canonical > > name in config.guess/config.sub. Have you already sent

Re: [PATCH] GNU/kOpenSolaris port

2008-12-29 Thread Ralf Wildenhues
Hello Robert, thank you for the patch and the testing. * Robert Millan wrote on Mon, Dec 22, 2008 at 03:40:34AM CET: > > I ported libtool to GNU/kOpenSolaris [1]. Because the checks are > always the same for all Glibc-based systems, and having to wait for > libtool to propagate to every package

Re: Avoid accidental disastrous space cleanups

2008-12-19 Thread Ralf Wildenhues
Hello Eric, Akim, * Eric Blake wrote on Fri, Dec 19, 2008 at 02:26:57PM CET: > According to Akim Demaille on 12/19/2008 5:08 AM: > > I have been bitten by this :( > > So has autoconf, which is why all m4sugar scripts currently start with the > shorter: > > IFS=" "" $as_nl" > > Which makes

Re: GCC_NO_EXECUTABLES and Libtool

2008-12-18 Thread Ralf Wildenhues
Hello Paolo, Peter, * Peter O'Gorman wrote on Tue, Dec 16, 2008 at 07:15:06AM CET: > On 13-Dec-08, at 7:45 AM, Ralf Wildenhues wrote: >> Add cache variables to link tests. >> * libltdl/m4/libtool.m4 (_LT_SYS_DYNAMIC_LINKER) >> (_LT_LINKER_SHLIBS): Add cac

[patch #6691] FreeMiNT support to libtool 1.5.26 (tested)

2008-12-14 Thread Ralf Wildenhues
Follow-up Comment #1, patch #6691 (project libtool): Thanks for the patch. We'd much prefer it if you redid the patch against the current git tree (or 2.2.6 if you like), ran the testsuite (make -k check; see README for details and verbose output), and posted verbose output for all failed test g

GCC_NO_EXECUTABLES and Libtool

2008-12-13 Thread Ralf Wildenhues
ned_flag |else | cat conftest.err 1>&5 |fi |$RM conftest* | - AC_MSG_RESULT([$_LT_TAGVAR(archive_cmds_need_lc, $1)]) | + ]) | + _LT_TAGVAR(archive_cmds_need_lc, $1)=$lt_cv_[]_LT_TAGVAR(archive_cmds_need_lc, $1) |;; |

Re: Typo in the documentation?

2008-11-28 Thread Ralf Wildenhues
Hello Paolo, Akim, * Paolo Bonzini wrote on Fri, Nov 28, 2008 at 04:41:23PM CET: > Akim Demaille wrote: > > Do you really mean --recheck? In that case there is no reason for > > config.status to recreate libtool, unless I missing something. Don't > > you mean "./config.status libtool"? > > Seei

Re: [SCM] GNU Libtool branch, master, updated. v2.2.6-56-g9c9cde2

2008-11-28 Thread Ralf Wildenhues
[ moving from libtool-commit ] Hi Paolo, * Paolo Bonzini wrote on Fri, Nov 28, 2008 at 04:10:33PM CET: > Do not use $ECHO within eval-ed `...`, as %s\\n becomes %s\n. > > * HACKING: Document func_echo_all. > * libltdl/m4/libtool.m4: Use it within backticks that appear in > ev

Re: use $EXEEXT consistently in new testsuite

2008-11-27 Thread Ralf Wildenhues
Hi Roumen, * Roumen Petrov wrote on Wed, Nov 26, 2008 at 12:46:00AM CET: > I will check this failure: > 28: Runpath in libtool library filesFAILED > (runpath-in-lalib.at:59) Please post tests/testsuite.log, so we can see the verbose failure output. You can also run make check

Re: $ECHO change uncovered underquoting

2008-11-26 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Wed, Nov 26, 2008 at 08:54:29AM CET: > > > 1) Add more escaping to the input lines. E.g., > > _LT_TAGVAR(archive_cmds, $1)='$CC -o $lib $libobjs $compiler_flags > > `$ECHO "$deplibs" | $SED '\''s/ -lc$//'\''` -link -dll~linknames=' > > > > would become > > _

Re: [PATCH] [cygwin|mingw] fix dlpreopen with --disable-static

2008-11-24 Thread Ralf Wildenhues
Hello Brian, * Brian Dessent wrote on Thu, Nov 13, 2008 at 11:16:24PM CET: > Ralf Wildenhues wrote: > > > I'm actually not sure whether _GLOBAL__F[ID]_.* can appear on w32. > > Do you know? They should happen with C++ code using constructors > > and destructors

Re: Revenge of the $ECHO. Kill most uses of Xsed.

2008-11-24 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Sun, Nov 16, 2008 at 11:48:01PM CET: > There is at least one more place with problematic usage of $ECHO, in > func_infer_tag (several instances in two lines). Not yet sure how to > fix that in a concise way. Like this. I also added some initial testsuite

Fix expout and experr line ending issues.

2008-11-23 Thread Ralf Wildenhues
> * Roumen Petrov wrote on Tue, Nov 18, 2008 at 11:12:56PM CET: > > >> Test 48 "enforced lib prefix" FAILED (need_lib_prefix.at:184) - again > >> "--mode install" problem (see test 29). > > It is not "--mode install" problem. It is related to end of line. Fixed like this. Cheers, Ralf

Relax link-order2 test for systems with partial wrong order.

2008-11-23 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Wed, Nov 19, 2008 at 07:42:02AM CET: > * Roumen Petrov wrote on Tue, Nov 18, 2008 at 11:12:56PM CET: > > [SNIP] > >> Test 25 "Link order of deplibs" skipped (link-order2.at:129) - in my > >> todo for investigation. > > >

use $EXEEXT consistently in new testsuite

2008-11-23 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Sun, Nov 23, 2008 at 10:41:46PM CET: > > To fix the loose end wrt. the exec checks, there was some exit status > normalization needed here. I think this patch should be sufficient, > once the $EXEEXT patch is in (comes next). Here it is. I went over te

Re: testsuite.at for version 2.x Was: ... mingw-cross, check-local fail for DESTDIR tests

2008-11-23 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Wed, Nov 19, 2008 at 07:42:02AM CET: > > >> Test 29 "static linking flags for programs" skipped (static.at:177) - > >> problem is with "libtool --mode install" since real executable is in > >> .libs subdirectory.

No trailing period in Autotest test group names.

2008-11-23 Thread Ralf Wildenhues
Software Foundation, Inc. # Written by Alexandre Oliva & Ralf Wildenhues, 2005 # # This file is part of GNU Libtool. @@ -23,7 +23,7 @@ -AT_SETUP([Link order test.]) +AT_SETUP([Link order test]) AT_KEYWORDS([libtool]) eval `$LIBTOOL --config | $EGREP '^(hardco

Re: [PATCH 6/6] use print or printf or cat as $ECHO (really)

2008-11-23 Thread Ralf Wildenhues
* Eric Blake wrote on Sun, Nov 23, 2008 at 03:26:41PM CET: > According to Ralf Wildenhues on 11/23/2008 1:10 AM: > > Are we certain enough no bash or zsh version has problems with a long > > format string? > > I'm not certain they are bug-free on all possible form

Avoid newlines in some AT_CHECK commands.

2008-11-23 Thread Ralf Wildenhues
It bothered me while testing these test groups that TESTSUITEFLAGS='-v -x' doesn't show the trace output for these commands. Fixed with this trivial patch by avoiding newlines inside, pushed. Cheers, Ralf Avoid newlines in some AT_CHECK commands. * tests/standalone.at (linking li

Replace uses of $SED $basename.

2008-11-23 Thread Ralf Wildenhues
Further simplification: no more $SED $basename outside of func_*basename. Pushed. Cheers, Ralf Replace uses of $SED $basename. * libltdl/config/ltmain.m4sh (func_mode_link): Replace instances of `$ECHO ... | $SED' with calls to func_basename. * libtoolize.m4sh (fu

No need to prepend $progname with './'.

2008-11-23 Thread Ralf Wildenhues
The new $ECHO allows for this simplification, pushed. Cheers, Ralf No need to prepend $progname with './'. * libltdl/config/general.m4sh (func_dirname_and_basename): $ECHO copes with leading hyphens now, so do not mangle $progname any more. diff --git a/libltdl/config/genera

Re: Revenge of the $ECHO. Kill most uses of Xsed.

2008-11-23 Thread Ralf Wildenhues
I've fixed the issues with the first version of the $Xsed killer patch now, with the incremental diff as shown below, and pushed the combined result. Cheers, Ralf diff --git a/libltdl/config/general.m4sh b/libltdl/config/general.m4sh index ce80241..9fe5873 100644 --- a/libltdl/config/general.m4sh

Re: [OMPI devel] RFC: Add SunStudio/Libtool helper script for post-configure

2008-11-23 Thread Ralf Wildenhues
[ Libtool list readers: this thread is ] I'm applying this patch to Libtool master, to fix $wl for the Sun Ceres Fortran compiler. Cheers, Ralf Fix matching of Sun Fortran compiler on Linux. * libltdl/m4/libtool.m

Re: [PATCH 6/6] use print or printf or cat as $ECHO (really)

2008-11-23 Thread Ralf Wildenhues
* Eric Blake wrote on Sat, Nov 22, 2008 at 03:37:03PM CET: > According to Eric Blake on 11/22/2008 7:31 AM: > > Maybe a better suggested test would be one that checks for either print or > > printf (that way, a Solaris machine will let ksh pass the suggested test). > >Since the overall test is

Re: [PATCH 6/6] use print or printf or cat as $ECHO (really)

2008-11-22 Thread Ralf Wildenhues
Hello again, * Paolo Bonzini wrote on Mon, Nov 10, 2008 at 02:16:03PM CET: > * libltdl/m4/libtool.m4 (LT_INIT): Add _LT_SHELL_INIT to > work around Autoconf <2.64 bug. > (_LT_OUTPUT_LIBTOOL_COMMANDS_INIT): Add func_fallback_echo. > Eliminate lt_ECHO requoting. > (_LT_SHELL_INIT): Use a public M4sh

Re: [PATCH] Fix --verbose option and add new --no-silent option

2008-11-21 Thread Ralf Wildenhues
Hi Charles, apologies: I actually wrote this reply a few days ago, and then forgot to send it. :-( Sorry. * Charles Wilson wrote on Mon, Nov 17, 2008 at 02:31:56AM CET: > +Thus, to restore the default state, the following two options > +must be given (in order): > [EMAIL PROTECTED] > [EMAIL PROT

Fix match patterns for cegcc*.

2008-11-19 Thread Ralf Wildenhues
In Libtool, we commonly match $host names with a trailing '*' to allow for (present or future) version numbers, on the grounds that most systems remain compatible with their older versions, and to let Libtool code require fewer changes. I'm thus pushing this to match more uniformly for cegcc. Che

Re: 2.2.6: libltdlcS.c:22: error: 'lt_ptr' undeclared here (not in a function)

2008-11-19 Thread Ralf Wildenhues
[ moving from bug-libtool ] * Ralf Wildenhues wrote on Mon, Nov 17, 2008 at 10:42:09PM CET: > * Akim Demaille wrote on Mon, Nov 17, 2008 at 09:38:56PM CET: > > > > Actually, grepping inside my original 2.2.6 tarball does find quite a > > few lt_ptr (and not only in the ch

Re: testsuite.at for version 2.x Was: ... mingw-cross, check-local fail for DESTDIR tests

2008-11-18 Thread Ralf Wildenhues
Hi Roumen, let's drop bug-libtool from the Cc:. * Roumen Petrov wrote on Tue, Nov 18, 2008 at 11:12:56PM CET: > [SNIP] >> Test 25 "Link order of deplibs" skipped (link-order2.at:129) - in my >> todo for investigation. > > Still skipped. The executable exit code is 1 instead 2. No idea what is

Re: Revenge of the $ECHO. Kill most uses of Xsed.

2008-11-18 Thread Ralf Wildenhues
* Eric Blake wrote on Wed, Nov 19, 2008 at 06:12:27AM CET: > >>> - RM="$ECHO $RM" > >>> + RM="echo $RM" > >> Are all of these safe, considering mingw might have \ in an absolute > >> pathname to these tools? > > > > No, and I'm glad you ask about this. The poin

Re: Revenge of the $ECHO. Kill most uses of Xsed.

2008-11-16 Thread Ralf Wildenhues
Hi Eric, * Eric Blake wrote on Mon, Nov 17, 2008 at 12:49:25AM CET: > According to Ralf Wildenhues on 11/16/2008 3:48 PM: > > -my_directory_path=`$ECHO "X$my_directory_path" | $Xsed -e > > "$dirname"` > > +my_directory_path=`$ECHO

Fix $ECHO abuse exposed by recent patch.

2008-11-16 Thread Ralf Wildenhues
The $ECHO changes (now using printf on GNU/Linux) exposed a buglet, fixed as below. Cheers, Ralf Fix $ECHO abuse exposed by recent patch. * libltdl/m4/libtool.m4 (_LT_SYS_DYNAMIC_LINKER): Require _LT_CHECK_SHELL_FEATURES, for $lt_NL2SP. Always quote argument to $ECHO. diff

Re: [PATCH] Fix --verbose option and add new --no-silent option

2008-11-16 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Sat, Nov 15, 2008 at 10:40:26AM CET: > * libltdl/config/ltmain.m4sh (usage): Document > new --no-silent/--no-quiet options. Spacing changes. > (func_enable_tag): Handle --no-silent/--no-quiet options. > Modified --verbose to actually activate opt_verbose. > Ne

Re: executable wrapper on mingw mangles arguments

2008-11-16 Thread Ralf Wildenhues
Hi Bruno, * Bruno Haible wrote on Fri, Nov 14, 2008 at 12:47:44PM CET: > Find attached the resulting logs (stdout+stderr, and > tests/testsuite.dir/36/testsuite.log). > > The test passes. But it also passes when I comment out the line > newargv = prepare_spawn (newargv); > in ltmain.sh and do a

Re: [PATCH] [cygwin|mingw] fix dlpreopen with --disable-static

2008-11-13 Thread Ralf Wildenhues
* [EMAIL PROTECTED] wrote on Fri, Nov 14, 2008 at 12:28:36AM CET: > > The point is, we perhaps STARTED with the .la file, but the whole point > of the dlpreopen $pass is to replace each .la file in $dlprefiles with > the name of the object from which the symbols should be extracted, to > build the

Re: [PATCH] [cygwin|mingw] fix dlpreopen with --disable-static

2008-11-13 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Thu, Nov 13, 2008 at 06:09:20AM CET: > Ralf Wildenhues wrote: > > > > Well, --verbose is documented to be a reversal of --silent, and > > documented to be the default. The fact that opt_verbose is never set is > > a limitation. I

Re: [PATCH] [cygwin|mingw] fix dlpreopen with --disable-static

2008-11-12 Thread Ralf Wildenhues
Hello Charles, thanks for the patch! Quoting a bit out of order: * Charles Wilson wrote on Sat, May 31, 2008 at 07:01:45PM CEST: > * libltdl/config/ltmain.m4sh (func_enable_tag): allow > --verbose to set opt_verbose [...] > A) libtool --verbose does not actually set opt_verbose. In fact, >no

Re: testsuite.at for version 2.x Was: ... mingw-cross, check-local fail for DESTDIR tests

2008-11-12 Thread Ralf Wildenhues
Let's drop bug-libtool. * Roumen Petrov wrote on Wed, Nov 12, 2008 at 10:01:56PM CET: > Please find attached file "convenience.at-log" Thanks. > [SRCDIR]/tests/convenience.at:61: $LIBTOOL --mode=link $CC $CFLAGS $LDFLAGS > -o main main$i.lo liba$conv.la > stderr: > stdout: > libtool: link: i386

Re: libtool(20080421), mingw-cross, check-local fail for DESTDIR tests

2008-11-12 Thread Ralf Wildenhues
Hi Roumen, * Roumen Petrov wrote on Wed, Nov 12, 2008 at 08:41:09PM CET: > Ralf Wildenhues wrote: >> --- a/tests/destdir.at >> +++ b/tests/destdir.at >> @@ -56,7 +56,7 @@ $LIBTOOL --mode=compile $CC $CPPFLAGS $CFLAGS -c a.c > [SNIP] > >> AT_CHECK([$LIBTOO

Re: help2man missing

2008-11-12 Thread Ralf Wildenhues
Hi Bob, * Bob Friesenhahn wrote on Wed, Nov 12, 2008 at 06:33:12PM CET: > While building libtool HEAD today, I encountered this problem: > > PATH=.:$PATH; export PATH; /bin/bash > /home/bfriesen/src/gnu/libtool-head/libltdl /config/missing --run > help2man --output=/home/bfriesen/src/gnu/libto

Use -no-undefined in install tests.

2008-11-11 Thread Ralf Wildenhues
Another trivial testsuite failure with w32 (cross compiled; the failure is exposed with the pending *EXEC_CHECK patch applied). Pushed. Cheers, Ralf Use -no-undefined in install tests. * tests/install.at (Install tests): Use `-no-undefined'. diff --git a/tests/install.at b/tests

Re: testsuite.at for version 2.x Was: ... mingw-cross, check-local fail for DESTDIR tests

2008-11-11 Thread Ralf Wildenhues
Hello Roumen, all, apologies for the huge delay. * Roumen Petrov wrote on Sat, May 10, 2008 at 12:25:48PM CEST: > The libtool version before 2.x (as example 1.5x) in mingw-cross > compilation environment create executables as follow: > - foo : wrapper shell script > - foo.exe : libtool wrapper

Do not use `test ... -a ...'.

2008-11-11 Thread Ralf Wildenhues
Spotted, and pushed this trivial patch. (Yes, on HP-UX all shells know this, but being consistent is easy here.) Cheers, Ralf Do not use `test ... -a ...'. * libltdl/m4/libtool.m4 (_LT_LINKER_SHLIBS) [hpux10, hpux11]: Replace test -a with test && test. diff --git a/liblt

Re: libtool(20080421), mingw-cross, check-local fail for DESTDIR tests

2008-11-11 Thread Ralf Wildenhues
Hello Charles, Roumen, * Charles Wilson wrote on Sat, Nov 08, 2008 at 05:12:18PM CET: > Roumen Petrov wrote: > > With attached patch now I get for libtool 2.2.4: > > DESTDIR tests > > 38: Simple DESTDIR install ok > > 39: DESTDIR with in-package deplibs o

Re: Simplify cwrapper handling a bit

2008-11-11 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Tue, Nov 11, 2008 at 07:14:54AM CET: > Ralf Wildenhues wrote: > > OK to apply? > > Not this bit: Thanks for the reminder, and the speedy review. I left out that hunk, fixed a couple more coding style issues, and committed this. Cheers, Ral

Re: [PATCH 6/6] use print or printf or cat as $ECHO (really)

2008-11-11 Thread Ralf Wildenhues
Hi Paolo, * Paolo Bonzini wrote on Mon, Nov 10, 2008 at 09:58:32PM CET: > > > > Regarding the 6/6 patch, you have posted several variants, also there > > are pending issues. For me it'd be easiest to review the final one > > when things have been hashed out; I think I've voiced all concerns tha

Re: [PATCH 6/6] use printf as $ECHO

2008-11-10 Thread Ralf Wildenhues
Hello Sven, thanks for the quick feedback! * Sven Mascheck wrote on Tue, Nov 11, 2008 at 01:00:26AM CET: > On Mon, Nov 10, 2008 at 08:34:31PM +0100, Ralf Wildenhues wrote: > > > func_fallback_echo() > > { > > eval 'cat <<_LTEOF > > $* > > _LTE

Simplify cwrapper handling a bit

2008-11-10 Thread Ralf Wildenhues
OK to apply? FWIW, IFS splitting of $list in for item in $list; do ... done is done once, at the beginning of the list. Thanks, Ralf 2008-11-11 Ralf Wildenhues <[EMAIL PROTECTED]> Simplify cwrapper handling a bit. * libltdl/config/ltmain.m4sh (func_emit_wrapper

Re: executable wrapper on mingw mangles arguments

2008-11-10 Thread Ralf Wildenhues
[ Cc:ing the list ] * Bruno Haible wrote on Tue, Nov 11, 2008 at 12:21:10AM CET: > > > If you prefer a tarball for testing, then I can also just push the > > combined patch and you can use the nightly build from the Libtool > > web page. > > Yes please. It's safer and less troublesome if I can u

Re: executable wrapper on mingw mangles arguments

2008-11-10 Thread Ralf Wildenhues
Hello Bruno, Charles, apologies for the long delay. * Bruno Haible wrote on Sat, Oct 11, 2008 at 02:38:31AM CEST: > > +for arg in \ > > + 'def ghi' \ > > + 'd"e' 'f"g' \ > > + 'd\"e' 'f\"g' \ [...] > Well, I meant these as pairs of arguments. When you pass just one argument > > d"e >

Update to GFDL 1.3.

2008-11-10 Thread Ralf Wildenhues
I've updated the manual to use version 1.3 of the FDL, as recommended by the FSF. Cheers, Ralf Update to GFDL 1.3. * doc/fdl.texi: Update to GFDL 1.3. * doc/libtool.texi: Adjust. * NEWS: Update. diff --git a/NEWS b/NEWS index 33ee277..4945c51 100644 --- a/NEWS +++

Re: [PATCH 6/6] use print or printf or cat as $ECHO (really)

2008-11-10 Thread Ralf Wildenhues
Hi Paolo, * Paolo Bonzini wrote on Mon, Nov 10, 2008 at 02:16:03PM CET: > > Regarding the other patches, I retested them successfully using Autoconf > 2.59 for the tests/*demo* directories. Thanks. > When pushing I got this error > > error: unable to create temporary sha1 filename ./objects/9

Re: [PATCH 6/6] use printf as $ECHO

2008-11-10 Thread Ralf Wildenhues
[ ] Hello Sven, below is a question regarding your shell pages. * Paolo Bonzini wrote on Mon, Nov 10, 2008 at 12:07:12PM CET: > > > Esp., I'm quite sure that we need the cat here-document fallback; > > [...] I am quite con

Re: [PATCH 6/6] use printf as $ECHO

2008-11-09 Thread Ralf Wildenhues
Hello, I haven't fully reviewed this patch yet; there are more issues with it. However: * Paolo Bonzini wrote on Sat, Nov 08, 2008 at 11:13:32AM CET: > > >> M4SH_VERBATIM([[ > > > > Hmm. This is a nicer name for m4_echo, for how it is being used; maybe it > > is > > time to migrated it into

Re: executable wrapper on mingw mangles arguments

2008-11-09 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Sun, Nov 09, 2008 at 09:11:28PM CET: > Ralf Wildenhues wrote: > > I think instead of Bruno's patch a much simpler fix is possible by > > including a different header: > > <http://thread.gmane.org/gmane.comp.gnu.mingw.user/27968/fo

Re: [PATCH 4/6] tweak to config.status->libtool quoting (correct?)

2008-11-09 Thread Ralf Wildenhues
* Paolo Bonzini wrote on Fri, Nov 07, 2008 at 01:02:44PM CET: > * libltdl/m4libtool.m4 (_LT_OUTPUT_LIBTOOL_COMMANDS_INIT): Double > quote ECHO-ed variable. At one point, I was fairly certain that this whole section could be done *a lot* simpler and more efficient. I forgot the details about what

Re: [PATCH 3/6] use documented m4sh interfaces

2008-11-09 Thread Ralf Wildenhues
I already approved this, but ... * Paolo Bonzini wrote on Fri, Nov 07, 2008 at 09:40:42AM CET: > --- a/libltdl/m4/libtool.m4 > +++ b/libltdl/m4/libtool.m4 > @@ -553,6 +553,34 @@ esac > _LT_OUTPUT_LIBTOOL_INIT > ]) > > +# AS_INIT_GENERATED(FILE, [COMMENT]) > +# -

Re: [PATCH 1/6] Avoid that autom4te complains about unexpanded macros

2008-11-09 Thread Ralf Wildenhues
Hi Paolo, Eric, * Paolo Bonzini wrote on Fri, Nov 07, 2008 at 08:58:24AM CET: > This patch series has the purpose of removing the "fallback echo" > code from Libtool, replacing it with usage of printf. More detailed > information on the tradeoffs and the rationale are in the description > of patc

Re: executable wrapper on mingw mangles arguments

2008-11-09 Thread Ralf Wildenhues
Hello Charles, Bruno, * Charles Wilson wrote on Sat, Nov 08, 2008 at 05:16:03PM CET: > Bruno Haible wrote: > > This is a severe regression in libtool 2.x w.r.t. 1.5.26. On mingw, several > > of the gettext tests fail because the executable wrapper created by the > > libtool script does two bad thi

Re: [2.2.6] old tests checking for allow_undefined=unsupported

2008-10-28 Thread Ralf Wildenhues
Hello Michael, thanks for the bug report. * Michael Haubenwallner wrote on Tue, Oct 28, 2008 at 04:12:40PM CET: > > maybe I'm just puzzled, but to me it seems there are the same two > problems in each tests/cdemo-undef.test and tests/tagdemo-undef.test. Yep. > Consider this line: > gre

Parallel make fix in testsuite.

2008-10-26 Thread Ralf Wildenhues
Hello, with MAKE="make -j3", the new testsuite would cause spurious failure because libtool isn't created strictly before its use. This patch fixes it. Pushed. I hope that was ok. Cheers, Ralf * tests/testsuite.at (_LTDL_PROJECT_FILES): Fix dependencies for parallel make. dif

Fix typo in libtoolize error message.

2008-10-26 Thread Ralf Wildenhues
Hello, I've pushed this trivial patch. Cheers, Ralf * libtoolize.m4sh (func_nonemptydir_p): Fix typo in error message. diff --git a/libtoolize.m4sh b/libtoolize.m4sh index b6d35f2..f74cd7c 100644 --- a/libtoolize.m4sh +++ b/libtoolize.m4sh @@ -1389,7 +1389,7 @@ func_nonemptydir_

support for ports in libtool (was: libtool wrapper: mingw32ce does not support errno)

2008-10-09 Thread Ralf Wildenhues
Hello Vincent, Eric, all, * Vincent Torri wrote on Thu, Oct 09, 2008 at 03:08:20PM CEST: > On Thu, 9 Oct 2008, Eric Blake wrote: >> According to Vincent Torri on 10/9/2008 12:22 AM: You've gone this far now, how about you try to redo the patch based on the comment above, then try to writ

Re: libtool wrapper: mingw32ce does not support errno

2008-10-08 Thread Ralf Wildenhues
Hello Vincent, * Vincent Torri wrote on Thu, Oct 09, 2008 at 07:02:56AM CEST: > > here is a patch that removes compilation errors with mingw32ce because it > does not support the errno system. Is __MINGW32CE__ even defined for that system? Because the patch just looks like it's doing exactly th

Re: problem when cross compiling with mingw32ce

2008-10-05 Thread Ralf Wildenhues
armv4, flags 0x0039: > > But grep is used to check "i386", and not arm-wince-little. Please post the output of /home/torri/local/opt/mingw32ce/bin/arm-mingw32ce-nm -B -f posix -A \ /home/torri/local/opt/mingw32ce/arm-mingw32ce/lib/libws2.a | sed 100q and try the patch

Re: [PATCH] Strip DWARF-2 debug information for libraries on Darwin.

2008-10-05 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Sun, Oct 05, 2008 at 08:04:44PM CEST: > > Do all conceivable strip programs on darwin understand -S? Or you could use $old_striplib, on the theory that when this needs fixing, at least you will only have to fix one location. Cheers, Ralf

Re: [PATCH] Strip DWARF-2 debug information for libraries on Darwin.

2008-10-05 Thread Ralf Wildenhues
Hi Peter, * Peter O'Gorman wrote on Sun, Oct 05, 2008 at 07:03:24PM CEST: > > I don't really like running strip (or dsymutil) on the library, but the > debug information that is stored in the library/executable is really > only paths to and uuids of the object files, and libtool may have > remove

Re: libtool 2.2.6 does not install libraries correctly on HP-UX

2008-10-05 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Wed, Oct 01, 2008 at 11:20:00PM CEST: > [ moving from bug-libtool; this is > <http://thread.gmane.org/gmane.comp.gnu.libtool.bugs/6539> ] [...] > Atomic shared library install permissions on HP-UX. > > The HP-UX 11 runtime linker

Re: libtool 2.2.6 does not install libraries correctly on HP-UX

2008-10-01 Thread Ralf Wildenhues
mode=555 ;; interix[[3-9]]*) @@ -2706,6 +2708,8 @@ _LT_DECL([], [library_names_spec], [1], The last name is the one that the linker finds with -lNAME]]) _LT_DECL([], [soname_spec], [1], [[The coded name of the library, if different from the real name]]) +_LT_DECL([], [install_override_mode],

Re: avoid gcc compiler warning

2008-09-29 Thread Ralf Wildenhues
Hi Eric, * Eric Blake wrote on Mon, Sep 29, 2008 at 01:58:41PM CEST: > According to Ralf Wildenhues on 9/26/2008 10:03 AM: > >> > >> cc1: warnings being treated as errors > >> ../ltdl/loaders/loadlibrary.c: In function 'vm_open': > >> ../ltdl

Re: avoid gcc compiler warning

2008-09-26 Thread Ralf Wildenhues
Hi Eric, * Eric Blake wrote on Fri, Sep 26, 2008 at 05:57:06PM CEST: > When I upgraded from 3.4.4 to gcc 4.3.2, I started seeing these warnings on > cygwin: > > cc1: warnings being treated as errors > ../ltdl/loaders/loadlibrary.c: In function 'vm_open': > ../ltdl/loaders/loadlibrary.c:159: erro

Re: libtool man page

2008-09-20 Thread Ralf Wildenhues
* Peter O'Gorman wrote on Sat, Sep 20, 2008 at 05:11:19AM CEST: > > This looks ok to me, thank you. Thanks. Now that it's a new morning, I see several shortcomings: * --help-all was not documented at all, * -h not document in .texi, * I forgot finish mode in --help-all, * -Wc,FLAG was not docume

Re: libtool man page

2008-09-19 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Fri, Sep 19, 2008 at 08:59:37PM CEST: > * Peter O'Gorman wrote on Fri, Sep 19, 2008 at 08:42:38PM CEST: > > > > I have not applied the patch to check, but doesn't this just give a > > manpage with the default --help output, and no informa

Re: libtool man page

2008-09-19 Thread Ralf Wildenhues
Hi Peter, * Peter O'Gorman wrote on Fri, Sep 19, 2008 at 08:42:38PM CEST: > Ralf Wildenhues wrote: > > > > OK to install and put Karl in THANKS? > > I have not applied the patch to check, but doesn't this just give a > manpage with the default --help output, an

Re: libtool man page

2008-09-19 Thread Ralf Wildenhues
[ moving from bug-libtool ] * Ralf Wildenhues wrote on Thu, Sep 18, 2008 at 07:11:58AM CEST: > * Karl Berry wrote on Sat, Sep 06, 2008 at 01:03:44AM CEST: > > Perhaps it would be worthwhile creating/installing a man page for > > libtool with help2man, as we do for so many

Re: [SCM] GNU Libtool branch, master, updated. v2.2.4-43-gcbe0c8e

2008-09-07 Thread Ralf Wildenhues
* Gary V. Vaughan wrote on Sun, Sep 07, 2008 at 10:12:30AM CEST: > On 7 Sep 2008, at 15:21, Ralf Wildenhues wrote: >>>>> +# HELP?!? How to check for git push conflicts? >> Doesn't `git push --dry-run' provide this information (untested)? > > I'm

<    3   4   5   6   7   8   9   10   11   12   >