_GLOBAL__FD_libfoo_so

2008-01-10 Thread Ralf Wildenhues
/html/bug-libtool/2008-01/msg9.html (the respective failure is seen in the attachment) 2008-01-10 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/m4/libtool.m4 (_LT_LINKER_SHLIBS) exclude_expsyms: Also exclude `_GLOBAL__F[ID]_.*'. Also set for the C++ tag. Fixes esp

cope with dlopen(NULL) failing on AIX (was: support AIX 6.1 in config.rpath)

2008-01-12 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Thu, Jan 10, 2008 at 10:20:30PM CET: * Rainer Tammer wrote on Tue, Jan 08, 2008 at 05:03:30PM CET: did not find the `myfunc' function error was: Function not implemented (myfunc) did not find the `myvar' variable error was: Function not implemented

Re: FYI: 333-gary-refactor-LTDL_INIT.patch

2008-01-14 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Tue, Jan 08, 2008 at 08:29:05PM CET: * Gary V. Vaughan wrote on Tue, Jan 08, 2008 at 05:23:24AM CET: On 8 Jan 2008, at 04:40, Ralf Wildenhues wrote: With this patch, I see in the log of make distcheck that the libltdl subdirectory is being configured

Re: make -s

2008-01-14 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Thu, Jan 10, 2008 at 08:29:33AM CET: For whatever output is left done by libtool I expect that whoever want's it silenced hard enough will have enough motivation to send a patch to libtool-patches@gnu.org. Since repeatedly nobody stepped forward to do this, I wrote

Re: FYI: 333-gary-refactor-LTDL_INIT.patch

2008-01-14 Thread Ralf Wildenhues
Hello Gary, I've applied this patch to fix an omission in your change. Cheers, Ralf 2008-01-14 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/m4/ltoptions.m4 (AC_ENABLE_SHARED): Fix to use new _LT_SET_OPTION semantics. Index: libltdl/m4/ltoptions.m4

FYI: update `fetch' target

2008-01-14 Thread Ralf Wildenhues
Hello, I've fixed the `fetch' target as below, in HEAD, and likewise in branch-1-5. Cheers, Ralf * Makefile.maint (fetch): Update to match upstream git sources. Pull `INSTALL' from gnulib instead of from Autoconf. Index: Makefile.maint

Re: FYI: 333-gary-refactor-LTDL_INIT.patch

