Re: FYI - Use AC_CHECK_TOOL for lipo too.

2008-04-21 Thread Ralf Wildenhues
Hi Bob, * Bob Friesenhahn wrote on Mon, Apr 21, 2008 at 07:25:10PM CEST: Since the full patch appears on the [EMAIL PROTECTED] list, perhaps it is not necessary to manually duplicate already committed patches on this list? It's an idea. However, those commit messages - don't allow to add

Re: libtool performance status

2008-04-21 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Mon, Apr 21, 2008 at 05:14:00PM CEST: On Mon, 21 Apr 2008, Ralf Wildenhues wrote: If they were lower in between, and since increased, there must have been at least one regression along the way. Can you use git bisect to identify one? Remember that you fixed

Re: libtool performance status

2008-04-22 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Tue, Apr 22, 2008 at 05:18:34PM CEST: On Tue, 22 Apr 2008, Ralf Wildenhues wrote: Can you please check whether libtool --tag=CXX --config is identical for each of the builds you timed? It is really not as easy as you presume for me to relibtoolize

Re: Feature request: setting env vars for binary wrappers

2008-04-22 Thread Ralf Wildenhues
Hi Paolo, * Paolo Bonzini wrote on Tue, Apr 22, 2008 at 04:32:23PM CEST: In GNU Smalltalk, ./gst is used if you don't need to load any plugin, while tests/gst is used if you need plugins; tests/gst is created by config.status. Most of the time launching ./gst is enough; and since its

Re: [PATCH 366] Libtoolize now advises AC_CONFIG_MACRO_DIR use where appropriate.

2008-04-22 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Tue, Apr 22, 2008 at 08:51:39AM CEST: * libtoolize.at (func_check_macros): Always advise use of AC_CONFIG_MACRO_DIR when not able to copy libtool macros into the project tree. * tests/libtoolize.at: Update expected test output. * NEWS: Updated. ---

Re: [PATCH 366] Libtoolize now advises AC_CONFIG_MACRO_DIR use where appropriate.

2008-04-22 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Tue, Apr 22, 2008 at 07:49:25PM CEST: * Gary V. Vaughan wrote on Tue, Apr 22, 2008 at 08:51:39AM CEST: * libtoolize.at (func_check_macros): Always advise use of AC_CONFIG_MACRO_DIR when not able to copy libtool macros into the project tree. * tests/libtoolize.at

Re: AC_LIBLTDL_CONVENIENCE

2008-04-22 Thread Ralf Wildenhues
2008-04-22 Andreas Schwab [EMAIL PROTECTED] and Ralf Wildenhues [EMAIL PROTECTED] Fix default argument for AC_LIBLTDL_CONVENIENCE. * libltdl/m4/ltdl.m4 (AC_LIBLTDL_CONVENIENCE): Default argument to libltdl. (AC_LIBLTDL_INSTALLABLE): Likewise. * tests

Re: libtool 2.2.2, ccache and -all-static

2008-04-22 Thread Ralf Wildenhues
/lib/libogg.a Confirmed. Fixed as below, committed, put you in THANKS. Cheers, Ralf 2008-04-22 Ralf Wildenhues [EMAIL PROTECTED] Fix regression over 1.5.26 with ccache $CC -all-static. * libltdl/config/ltmain.m4sh (func_mode_link): Add $link_static_flag

Re: [PATCH 365] Only complain that files are up-to-date with --force.

2008-04-22 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Tue, Apr 22, 2008 at 08:49:54AM CEST: Okay to Apply? I'd say yes. I have some nits below; I haven't had the time to check all the logic changes in detail, but I suppose user feedback will weed out remaining issues. ;-) Thank you very much for tackling

Re: [PATCH 365] Only complain that files are up-to-date with --force.

2008-04-22 Thread Ralf Wildenhues
* Gary V. Vaughan wrote on Wed, Apr 23, 2008 at 12:18:39AM CEST: On 22 Apr 2008, at 16:16, Ralf Wildenhues wrote: +eval $1= Those double quotes are pretty superfluous, as they won't survive the eval. Not that they are a problem at all. :-) I quite like the fact that the say

Re: libtool 2.2.2, ccache and -all-static

