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

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: [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!

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: 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: [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: [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] 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: 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 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: 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: 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: [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

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

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

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

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

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

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

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

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 #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

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

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

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?

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

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

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

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