2008-01-14 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Tue, Jan 15, 2008 at 03:22:07AM CET: On 15 Jan 2008, at 03:24, Ralf Wildenhues wrote: OK to apply this fix (feeling a bit hackish, but does exactly what's needed)? Absolutely. Sorry I didn't get to it sooner myself. I can't think of a cleaner way

Re: withGCC unset

2008-01-15 Thread Ralf Wildenhues
Hi Peter, * Peter O'Gorman wrote on Tue, Jan 15, 2008 at 04:01:20PM CET: Ralf Wildenhues wrote: Intended before 2.2? Does not have to be (similarly, this one could be considered cosmetic, and wait 'til after 2.2 - I'll leave that up to you). If you ask me, you can apply this one now

Do not use AM_CONDITIONAL if it's not defined

2008-01-16 Thread Ralf Wildenhues
I've applied this to not depend upon aclocal. If it's used, it will pull in AM_CONDITIONAL, so no difference in that case. Cheers, Ralf 2008-01-16 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/m4/ltdl.m4 (_LT_ENABLE_INSTALL): Use AM_CONDITIONAL only if defined. Allows to use

Re: FYI: 333-gary-refactor-LTDL_INIT.patch

2008-01-16 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Sat, Jan 12, 2008 at 03:15:03PM CET: So I ask to please clarify once and for all the order in which things should be (including a notice in NEWS), the rationale for it, and please to make it clear in the manual whether LTDL_INIT requires that LT_INIT also be called

Re: FYI: 333-gary-refactor-LTDL_INIT.patch

2008-01-16 Thread Ralf Wildenhues
Hi Peter, * Peter Rosin wrote on Wed, Jan 16, 2008 at 08:13:59PM CET: On Wed, Jan 16, 2008 at 08:07:41PM +0100, Ralf Wildenhues wrote: [EMAIL PROTECTED] macro (after the call to @samp{LTDL_INIT} Is the second instance a typo? Should be LT_INIT, right? Yes. However, as usual, I don't

Re: FYI: 333-gary-refactor-LTDL_INIT.patch

2008-01-16 Thread Ralf Wildenhues
that this is still a bit fragile when Libtool evolves (really only m4 can parse m4), which is why I'm adding a testcase, too. Cheers, Ralf 2008-01-16 Ralf Wildenhues [EMAIL PROTECTED] * libtoolize.m4sh (func_scan_files): Avoid matching our own macro code when scanning configure.ac

Re: FYI: 333-gary-refactor-LTDL_INIT.patch

2008-01-16 Thread Ralf Wildenhues
Hi Bob, * Bob Friesenhahn wrote on Thu, Jan 17, 2008 at 01:17:58AM CET: On Wed, 16 Jan 2008, Ralf Wildenhues wrote: However, as usual, I don't feel qualified to review other aspects of this patch... Well, thanks anyway for spotting this typo! With this change, I am ok

Re: FYI: 333-gary-refactor-LTDL_INIT.patch

2008-01-16 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Thu, Jan 17, 2008 at 05:29:17AM CET: On 16 Jan 2008, at 14:22, Ralf Wildenhues wrote: OK to apply? I have some nits first. Please commit once they are addressed. Addressed and committed as noted below. Thanks for your quick review. * NEWS: Update

Re: FYI: 333-gary-refactor-LTDL_INIT.patch

2008-01-16 Thread Ralf Wildenhues
* Gary V. Vaughan wrote on Thu, Jan 17, 2008 at 06:20:31AM CET: On 17 Jan 2008, at 03:37, Ralf Wildenhues wrote: OK to apply this patch to fix it and some other cases? Yes please. Done as below, thanks. Our libtoolize test coverage is still extremely poor. When you commit could you also

Re: 334-gary-remove-libtool-dvi.patch

2008-01-18 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Fri, Jan 18, 2008 at 10:43:30AM CET: [[FYI: bug-automake]] I consider this a texinfo bug, sorry for not pointing out earlier: http://thread.gmane.org/gmane.comp.tex.texinfo.bugs/3930 Please apply, but don't blame Automake in the patch. Thanks, Ralf from

Re: multilib dirs and ld.so

2008-01-21 Thread Ralf Wildenhues
Hi Peter, * Peter O'Gorman wrote on Mon, Jan 21, 2008 at 07:29:08PM CET: I give up on trying to find fancy ways to set the paths, ok to apply this to branch-1-5 (and similar for HEAD)? I suppose, yes, but I guess some mention in the documentation would not be bad. Thank you, Ralf

Re: FYI backport notes.texi from HEAD

2008-01-23 Thread Ralf Wildenhues
Hi Peter, * Peter O'Gorman wrote on Thu, Jan 24, 2008 at 08:29:08AM CET: I applied this to branch-1-5. Guess that makes the section in README obsolete. Wanna kill it or replace it with a pointer to notes.texi? 2008-01-24 Peter O'Gorman [EMAIL PROTECTED] * doc/libtool/texi:

Re: FYI backport notes.texi from HEAD

2008-01-24 Thread Ralf Wildenhues
* Peter O'Gorman wrote on Thu, Jan 24, 2008 at 05:49:17PM CET: Tim Rice wrote: On Thu, 24 Jan 2008, Ralf Wildenhues wrote: Guess that makes the section in README obsolete. Wanna kill it or replace it with a pointer to notes.texi? Keep in mind not all systems have texinfo installed

Re: Autoreconf calling order order [WAS FYI: 333-gary-refactor-LTDL_INIT.patch]

2008-01-24 Thread Ralf Wildenhues
* Gary V. Vaughan wrote on Thu, Jan 24, 2008 at 06:19:37PM CET: I haven't picked through the logic inside autoreconf, but it seems completely wrong to me for it to run any tool that affects the contents of aclocal.m4 after aclocal itself (unless, perhaps, it reruns aclocal again later).

Re: Autoreconf calling order order

2008-01-24 Thread Ralf Wildenhues
* Eric Blake wrote on Thu, Jan 24, 2008 at 07:57:20PM CET: According to Ralf Wildenhues on 1/24/2008 11:00 AM: | * Gary V. Vaughan wrote on Thu, Jan 24, 2008 at 06:19:37PM CET: | I haven't picked through the logic inside autoreconf, but it seems | completely wrong to me for it to run any tool

Re: testsuite -C dir

2008-01-29 Thread Ralf Wildenhues
Libtool mainline uses Autotest and a section m4_divert_push([PREPARE_TESTS]) ... m4_divert_pop([PREPARE_TESTS]) to set some values, e.g., eval `$LIBTOOL --config | grep '^EGREP='` This fails with Autoconf mainline now (after the recent patches) because $LIBTOOL is not set yet; it is set

Re: multilib patch

2008-01-30 Thread Ralf Wildenhues
Hi Peter, * Peter O'Gorman wrote on Wed, Jan 30, 2008 at 09:16:38AM CET: Not much response last time, here we go again. Sorry about the delay. Ok to apply to branch-1-5 and forward port to HEAD? Nits below. FWIW, on Debian x86_64, this patch causes for me /usr/lib to appear in the front,

Re: multilib patch

2008-01-30 Thread Ralf Wildenhues
* Peter O'Gorman wrote on Thu, Jan 31, 2008 at 08:00:59AM CET: I do not know if it is best to leave in the symlink and the original dir ( i.e. do cd $dir pwd as well as cd $dir pwd -P), which on debian would give something like: /usr/lib/gcc/x86_64-linux/gnu/4.1.2 /usr/lib64 /usr/lib /lib64

Re: [libtool 2.1b] testsuite: 16 54 55 56 failed

2008-02-01 Thread Ralf Wildenhues
which it had ever since AC_PROG_CXX was added 1994 in commit 8202df5faf642cde4e5b1fa02cd9cc4411a7a477 I'll propose to remove that in Autoconf. Cheers, Ralf 2008-02-01 Ralf Wildenhues [EMAIL PROTECTED] * tests/testsuite.at (LT_AT_TAG): For the CXX tag, if $CXX is g++, check

Re: multilib patch

2008-02-01 Thread Ralf Wildenhues
Hi Peter, * Peter O'Gorman wrote on Thu, Jan 31, 2008 at 08:00:59AM CET: I installed amd64 debian in a VM and with this patch I get: /usr/lib/gcc/x86_64-linux/gnu/4.1.2 /usr/lib /lib I do not know if it is best to leave in the symlink and the original dir ( i.e. do cd $dir pwd as well as

Re: CVS HEAD test 30

2008-02-07 Thread Ralf Wildenhues
this patch. Cheers, Ralf 2008-02-08 Ralf Wildenhues [EMAIL PROTECTED] * tests/archive-in-archive.at (static library contains static library): Test also the actual broken link command, and the following install command, as UnixWare ar rightly refuses to put an archive

Fix `-Wl,' for Intel Fortran compiler ifort

2008-02-08 Thread Ralf Wildenhues
!) Cheers, Ralf 2008-02-09 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/m4/libtool.m4 (_LT_COMPILER_PIC) [linux] ifort: Treat like icc and ecc, for fixed `-Wl,' and `-KPIC' settings. Index: libltdl/m4/libtool.m4 === RCS

Re: [patch #6416] AmigaOS4 support in libtool

2008-02-10 Thread Ralf Wildenhues
Hello Henning, * Henning Nielsen Lund wrote on Sun, Feb 10, 2008 at 07:16:17PM CET: Summary: AmigaOS4 support in libtool Project: GNU Libtool Since July 2007 AmigaOS4 officially supports .so libraries. It would be nice if it would be possible to add

Fix mdemo-dryrun failure with OpenBSD make

2008-02-11 Thread Ralf Wildenhues
, huh? Fixed as shown below. No, I do not care to list all of the source files of libltdl there: toplevel check-local already depends on libltdl/libltdlc.la, so it should anyway be up to date before running the mdemo tests. Cheers, Ralf 2008-02-11 Ralf Wildenhues [EMAIL PROTECTED

Re: libtoolize /ltmain.sh bug

2008-02-13 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Mon, Feb 11, 2008 at 11:01:53PM CET: In an empty directory this happens: $ libtoolize --copy --ltdl touch: cannot touch `/ltmain.sh': Permission denied libtoolize: can not copy `/home/ralf/local/share/libtool/config/ltmain.sh' to `/' libtoolize: copying file

libltdl/configure.ac has a version, too

2008-02-13 Thread Ralf Wildenhues
Applied to HEAD and branch-1-5, respectively. Ahem, version 1.2. Cheers, Ralf 2008-02-13 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/configure.ac: Bump package version. * HACKING: Reminder to do it next time, too. Index: HACKING

Re: [patch/rfc] setting version type based on vendor

2008-02-18 Thread Ralf Wildenhues
Hello Mike, Please post patches to the libtool-patches list in the future, thanks. * Mike Frysinger wrote on Sat, Feb 16, 2008 at 09:48:52AM CET: i dont know if there's a policy on this, but we've been treating the shared libraries in the Gentoo/FreeBSD port like Linux shared libraries. we

Re: patch adding argz_add and argz_count implementation

2008-02-26 Thread Ralf Wildenhues
Hi Karl, * Karl Berry wrote on Tue, Feb 26, 2008 at 06:11:34PM CET: Good, except that I'd prefer if argz_count used strlen instead of Ok, since you prefer it, I copied the strlen loop from libc. Below is a revised patch (for both files for convenience). Thanks. Applied as shown below.

Re: FYI: 353-gary-argz-ac-check-funcs.patch

2008-02-29 Thread Ralf Wildenhues
Hi Gary, On Fri, Feb 29, 2008 at 04:36:17PM -0500, Gary V. Vaughan wrote: Applied to HEAD. Index: ChangeLog from Gary V. Vaughan [EMAIL PROTECTED] * libltdl/m4/argz.m4 (AC_CHECK_FUNCS): Also use our replacement argz if the system argz_add or argz_count are missing. Report

Re: Libtool head test status

2008-03-02 Thread Ralf Wildenhues
Hi Bob, * Bob Friesenhahn wrote on Sat, Mar 01, 2008 at 08:07:01AM CET: These are results of tests I ran this evening. These are all builds from outside the source tree, including MinGW and Cygwin, which used a SMB mount to a Unix filesystem. Thanks for your testing efforts. Can you

Re: libltdl memory corruption

2008-03-03 Thread Ralf Wildenhues
I'd appreciate a review of this, and also test results on systems with loaders other than preopen and dlopen. (I haven't even tested successful compilation on those other systems.) Thanks, Ralf 2008-03-03 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/loaders/dld_link.c (vl_exit): New

git? branch-2-2?

2008-03-04 Thread Ralf Wildenhues
Hello libtoolers, 1) Can we move to use git now? 2) Can we create a branch-2-2 and cherry-pick bugfixes from HEAD into it, aiming for a soonish 2.2.2 (let's say, in a few weeks; already a few important issues are known)? 3) If yes and yes, do you agree with the proposed git policy: master

Re: libltdl memory corruption

2008-03-04 Thread Ralf Wildenhues
Hi Peter, * Peter O'Gorman wrote on Tue, Mar 04, 2008 at 07:14:51AM CET: Ralf Wildenhues wrote: So I'd appreciate a review of this, and also test results on systems with loaders other than preopen and dlopen. (I haven't even tested successful compilation on those other systems.) I

FYI: fix doc typo

2008-03-04 Thread Ralf Wildenhues
I applied this tiny fix. Cheers, Ralf 2008-03-04 Ralf Wildenhues [EMAIL PROTECTED] * doc/libtool.texi (Module loaders for libltdl): Fix typo. Index: doc/libtool.texi === RCS file: /cvsroot/libtool/libtool/doc

libtool generated by GNU $PACKAGE (was: [libtool 2.2] testsuite: 34 failed)

2008-03-04 Thread Ralf Wildenhues
a reporter. The hard part with this patch was ensuring that none of the libtool code uses this bit in a sed pattern (in some parts script headers are checked, but not this one, apparently). Cheers, and thanks to both of you for the report (I put you in THANKS), Ralf 2008-03-04 Ralf Wildenhues

mode=execute argument munging bug (was: [libtool 2.2] testsuite: 34 failed)

2008-03-04 Thread Ralf Wildenhues
-*- +# +# 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 can redistribute it and/or +# modify it under the terms of the GNU General Public License as +# published by the Free

Re: git? branch-2-2?

2008-03-04 Thread Ralf Wildenhues
not. On Tue, 4 Mar 2008, Ralf Wildenhues wrote: 3) If yes and yes, do you agree with the proposed git policy: master should usually not contain merges except for merges from public topic branches, should we have such in the future. Generally, master should be the first to receive a patch

libltdl skips dlopen loader if there are others

2008-03-04 Thread Ralf Wildenhues
Hello, this fixes one of the most long-standing regressions of 2.2 over branch-1-5. It causes issues on every system that has not only the dlopen loader, but also another one. I bet this is the cause for some long-standing issues on w32. On HP-UX, the lt_dladvise test fails with |

Re: git? branch-2-2?

2008-03-04 Thread Ralf Wildenhues
Hi Gary, * Gary V. Vaughan wrote on Wed, Mar 05, 2008 at 12:10:30AM CET: On 4 Mar 2008, at 15:56, Ralf Wildenhues wrote: 2) Can we create a branch-2-2 and cherry-pick bugfixes from HEAD into it, aiming for a soonish 2.2.2 (let's say, in a few weeks; already a few important issues are known

Re: mode=execute argument munging bug

2008-03-05 Thread Ralf Wildenhues
* Roberto Bagnara wrote on Wed, Mar 05, 2008 at 07:37:58AM CET: It is better now, but there is still the problem that, apparently, libtool redirects stdin for the program it is running. Gosh. How embarrassing. I've applied this patch. Thanks for testing! Ralf 2008-03-05 Ralf Wildenhues

Re: libtool generated by GNU $PACKAGE

2008-03-05 Thread Ralf Wildenhues
Hi Eric, * Eric Blake wrote on Wed, Mar 05, 2008 at 04:28:51AM CET: According to Ralf Wildenhues on 3/4/2008 2:14 PM: | Fixed with the patch below. I don't care much that, in the Libtool | package itself, the will result in a libtool script with the line | # Generated automatically

Re: Unhelpful error message in libltdl

2008-03-06 Thread Ralf Wildenhues
] (tiny change) Ralf Wildenhues [EMAIL PROTECTED] * libltdl/ltdl.c (find_module): If we found FILENAME, but open failed for a different reason, stop searching and provide a helpful error message. * tests/lt_dlopen.at (lt_dlopen error messages): New test

Re: [libtool 2.2] testsuite: 19 35 36 44 45 46 48 49 50 51 52 53 60 61 62 64 failed [GNU/Linux PowerPC]

2008-03-06 Thread Ralf Wildenhues
* Gary V. Vaughan wrote on Thu, Mar 06, 2008 at 10:41:47PM CET: On 6 Mar 2008, at 15:03, Bob Friesenhahn wrote: There needs to be a way to output any warnings at the tail end of configure so that at least someone is more likely to see them. Without adequate notification to the user, the

Re: [libtool 2.2] testsuite: 19 64 failed [GNU/Linux IA-32]

2008-03-06 Thread Ralf Wildenhues
* Peter O'Gorman wrote on Fri, Mar 07, 2008 at 08:40:08AM CET: Peter O'Gorman wrote: Ralf has already checked in a workaround for gcj being unable to create objects/executables. I guess I will add to that so it tests that an executable created by the compiler will actually run. Ok?

fix installcheck for --program-transform-name

2008-03-08 Thread Ralf Wildenhues
OK to apply? This fixes ./configure --program-prefix=g make all install installcheck and also causes the testsuite to be correctly rebuilt upon installcheck. Thanks, Ralf 2008-03-08 Ralf Wildenhues [EMAIL PROTECTED] Fix installcheck dependencies, fix for --program-transform-name

w32 ports of Libtool

2008-03-09 Thread Ralf Wildenhues
Hello Peter, Markus, in order to give some perspective for both of your w32 ports of Libtool: when we make the switch to git as primary repo, we intend to import your patch series in topic branches to allow for easier work and integration of them into the master tree (and to hopefully synchronize

Re: [patch #6416] AmigaOS4 support in libtool

2008-03-12 Thread Ralf Wildenhues
Hello Henning, I have applied your patch now, including a NEWS entry, as below, and put you in THANKS. Please check that I did not make any errors, thanks. Gathering from the testsuite failure, the shared library support has some work ahead yet. Cheers, Ralf 2008-03-12 Henning Nielsen Lund

[patch #6416] AmigaOS4 support in libtool

2008-03-12 Thread Ralf Wildenhues
Update of patch #6416 (project libtool): Open/Closed:Open = Closed ___ Follow-up Comment #1: Applied to CVS HEAD. ___ Reply to

[patch #6104] reduce space complexity for piecewise linking

2008-03-12 Thread Ralf Wildenhues
Update of patch #6104 (project libtool): Open/Closed:Open = Closed ___ Follow-up Comment #1: This has been dealt with on 2007-2007-07-23.

[patch #40] Use ln -sf when installing library links

2008-03-12 Thread Ralf Wildenhues
Update of patch #40 (project libtool): Open/Closed:Open = Closed ___ Follow-up Comment #3: This has been fixed already on 2005-08-24, by an independent bug report.

Re: cross compilation to w32

2008-03-12 Thread Ralf Wildenhues
Hello Charles, * Charles Wilson wrote on Mon, Mar 10, 2008 at 03:23:04AM CET: As far as I can tell, none of these failures are related to the issues raised in this thread or addressed by this patch. I agree. However, I have no way of knowing *for sure* if this patch fixes the

cross compile testing fixups

2008-03-15 Thread Ralf Wildenhues
, which also happens to speed up the old testsuite a bit by running 'libtool --config' less often. Cheers, Ralf 2008-03-15 Ralf Wildenhues [EMAIL PROTECTED] * tests/defs.m4sh (func_get_config): Build egrep regex to speed up extracting several variables at once. Use it to extract

Re: [patch] quote language related variables in case they have garbage

2008-03-16 Thread Ralf Wildenhues
Hello Mike, * Mike Frysinger wrote on Sun, Mar 16, 2008 at 08:36:20AM CET: if you set a language variable with quotes or semicolons or other weird things, you may get ugly warnings/errors when indirectly evaluating them. Please provide an example, including all output, that exposes this. I

Re: weird tags in libtool's experimental git repo

2008-03-16 Thread Ralf Wildenhues
* Jim Meyering wrote on Sun, Mar 16, 2008 at 12:03:12PM CET: How to clean-up: - remote bogus tags and prune - simply regenerate libtool's git repo from CVS What would you like? I tend to prefer the latter, but people will have to clone again. While we're at cleaning the repo, let's

fix 'autoconf -Wall' warning

2008-03-17 Thread Ralf Wildenhues
Applied to HEAD as obvious, autoconf warns about it with -Wall. The AC_MSG_* have been escaping since Autoconf 2.53 or so, and we require 2.58 anyway. Cheers, Ralf 2008-03-17 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/m4/libtool.m4 (_LT_CHECK_BUILDDIR): No need to escape

Fix disable-static -all-static (was: testsuite: 24 25 64 failed)

2008-03-24 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Mon, Mar 24, 2008 at 12:56:56PM CET: I'll address 25 separately (confirmed with --disable-static, BTW). Proposed, slightly ugly patch. Can you try it please, and do you have a name and email address to put in the ChangeLog and THANKS? Thus, OK to apply? Note

Re: Fix disable-static -all-static

2008-03-24 Thread Ralf Wildenhues
On Mon, Mar 24, 2008 at 01:42:19PM +, Nix wrote: * Ralf Wildenhues wrote on Mon, Mar 24, 2008 at 12:56:56PM CET: I'll address 25 separately (confirmed with --disable-static, BTW). Proposed, slightly ugly patch. It's only ugly if you consider it nasty to encode knowledge

Re: libtool 1.5.26: link fails on Darwin 5.5 because of wrong detection of -single_module flag support

2008-03-27 Thread Ralf Wildenhues
Hi Peter, * Peter O'Gorman wrote on Thu, Mar 27, 2008 at 03:56:17AM CET: Ok? Yes, with nits below addressed. Thanks, Ralf 2008-03-26 Peter O'Gorman [EMAIL PROTECTED] * libltdl/m4/libtool.m4 [_LT_REQUIRED_DARWIN_CHECKS]: Fix the test for -single_module and log it to

compile mode, XSI shell: do not fork for $xform

2008-04-09 Thread Ralf Wildenhues
OK to apply? Thanks, Ralf 2008-04-10 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/m4/libtool.m4 (_LT_PROG_XSI_SHELLFNS) func_xform: New function. * libltdl/config/ltmain.m4sh (func_mode_compile): Use it to avoid two forks with XSI shells. Index: libltdl/config

Speed up progname, progpath setting

2008-04-09 Thread Ralf Wildenhues
OK to apply? What about eliminating (when progdir='.') curwd=`cd . pwd` Do we know when (with which shells) this can be replaced by $PWD portably? Thanks, Ralf 2008-04-10 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/m4/libtool.m4 (_LT_PROG_XSI_SHELLFNS): Move non-XSI

No need for object file removal before compilation

2008-04-09 Thread Ralf Wildenhues
OK to apply? I fail to see why it's necessary to remove these files. If they existed from a previous compile, they are outdated, so what. But I may be overlooking something here. Thanks, Ralf 2008-04-10 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/config/ltmain.m4sh (func_mode_compile

Re: move func_mode_compile a little earlier

2008-04-10 Thread Ralf Wildenhues
* Peter O'Gorman wrote on Thu, Apr 10, 2008 at 08:36:32AM CEST: 2008-04-10 Peter O'Gorman [EMAIL PROTECTED] * libltdl/config/ltmain.m4sh (func_win32_libid, func_generate_dlsyms, func_extract_an_archive, func_extract_archives, func_mode_help): move these so they appear after

Re: Avoid warning on cygwin

2008-04-10 Thread Ralf Wildenhues
* Eric Blake wrote on Thu, Apr 10, 2008 at 06:56:07PM CEST: src/.libs/lt-m4.c: In function `main': src/.libs/lt-m4.c:288: warning: implicit declaration of function `_setmode' OK to apply this? OK provided it's been tested under MinGW. I tested linux - mingw cross, where _setmode is declared,

Re: compile mode, XSI shell: do not fork for $xform

2008-04-10 Thread Ralf Wildenhues
* Peter O'Gorman wrote on Thu, Apr 10, 2008 at 02:13:02AM CEST: $ echo ada.ada.ada | sed -e s/\.[^.]*/.lo/ ada.lo.ada A $ helps: It's ok with this changed. Applied like this. Cheers, Ralf 2008-04-10 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/m4/libtool.m4

Re: Speed up progname, progpath setting

2008-04-10 Thread Ralf Wildenhues
* Peter O'Gorman wrote on Thu, Apr 10, 2008 at 11:21:01PM CEST: Peter O'Gorman wrote: Ralf Wildenhues wrote: curwd=`cd . pwd` Do we know when (with which shells) this can be replaced by $PWD portably? The ones that are not buggy and comply with the standards? :) (I don't know

Re: Speed up progname, progpath setting

2008-04-10 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Fri, Apr 11, 2008 at 12:12:45AM CEST: On Thu, 10 Apr 2008, Ralf Wildenhues wrote: On systems where $PWD happens to not work, we shouldn't care about performance, they are so rare. We should define what it means for PWD to work. My intention was to use it only

Re: No need for object file removal before compilation

2008-04-12 Thread Ralf Wildenhues
already be used by another libtool instance, it could've grabbed the lock already. Fixed by updating $removlist before removing the lockfile. I would have been inclined to ok this one. OK for the updated one? It makes for 5% faster libtool --mode=compile gcc -c a.c Thanks, Ralf 2008-04-12 Ralf

Re: No need for object file removal before compilation

2008-04-12 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Sat, Apr 12, 2008 at 08:56:24AM CEST: OK for the updated one? It makes for 5% faster libtool --mode=compile gcc -c a.c I should be more precise: the above holds for an empty file a.c, measured using a script that calls the above 50 times, pre and post patch. I

switching to git, please no more CVS commits

2008-04-12 Thread Ralf Wildenhues
Hello, please do not commit to CVS for now, until further notification. Thanks, Ralf

Re: add cegcc support

2008-04-13 Thread Ralf Wildenhues
Hello Vincent, * Vincent Torri wrote on Sun, Apr 13, 2008 at 11:51:00PM CEST: Here is a patch to make libtool support the cegcc compiler. The patch is made wrt the latest cvs. Thank you for the patch. Please bootstrap Libtool 2.2.2 with this patch, build it for this system (cross-compile

Re: add cegcc support

2008-04-14 Thread Ralf Wildenhues
* Vincent Torri wrote on Mon, Apr 14, 2008 at 08:39:02AM CEST: Here is a patch to make libtool support the cegcc compiler. The patch is made wrt the latest cvs. Thank you for the patch. Please bootstrap Libtool 2.2.2 with this patch, build it for this system (cross-compile if you have to),

Re: Allow bootstrap from git repository

2008-04-14 Thread Ralf Wildenhues
Hi Peter, * Peter O'Gorman wrote on Tue, Apr 15, 2008 at 05:43:36AM CEST: * Makefile.am: Expect mkstamp to return only rev date. * libltdl/config/mkstamp: return a revision and date based on the number of lines in the input and the topmost date. Hmm, I was almost saying OK, except the last

.gitignore

2008-04-14 Thread Ralf Wildenhues
OK? (No ChangeLog entry.) Cheers, Ralf Ignore generated files. diff --git a/.gitignore b/.gitignore new file mode 100644 index 000..252219d --- /dev/null +++ b/.gitignore @@ -0,0 +1,47 @@ +*.aux +*.cp +*.cps +*.dvi +*.fn +*.ky +*.la +*.lo +*.log +*.o +*.pg +*.toc +*.tp +*.vr ++build

Re: Allow bootstrap from git repository

2008-04-14 Thread Ralf Wildenhues
* Peter O'Gorman wrote on Tue, Apr 15, 2008 at 07:40:54AM CEST: Ralf Wildenhues wrote: But how do you ensure @MACRO_SERIAL@ increases after ChangeLog rotation? Ha! Having never done a ChangeLog rotation or paid it much attention, I figured that the new ChangeLog would have a commit

Re: Allow bootstrap from git repository

2008-04-15 Thread Ralf Wildenhues
Hi Peter, * Peter O'Gorman wrote on Tue, Apr 15, 2008 at 04:54:10PM CEST: I looked again, and at first counted all the lines in all the changelogs, but then it seemed better to count only what looked like dates. Better. I don't mind that multiple commits like this count as one: 2008-xx-xx

Re: Allow bootstrap from git repository

2008-04-15 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Tue, Apr 15, 2008 at 08:15:45PM CEST: If we need an increasing serial number, can't we obtain one from git? Does git offer the ability to update an RCS-style string in a text file each time it is modified? If that is the case, then we would just need to arrange

Re: add cegcc support

2008-04-15 Thread Ralf Wildenhues
* Vincent Torri wrote on Tue, Apr 15, 2008 at 06:34:11PM CEST: On Mon, 14 Apr 2008, Ralf Wildenhues wrote: Against CVS HEAD is just fine, sorry for the confusion. The important thing is that we see how the testsuite fares in order to get an idea how well the port works, and can fix simple

git best practices

2008-04-16 Thread Ralf Wildenhues
I'm writing this in the hope that it will be useful. Some things are certainly up for debate, and I don't have a strong opinon on them. This is not a git tutorial, or a how to use git effectively with GNU packages tutorial. I believe Jim has written something the latter end before for coreutils

Re: git ChangeLog merge driver

2008-04-16 Thread Ralf Wildenhues
Hi Eric, * Eric Blake wrote on Wed, Apr 16, 2008 at 10:57:12PM CEST: Subject: [PATCH] Make ChangeLog merges easier. * .gitattributes: New file. * HACKING: Mention how it helps. OK, thanks! Cheers, Ralf

Re: add cegcc support

2008-04-17 Thread Ralf Wildenhues
* Vincent Torri wrote on Thu, Apr 17, 2008 at 06:31:34PM CEST: The README file explains it: run make check VERBOSE=yes TESTS=tests/demo-shared.test tests/demo-make.test tests/demo-exec.test Also, please run the other half of the tests (the new testsuite) using make check-local and post

Re: add cegcc support

2008-04-17 Thread Ralf Wildenhues
Hello Vincent, * Vincent Torri wrote on Fri, Apr 18, 2008 at 06:51:45AM CEST: What about tests/testsuite.log? Thank you very much for providing this. # -*- compilation -*- 37. am-subdir.at:33: testing ... [...] checking build system type...

Re: add cegcc support

2008-04-17 Thread Ralf Wildenhues
* Vincent Torri wrote on Fri, Apr 18, 2008 at 07:25:01AM CEST: All failing tests fail due to config.sub files which are not up to date. Can you please go ahead and the config.sub file that comes with your installed Automake package to contain your proposed changes? It should be located at

Re: Feature request: setting env vars for binary wrappers

2008-04-18 Thread Ralf Wildenhues
* Roumen Petrov wrote on Fri, Apr 18, 2008 at 08:32:23PM CEST: About no way to fix this problem with autotools. Why ? As example libxml can run binaries from build dir. In one of the tests is created specific xml catalog and application is run with this catalog instead with system. OK,

Re: Feature request: setting env vars for binary wrappers

2008-04-18 Thread Ralf Wildenhues
* Roumen Petrov wrote on Fri, Apr 18, 2008 at 08:53:12PM CEST: In some cases application depend from other services. In this case a specific to project wrapper script has to run services, to check if service is run, to run project application and when application finish to stop service.

Re: Feature request: setting env vars for binary wrappers

2008-04-18 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Fri, Apr 18, 2008 at 09:15:55PM CEST: The only substantial change is for static builds which currently don't have a wrapper. Yes. The static build is a more significant concern since static builds are often used for debugging purposes and if we hide the static

Re: No need for object file removal before compilation

2008-04-18 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Sat, Apr 12, 2008 at 05:51:28PM CEST: The patch looks ok to me. We shall see about the actual performance improvement. Thanks, applied. Cheers, Ralf 2008-04-12 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/config/ltmain.m4sh (func_mode_compile): Avoid

Re: expr complaint on FreeBSD for MinGW cross

2008-04-19 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Sat, Apr 19, 2008 at 01:56:59AM CEST: Ouch. ETOOLATE. Of course $name is used, in libname_spec for example. Please ignore this, I'll post an updated patch. OK to apply? Thanks, Ralf 2008-04-19 Ralf Wildenhues [EMAIL PROTECTED] Fix nonportable use

Re: func_arith and func_len

2008-04-19 Thread Ralf Wildenhues
[ http://lists.gnu.org/archive/html/libtool-patches/2008-04/msg00096.html ] Hello Bob, Roland, * Bob Friesenhahn wrote on Sat, Apr 19, 2008 at 05:13:13PM CEST: On Sat, 19 Apr 2008, Ralf Wildenhues wrote: Let's reuse at_func_arith from Autotest, to redo one of the link mode speedups that fell

Re: expr complaint on FreeBSD for MinGW cross

2008-04-19 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Sat, Apr 19, 2008 at 06:13:58PM CEST: On Sat, 19 Apr 2008, Ralf Wildenhues wrote: OK to apply? While I have not tested this patch, it looks ok to apply to me. Thanks, pushed as well. Cheers, Ralf

Re: func_arith and func_len

2008-04-19 Thread Ralf Wildenhues
Hello Roland, * Roland Mainz wrote on Sat, Apr 19, 2008 at 11:46:02PM CEST: On 4/19/08, Ralf Wildenhues [EMAIL PROTECTED] wrote: [ http://lists.gnu.org/archive/html/libtool-patches/2008-04/msg00096.html ] [snip] 2008-04-19 Ralf Wildenhues [EMAIL PROTECTED] Exploit shell

Re: func_arith and func_len

2008-04-19 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Sun, Apr 20, 2008 at 02:42:00AM CEST: On Sun, 20 Apr 2008, Ralf Wildenhues wrote: Sorry, but: first, there is no floating point in either Libtool, nor Posix shell semantics. Second, if you can prove to me (by posting timing comparisons) that conversion from integer

Re: func_arith and func_len

2008-04-19 Thread Ralf Wildenhues
* Roland Mainz wrote on Sun, Apr 20, 2008 at 03:03:24AM CEST: On 4/20/08, Ralf Wildenhues [EMAIL PROTECTED] wrote: Second, if you can prove to me (by posting timing comparisons) that conversion from integer to string or vice versa, Here come the requested numbers: -- snip -- $ timex

Re: [PATCH 363] Allow better compile time type checking for lt_dladvise

2008-04-20 Thread Ralf Wildenhues
* Gary V. Vaughan wrote on Mon, Apr 21, 2008 at 02:39:37AM CEST: * libltdl/lt__private.h (lt__handle, lt__advise): Typedefs removed in favour of... * libltdl/ltdl.h (lt_dlhandle), libltdl/libltdl/lt_dlloader.h (lt_dladvise): ...changing from void* to forward declarations. *

Re: libtool performance status

2008-04-20 Thread Ralf Wildenhues
Hi Bob, * Bob Friesenhahn wrote on Mon, Apr 21, 2008 at 12:55:08AM CEST: I am saddened to report that as of yesterday, build times for git libtool are now similar to libtool 2.2.2. This is not to say that there has not been progress. The reported system times have gone down (from 113 to

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