2008-04-22 Thread Ralf Wildenhues
2008-04-23 Ralf Wildenhues [EMAIL PROTECTED] * tests/static.at (ccache -all-static): Ignore link warnings. Report by Gary V. Vaughan. diff --git a/tests/static.at b/tests/static.at index 1a51d0d..b5e9946 100644 --- a/tests/static.at +++ b/tests/static.at @@ -372,6 +372,6

Re: [PATCH 366] Libtoolize now advises AC_CONFIG_MACRO_DIR use where appropriate.

2008-04-23 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Wed, Apr 23, 2008 at 12:04:02AM CEST: On 22 Apr 2008, at 15:27, Ralf Wildenhues wrote: I'm wondering a bit whether we should recommend putting ACLOCAL_AMFLAGS = -I MACRO-DIR in the toplevel Makefile.am. Agreed. I'll add that before I push. It seems

Re: [libtool 2.2.2] testsuite: ... 38 ... 56 ... failed

2008-04-23 Thread Ralf Wildenhues
Hello Michael, * Michael Haubenwallner wrote on Sat, Apr 19, 2008 at 11:46:32AM CEST: On Fri, 2008-04-18 at 21:24 +0200, Ralf Wildenhues wrote: * Michael Haubenwallner wrote on Fri, Apr 18, 2008 at 04:22:03PM CEST: [lt-2.2.2-testsuite-usrlocal.patch] proposed patch to change any default

Re: Avoid using grep in func_lalib_p

2008-04-24 Thread Ralf Wildenhues
Hi Olly, * Olly Betts wrote on Thu, Apr 24, 2008 at 08:04:03AM CEST: I noticed that the sed and grep combination in func_lalib_p can be folded into a single use of sed. I don't think this is likely to be a hot spot, but it's an easy fix. I'm not an expert on sed portability but it doesn't

Re: [PATCH 366] Libtoolize now advises AC_CONFIG_MACRO_DIR use where appropriate.

2008-04-24 Thread Ralf Wildenhues
* Gary V. Vaughan wrote on Thu, Apr 24, 2008 at 05:34:39AM CEST: On 23 Apr 2008, at 17:26, Ralf Wildenhues wrote: It seems that one of your patches causes this test failure: --- expout 2008-04-23 22:41:22.0 +0200 +++ /home/ralf/libtool/write/build/tests/testsuite.dir/at-stdout

Re: [PATCH] Ensure $OBJDUMP is defined

2008-05-06 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Tue, May 06, 2008 at 02:23:05AM CEST: 2008-05-05 Charles Wilson ... Yaakov Selkowitz ... Ensure $OBJDUMP is defined * libltdl/m4/libtool.m4 (_LT_DECL_OBJDUMP): new macro ensures that $OBJDUMP is always defined sanely.

Re: [PATCH 370] Implement lt_dlopening of only preloaded modules.

2008-05-06 Thread Ralf Wildenhues
Hi Gary, a quick nit review (I haven't looked thoroughly yet): * Gary V. Vaughan wrote on Tue, May 06, 2008 at 07:24:46PM CEST: This makes it possible to build a libltdl client with some preloaded modules, and limit lt_dlopenadvise to open only the preloaded modules. --- a/NEWS +++ b/NEWS

Re: [PATCH] Ensure cwrapper compiles without warnings under -std=c99.

2008-05-06 Thread Ralf Wildenhues
Hi Charles, In addition to Eric's review, here's some extremely picky nits: * Charles Wilson wrote on Tue, May 06, 2008 at 02:23:05AM CEST: 2008-05-05 Charles Wilson ... Ensure cwrapper compiles without warnings under -std=c99. * libltdl/config/ltmain.m4sh

Re: [PATCH] [mingw|cygwin] Modify cwrapper to invoke target directly.

2008-05-06 Thread Ralf Wildenhues
Hi Charles, In addition to Eric's review: * Charles Wilson wrote on Tue, May 06, 2008 at 02:23:05AM CEST: * libltdl/config/ltmain.m4sh (func_to_native_path): new function. If $host is mingw, and $build is mingw or cygwin, convert path to mingw native format.

Re: --with-included-ltdl infers --enable-ltdl-convenience

2008-05-06 Thread Ralf Wildenhues
(via ./configure --help). I'm not sure whether that is a good idea. What do you think? Thanks, Ralf 2008-05-06 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/m4/ltdl.m4 (_LTDL_CONVENIENCE): Add AC_ARG_ENABLE for ltdl-convenience, to avoid configure warning for unknown

Re: [PATCH] Ensure $OBJDUMP is defined

2008-05-07 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Wed, May 07, 2008 at 04:07:13AM CEST: BTW, sorry for all the duplicates. I was trying to use ssmtp and/or msmtp under cygwin, to send the git format-patch results. I waited over an hour for them to show up in my inbox (I'm used to the archives being

Re: [PATCH] Cwrapper should not eat -- arguments

2008-05-25 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Sun, May 25, 2008 at 06:13:04AM CEST: * libltdl/config/ltmain.m4sh (func_emit_cwrapperexe_src) [file scope]: Defined all option strings in terms of macro LTWRAPPER_OPTION_PREFIX. Similarly defined all option string lengths in terms of macro

Re: [PATCH] Cwrapper should not eat -- arguments

2008-05-25 Thread Ralf Wildenhues
* Charles Wilson wrote on Sun, May 25, 2008 at 11:04:23PM CEST: Ralf Wildenhues wrote: As discussed here: http://lists.gnu.org/archive/html/libtool-patches/2008-05/msg00067.html ... this message specifies that the first argument not matching --lt- should end parsing, whereas your patch

[PATCH] Simplify argument parsing in cwrapper.

2008-05-25 Thread Ralf Wildenhues
* libltdl/config/ltmain.m4sh (func_emit_cwrapperexe_src): Use NULL where appropriate. Do not compute length of arguments needlessly. --- Let's make this a bit less redundant. Testsuite is still running (linux - mingw cross). OK to commit if it passes? Thanks, Ralf libltdl/config/ltmain.m4sh

Re: [PATCH] Cwrapper should not eat -- arguments

2008-05-25 Thread Ralf Wildenhues
* Charles Wilson wrote on Sun, May 25, 2008 at 11:26:17PM CEST: Ralf Wildenhues wrote: I don't understand. Programs on systems != mingw, cygwin should pass -- through as well. But inexperience with Autotest granted, I'll write a test. I just meant that we don't use the cwrapper at all

Re: LT 2.2 series regression

2008-05-27 Thread Ralf Wildenhues
Hi Jeff, * Jeff Squyres wrote on Tue, May 27, 2008 at 03:07:02PM CEST: On May 26, 2008, at 1:42 PM, Ralf Wildenhues wrote: I notice that you patched [opal/]libltdl/m4/libtool.m4 -- did you mean to patch config/libtool.m4? I tried patching both of these, but got the same result (listed

Re: arz_add_sep missing

2008-05-28 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Wed, May 28, 2008 at 09:52:30PM CEST: Ahem, wasn't that necessary for argz_add and argz_count already, which were added to Libtool in 2008-02-26? OK like this? The test would have exposed the issue on, say, Solaris. The test will skip on systems where argz.o

Re: [PATCH] Check for only regular files, not devices

2008-06-01 Thread Ralf Wildenhues
gdb, when a pseudo-terminal device /dev/pts/NN is given as one of the arguments to gdb, as in: libtool --mode=execute gdb -q -nw -i mi -tty /dev/pts/4 prog Signed-off-by: Stas Maximov [EMAIL PROTECTED] Signed-off-by: Ralf Wildenhues [EMAIL PROTECTED] diff --git a/ChangeLog

Re: [PATCH] [mingw] fix cross-compile-with-wine case

2008-06-01 Thread Ralf Wildenhues
Hi Charles, * Charles Wilson wrote on Sat, May 31, 2008 at 07:11:58PM CEST: * libltdl/config/ltmain.m4sh (func_to_host_path) [host=mingw, build!=mingw|cygwin]: check that winepath stdout is non-empty and exit code is 0, instead of checking stderr. Reported by: Roumen Petrov OK, thanks!

FYI: kill TABs from libtool.texi

2008-06-16 Thread Ralf Wildenhues
TABs inside texinfo files are mostly bad, see http://gcc.gnu.org/ml/gcc-patches/2008-06/msg00920.html. Applied to master. Cheers, Ralf 2008-06-17 Ralf Wildenhues [EMAIL PROTECTED] * doc/libtool.texi (Linking executables, Static libraries) (Modules for libltdl, Distributing

Re: Bug#485421: With icc, libtool tries -KPIC (removed option) instead of -fPIC

2008-06-18 Thread Ralf Wildenhues
[ http://bugs.debian.org/485421 ] Hello Vincent, and sorry for the slow reply, * Vincent Lefevre wrote on Mon, Jun 09, 2008 at 02:45:11PM CEST: On 2008-06-09 14:24:00 +0200, Ralf Wildenhues wrote: Sigh. They can't even keep compatibility with their own compiler. (IIRC at the point

Re: Bug#485421: With icc, libtool tries -KPIC (removed option) instead of -fPIC

2008-06-19 Thread Ralf Wildenhues
* Vincent Lefevre wrote on Wed, Jun 18, 2008 at 11:32:08PM CEST: On 2008-06-18 22:31:49 +0200, Ralf Wildenhues wrote: + icpc* ) + # Intel C++, used to be incompatible with GCC. + # ICC 10 doesn't accept -KPIC any more. _LT_TAGVAR(lt_prog_compiler_wl, $1)='-Wl

Re: Bug#485421: With icc, libtool tries -KPIC (removed option) instead of -fPIC

2008-06-22 Thread Ralf Wildenhues
* Vincent Lefevre wrote on Mon, Jun 23, 2008 at 03:08:17AM CEST: OK, finally I've just seen that on the other machine I mentioned (itanium), the old libtool was working, i.e. it selected -fPIC. If I understand the libtool code, libtool was selecting -KPIC only if the GCC test failed, and

Re: Bug#485421: With icc, libtool tries -KPIC (removed option) instead of -fPIC

2008-06-23 Thread Ralf Wildenhues
* Vincent Lefevre wrote on Mon, Jun 23, 2008 at 10:26:45AM CEST: On 2008-06-23 07:25:59 +0200, Ralf Wildenhues wrote: What I'd like to know is if mpfr works with patched Libtool, and how the Libtool testsuite fares with icc (see README for how to invoke it). I have installed the patched

Re: Bug#485421: With icc, libtool tries -KPIC (removed option) instead of -fPIC

2008-06-24 Thread Ralf Wildenhues
* Vincent Lefevre wrote on Tue, Jun 24, 2008 at 03:19:25AM CEST: On 2008-06-23 20:00:43 +0200, Ralf Wildenhues wrote: - put its $bindir early in PATH, or set LIBTOOLIZE=$bindir/libtoolize, I did: export LIBTOOLIZE=$HOME/x86_64/libtool-snapshot/bin/libtoolize and - either add a line

Re: [PATCH] cosmetic fix to ltmain.m4sh

2008-07-31 Thread Ralf Wildenhues
Hi Paolo, * Paolo Bonzini wrote on Thu, Jul 31, 2008 at 01:49:10PM CEST: This small patch makes sure that all braces are paired in ltmain.m4sh. 2008-07-31 Paolo Bonzini [EMAIL PROTECTED] * ltmain.m4sh (func_emit_cwrapperexe_src): Factor multiple closed braces into the next

Update NEWS nits.

2008-08-01 Thread Ralf Wildenhues
Hi Gary, I've pushed this to improve some of the statements. (linux w/o GNU, ts ts... ;-) Cheers, Ralf diff --git a/NEWS b/NEWS index 4e63e0f..52f8804 100644 --- a/NEWS +++ b/NEWS @@ -11,9 +11,9 @@ New in 2.2.??: 2008-08-??: git version 2.2.5a, Libtool team: * Changes in supported systems or

Re: [PATCH 370] Implement lt_dlopening of only preloaded modules.

2008-08-02 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Tue, May 06, 2008 at 07:24:46PM CEST: * libltdl/m4/ltdl.m4 (LTDL_INIT): Check for a libltdl that provides lt_dladvise_preopen when deciding if installed libltdl is 'new enough'. * libltdl/libltdl/lt__private.h (lt__advise): Add a new is_preload flag. *

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-02 Thread Ralf Wildenhues
Hi Peter, yeah, replying to a mail that's 5 months old: * Peter Rosin wrote on Tue, Mar 04, 2008 at 11:23:48AM CET: I have no problems with this patch series on either mingw, nor cygwin. Great. I've rebased your patches against current git Libtool, and put them in a git branch, named

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-04 Thread Ralf Wildenhues
* Peter Rosin wrote on Mon, Aug 04, 2008 at 03:51:29PM CEST: Ralf Wildenhues skrev: I've reformatted tests/nocase.at a bit, and sprinkled in more AT_CHECKs because the test fails for me on a GNU/Linux - MinGW cross compile (using i586-mingw32msvc-gcc): libFOO is found but -lfoo

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-05 Thread Ralf Wildenhues
.) Probably the setting of nm_file_list_spec should be moved to where $NM is set; not sure, as this is actually a Windows feature, not a $NM one. Cheers, Ralf 2008-08-05 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/m4/libtool.m4 (_LT_LINKER_SHLIBS) nm_file_list_spec: New tag variable

Re: Patch for IA64 HP-UX PIC mode compilation

2008-08-06 Thread Ralf Wildenhues
Hi Steve, * [EMAIL PROTECTED] wrote on Wed, Aug 06, 2008 at 08:01:40PM CEST: I would like to modify libtool to specify the -fPIC flag when using GCC to compile code for shared libraries on the IA64 HP-UX platform. FWIW, the corresponding discussion of this is

Re: libtool.m4 vs tpf, library_name_spec

2008-08-06 Thread Ralf Wildenhues
Hi DJ, * DJ Delorie wrote on Thu, Aug 07, 2008 at 05:36:21AM CEST: Minor typo: The TPF entry has library_names_spec spelled wrong. Thanks, pushed. Cheers, Ralf 2008-08-07 DJ Delorie [EMAIL PROTECTED] * libltdl/m4/libtool.m4 (_LT_SYS_DYNAMIC_LINKER) [tpf]

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-09 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Fri, Aug 08, 2008 at 11:42:11AM CEST: Peter Rosin skrev: 31: export.at:25 Export test Exporting variables. This patch fixes the above failure for MSVC. Cygwin/gcc and MinGW are still happy. Is there any reason for not __declspec(dllimport)ing all

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-09 Thread Ralf Wildenhues
* Peter Rosin wrote on Tue, Aug 05, 2008 at 08:38:28AM CEST: Ah, ok. That's bad. The misleading name i586-mingw32msvc-gcc caught me. Again. What in the world is msvc doing in there? Somebody (Brian Dessent?) explained it nicely, recently on some mailing list that I skim; I forgot where,

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-09 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Thu, Aug 07, 2008 at 02:10:41PM CEST: Peter Rosin skrev: To fix the test failure on the MinGW cross compile, it might be enough to disable the nocase stuff in libtool.m4 for cross compiles (take the default branch in the case $host_os statement, near the end

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-09 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Thu, Aug 07, 2008 at 10:10:29AM CEST: The previous patch was not enough, program linking happened in more than one place. Here's a new patch that fixes that and also adds the postlink_cmds variable as mentioned above. This patch actually fixes all mainfest

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-11 Thread Ralf Wildenhues
* Peter Rosin wrote on Sat, Aug 09, 2008 at 10:43:26AM CEST: Ralf Wildenhues skrev: * Peter Rosin wrote on Thu, Aug 07, 2008 at 02:10:41PM CEST: 2008-08-07 Peter Rosin [EMAIL PROTECTED] * libltdl/m4/libtool.m4 (_LT_CHECK_MAGIC_METHOD): Disable nocase handling for cross compiles

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-11 Thread Ralf Wildenhues
* Peter Rosin wrote on Sat, Aug 09, 2008 at 11:03:50AM CEST: Ralf Wildenhues skrev: Does this BTW mean that the manifest file ends up being part of the .exe and thus needs not be explicitly installed and uninstalled? That would be nice. Yes, the mt command embeds the manifest as a resource

Update HACKING for git.

2008-08-11 Thread Ralf Wildenhues
OK to apply? Thanks, Ralf 2008-08-11 Ralf Wildenhues [EMAIL PROTECTED] * HACKING: Update for git, fix some minor nits. diff --git a/HACKING b/HACKING index 778eab1..266e380 100644 --- a/HACKING +++ b/HACKING @@ -52,14 +52,15 @@ and is not part of a release distribution

Fix configure-iface for make -j.

2008-08-11 Thread Ralf Wildenhues
. Add `+' to parent make rulstdout: Cheers, Ralf 2008-08-11 Ralf Wildenhues [EMAIL PROTECTED] * tests/configure-iface.at (convenience libltdl): Ignore warning from parallel make. diff --git a/tests/configure-iface.at b/tests/configure-iface.at index 9a311dc..5e33eca 100644

Remove use of exit(2) in configure test.

2008-08-11 Thread Ralf Wildenhues
Jeff reported this off-list, applied to master as rather obvious. Cheers, and thanks! Ralf 2008-08-11 Ralf Wildenhues [EMAIL PROTECTED] Remove use of exit in configure test sources. * libltdl/m4/libtool.m4 (_LT_TRY_DLOPEN_SELF): Return from main, to avoid using

Re: [PATCH 370] Implement lt_dlopening of only preloaded modules.

2008-08-12 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Sat, Aug 02, 2008 at 12:51:36PM CEST: * Gary V. Vaughan wrote on Tue, May 06, 2008 at 07:24:46PM CEST: * libltdl/m4/ltdl.m4 (LTDL_INIT): Check for a libltdl that provides lt_dladvise_preopen when deciding if installed libltdl is 'new enough'. * libltdl/libltdl

Re: [PATCH 370] Implement lt_dlopening of only preloaded modules.

2008-08-12 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Tue, Aug 12, 2008 at 09:27:07PM CEST: On Tue, 12 Aug 2008, Ralf Wildenhues wrote: OK to revert? Or, rather: does anybody have problems with the reversal? Temporarily or permanently? Ah, chucks. Incomplete research. I take the above back, for now, and brood over

More parallel make testsuite fixes.

2008-08-12 Thread Ralf Wildenhues
)/libltdl/stamp-mk rule in the toplevel Makefile of Libtool. Cheers, Ralf 2008-08-12 Ralf Wildenhues [EMAIL PROTECTED] Parallel make testsuite fixes. * tests/configure-iface.at (installable libltdl) (--with-ltdl-include/lib, --with-included-ltdl): Add dlopened

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-12 Thread Ralf Wildenhues
in the cmdline_wrap.at test (73). I pushed the other patches[1][2], but this one is also on the list of things I like. Ok to push to the msvc branch? Sorry for the delay. I pushed that patch to the msvc branch. Cheers, Ralf 2008-08-13 Ralf Wildenhues [EMAIL PROTECTED] Peter Rosin

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-12 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Wed, Aug 13, 2008 at 07:21:07AM CEST: Sorry for the delay. I pushed that patch to the msvc branch. 2008-08-13 Ralf Wildenhues [EMAIL PROTECTED] Peter Rosin [EMAIL PROTECTED] Support for response files with $NM. * libltdl/m4/libtool.m4

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-12 Thread Ralf Wildenhues
* Peter Rosin wrote on Wed, Aug 06, 2008 at 09:47:28AM CEST: Ralf Wildenhues skrev: * Peter Rosin wrote on Tue, Aug 05, 2008 at 10:38:14AM CEST: Peter Rosin skrev: 16: duplicate_conv.at:25 duplicate convenience archive names MS link doesn't have reloadable objects (i.e. like ld -r

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-13 Thread Ralf Wildenhues
* Peter Rosin wrote on Wed, Aug 13, 2008 at 11:00:33AM CEST: Ralf Wildenhues skrev: - (eval $AR -NOLOGO -OUT:conftest.lib conftest.$ac_objext conftest.err) + (eval $AR -NOLOGO -OUT:conftest.lib conftest.$ac_objext conftest.err 21) Is there a reason for this, Yes, file 21 is completely

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-14 Thread Ralf Wildenhues
* Peter Rosin wrote on Wed, Aug 13, 2008 at 10:12:23PM CEST: 24: link-order.at:26 Link order test. Exporting int c variable. With MSVC, you can declare any variable with __decspec(dllimport), even if you are not actually importing it. The only thing that happens is that you get an extra

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-14 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Wed, Aug 13, 2008 at 09:40:17PM CEST: 72: stresstest.at:31 Link option thorough search test Automatic path conversion in MSYS doesn't kick in for the argument -OUT:/some/absolute/path so lib.exe barfs. Commenting out absolute paths from the stress test

lt_dladvise.at failure

2008-08-16 Thread Ralf Wildenhues
the test XFAIL. However, the test exercises many useful bits, more than just the part that is failing, and most other bits are more important. This is why I think we should not go that way. OK to apply? Thanks, Ralf 2008-08-16 Ralf Wildenhues [EMAIL PROTECTED] Avoid test failure due

Re: [PATCH 370 bis] Implement lt_dlopening of only preloaded modules.

2008-08-16 Thread Ralf Wildenhues
Hi Gary, late second round of review: * Gary V. Vaughan wrote on Wed, May 07, 2008 at 10:32:13PM CEST: * libltdl/m4/ltdl.m4 (LTDL_INIT): Check for a libltdl that provides lt_dladvise_preopen when deciding if installed libltdl is 'new enough'. * libltdl/libltdl/lt__private.h (lt__advise): Add

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-16 Thread Ralf Wildenhues
* Peter Rosin wrote on Fri, Aug 15, 2008 at 01:44:30PM CEST: Ralf Wildenhues skrev: * Peter Rosin wrote on Wed, Aug 13, 2008 at 09:40:17PM CEST: LIBA_SCOPE int (*const v12) (void); Why doesn't this one need LIBA_SCOPE_VAR annotation only? (I guess I'm to search for the answer to this one

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-16 Thread Ralf Wildenhues
* Peter Rosin wrote on Fri, Aug 15, 2008 at 11:36:14AM CEST: Ralf Wildenhues skrev: Please add Set to @code{false} on systems that cannot create reloadable objects to the reload_cmds documentation in libtool.texi. I used @samp(false) instead, as that seemed to be the usage for constants

Re: [patch #6448] [MSVC 7/7] Add MSVC Support

2008-08-20 Thread Ralf Wildenhues
Hello Markus, * Markus Duft wrote on Wed, Aug 20, 2008 at 09:26:10AM CEST: Could this break parity support? We try to ensure that it doesn't happen. I know It's not in the tree yet, but I still hope, that ralf comes to looking into my patch some day Yes. My plan is to get Peter's

Re: Linking with '-L /path/to/lib' (note the space!)

2008-08-21 Thread Ralf Wildenhues
-08-21 Ralf Wildenhues [EMAIL PROTECTED] Disallow `-L path', do not misparse it silently. * libltdl/config/ltmain.m4sh (func_mode_link): Diagnose `-L' without argument, and `-L path', i.e., with a space. * tests/fail.at (Failure tests): Test for these failures

Re: Linking with '-L /path/to/lib' (note the space!)

2008-08-21 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Thu, Aug 21, 2008 at 08:29:19PM CEST: On Thu, 21 Aug 2008, Ralf Wildenhues wrote: * Olaf Lenz wrote on Thu, Aug 21, 2008 at 02:56:10PM CEST: When I pass the option -L /path/to/lib (note the space between -L and /path/to/lib) to libtool in linking mode, libtool does

Re: Linking with '-L /path/to/lib' (note the space!)

2008-08-21 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Thu, Aug 21, 2008 at 09:09:50PM CEST: On Thu, 21 Aug 2008, Ralf Wildenhues wrote: Well, libtool *could* also remove the space. I'm just not sure whether it's worth supporting this. Removing the space might simply cause a failure later if it is an actual typo

Re: Linking with '-L /path/to/lib' (note the space!)

2008-08-21 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Thu, Aug 21, 2008 at 09:11:16PM CEST: * Bob Friesenhahn wrote on Thu, Aug 21, 2008 at 09:09:50PM CEST: On Thu, 21 Aug 2008, Ralf Wildenhues wrote: Well, libtool *could* also remove the space. I'm just not sure whether it's worth supporting this. Removing

Support for Lahey Fortran 8.1

2008-08-21 Thread Ralf Wildenhues
little chance of regression here, but would of course still welcome any review. Thanks, Ralf 2008-08-22 Ralf Wildenhues [EMAIL PROTECTED] Initial support for Lahey Fortran on GNU/Linux. * libltdl/m4/libtool.m4 (_LT_COMPILER_PIC, _LT_LINKER_SHLIBS) [linux] lf95: Add support

Re: Two small libltdl patches.

2008-08-26 Thread Ralf Wildenhues
--- /dev/null +++ b/tests/ltdl-api.at @@ -0,0 +1,36 @@ +# ltdl-api.at -- test libltdl API-*- Autotest -*- +# +# Copyright (C) 2008 Free Software Foundation, Inc. +# Written by Ralf Wildenhues, 2008. +# +# This file is part of GNU Libtool. +# +# GNU Libtool is free software; you

Re: Two small libltdl patches.

2008-08-26 Thread Ralf Wildenhues
Hi Peter, * Peter O'Gorman wrote on Sat, Aug 23, 2008 at 11:59:00PM CEST: Subject: [PATCH] Allow for extensions other than .a for preloaded modules. * libltdl/m4/ltdl.m4 (_LTDL_SETUP): Define LT_LIBEXT. * libltdl/ltdl.c (lt_dladvise_preload): Use it. Reported by Ralf Wildenhues. OK, thanks

Re: Two small libltdl patches.

2008-08-26 Thread Ralf Wildenhues
* Peter O'Gorman wrote on Tue, Aug 26, 2008 at 06:24:11PM CEST: Ralf Wildenhues wrote: I'd like this to fail for me on darwin (more likely to notice it), so I will probably add: +AT_CHECK([eval $NM \\$argz_o\ | $global_symbol_pipe], +[], [stdout], [ignore]) +AT_CHECK([grep ^T

Re: [PATCH] Increase Safety factor when ARG_MAX is less than 40KB.

2008-09-02 Thread Ralf Wildenhues
* Peter O'Gorman wrote on Tue, Sep 02, 2008 at 08:00:31PM CEST: It hit the error with ARG_MAX at 20K, max_cmd_len at 15K, and confusingly, it hit it doing the ld -r (so no driver overhead). The same ld -r command worked fine outside of libtool, but failed consistently when libtool attempted

Re: [PATCH] Increase Safety factor when ARG_MAX is less than 40KB.

2008-09-04 Thread Ralf Wildenhues
Hi Peter, * Peter O'Gorman wrote on Thu, Sep 04, 2008 at 11:53:11PM CEST: Ralf Wildenhues wrote: barring anything better, or better analysis, this is fine with me. When I looked at the size of the environment, it was less than 1.5K, so it only left me more confused. I am not convinced

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

2008-09-06 Thread Ralf Wildenhues
[ moving from libtool-commit ] Hi Gary, a couple of nits. Note that I currently don't feel the need for a commit script, as we have commit messages enabled through git anyway. But if it suits you... * Gary V. Vaughan wrote on Sat, Sep 06, 2008 at 08:41:55PM CEST: - Log

Re: [SCM] GNU Libtool branch, master, updated. v2.2.4-45-gc2b8636

2008-09-06 Thread Ralf Wildenhues
[ moving from libtool-patches ] Hi Gary, * Gary V. Vaughan wrote on Sat, Sep 06, 2008 at 11:12:33PM CEST: Prepare for next stable release. * libltdl/Makefile.inc (LTDL_VERSION_INFO): Update. * libltdl/m4/libtool.m4: Bump serial number to account for changes since 2.2.4

Re: [SCM] GNU Libtool branch, master, updated. v2.2.4-45-gc2b8636

2008-09-07 Thread Ralf Wildenhues
* Gary V. Vaughan wrote on Sun, Sep 07, 2008 at 09:06:59AM CEST: On 7 Sep 2008, at 06:41, Ralf Wildenhues wrote: * Gary V. Vaughan wrote on Sat, Sep 06, 2008 at 11:12:33PM CEST: Prepare for next stable release. * libltdl/Makefile.inc (LTDL_VERSION_INFO): Update. * libltdl/m4

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 09:15:10AM CEST: On 7 Sep 2008, at 06:34, Ralf Wildenhues wrote: a couple of nits. Note that I currently don't feel the need for a commit script, as we have commit messages enabled through git anyway. But if it suits you... Peter asked

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 sure it does, but I don't know what it looks like

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 other utilities. (We

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, and no information for the various --mode

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 information for the various --mode options

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

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: error:

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/loaders/loadlibrary.c:159: error: the address of 'wpath

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

2008-10-01 Thread Ralf Wildenhues
new file mode 100644 index 000..dd92100 --- /dev/null +++ b/tests/install.at @@ -0,0 +1,49 @@ +# install.at -- install mode tests -*- Autotest -*- +# +# Copyright (C) 2008 Free Software Foundation, Inc. +# Written by Ralf Wildenhues, 2008 +# +# This file is part of GNU Libtool

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 fails to mmap shared libraries

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 removed

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: problem when cross compiling with mingw32ce

2008-10-05 Thread Ralf Wildenhues
/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 below. Now we need someone who knows about wince to tell us whether this is the right thing to do. Thanks, Ralf 2008-10-05 Ralf Wildenhues [EMAIL PROTECTED

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 the

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: grep

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 patch

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/focus=28033 For the reasons listed

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 I

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 the

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 things:

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]) +# -- +#

<    6   7   8   9   10   11   12   13   14   15   >