Re: mdemo ltdl failure

2007-04-24 Thread Ralf Wildenhues
* Charles Wilson wrote on Wed, Apr 25, 2007 at 03:53:47AM CEST: > Ralf Wildenhues wrote: > >> +AS_IF([test -z "$ARGZ_H"], >> +[AC_CACHE_CHECK( >> +[if argz actually works], >> +[lt_cv_sys_argz_works], >> +[case $host_os in #( > > I like this trick. Fixes syntax highligh

Re: preparing for 1.5.24

2007-04-24 Thread Charles Wilson
Ralf Wildenhues wrote: Thanks also for the documentation suggestion. Slightly rewritten suggestion to come up. Ping? [antecedent reposted below] -- Chuck Around line 3546 [probably 4476, now] in libtool.texi, something like: --%< @emph{Wi

Re: [cygwin] cwrapper emits wrapper script

2007-04-24 Thread Charles Wilson
Ralf Wildenhues wrote: Certainly. I was merely trying to not infer that you'd have to do even more work than the lot that you're already doing. Of course if you're ambitious go for it. ;-) Thanks for fixing the MinGW case here. Sure. Hmm, maybe one after the `rm -f "$prefix/bin/..."' and

Re: [cygwin] cwrapper emits wrapper script

2007-04-24 Thread Charles Wilson
Ralf Wildenhues wrote: * Ralf Wildenhues wrote on Tue, Apr 24, 2007 at 08:53:46AM CEST: * Charles Wilson wrote on Tue, Apr 24, 2007 at 04:34:41AM CEST: Ralf Wildenhues wrote: for (i=0; i What's that extra ; for BTW? If !DEBUGWRAPPER, then LTWRAPPER_DEBUGPRINTF() goes away completely, and g

Re: mdemo ltdl failure

2007-04-24 Thread Charles Wilson
Ralf Wildenhues wrote: * Charles Wilson wrote on Fri, Apr 20, 2007 at 12:25:27AM CEST: Ralf suggests testing this patch on solaris. I can't, but if Ralf does then I expect that the results will be the same as (4) and (5). Done. AS_CASE was a problem, as it's not in Autoconf-2.59. !!! I co

Re: Bug#419228: unnecessary linkage when libtool convenience libraries (noinst_LTLIBRARIES) are used

2007-04-24 Thread Eric Blake
According to Ralf Wildenhues on 4/24/2007 3:05 PM: > Let's document a useful variant of it then. Is this wording OK? > > Cheers, > Ralf > > * README, README.alpha: Document how to test an arbitrary > installed libtool script. Looks great to me! Go ahead and install these hints. --

Re: Bug#419228: unnecessary linkage when libtool convenience libraries (noinst_LTLIBRARIES) are used

2007-04-24 Thread Ralf Wildenhues
* Kurt Roeckx wrote on Mon, Apr 23, 2007 at 10:08:44PM CEST: > On Mon, Apr 23, 2007 at 07:31:38PM +0200, Ralf Wildenhues wrote: > > > > make check-local TESTSUITEFLAGS="-v -d -x -k indirect \ > >LIBTOOL=/path/to/the/1.5.x/libtool" > > This is a great trick, thanks! Let's document a use

Re: [cygwin] cwrapper emits wrapper script

2007-04-24 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Tue, Apr 24, 2007 at 08:53:46AM CEST: > * Charles Wilson wrote on Tue, Apr 24, 2007 at 04:34:41AM CEST: > > Ralf Wildenhues wrote: > > >>>for (i=0; i >>>{ > >>> -DEBUG("(main) newargz[%d] : %s\n",i,newargz[i]); > >>> +LTWRAPPER_DEBUGPRINTF("(main) newar

FYI: HEAD: testsuite nits

2007-04-24 Thread Ralf Wildenhues
AT_CAPTURE_FILE wasn't in Autoconf-2.59. For simplicity let's just avoid it when that is used. Also, the shlibpath test is a libtool test. Committed as obvious. Cheers, Ralf * tests/cmdline_wrap.at: Avoid `AT_CAPTURE_FILE' if it not defined (Autoconf < 2.60). * tests/shl

Re: Bug#419228: unnecessary linkage when libtool convenience libraries (noinst_LTLIBRARIES) are used

2007-04-24 Thread Ralf Wildenhues
Hi Bob, * Bob Friesenhahn wrote on Tue, Apr 24, 2007 at 03:04:39AM CEST: > On Mon, 23 Apr 2007, Ralf Wildenhues wrote: >> >> Libtool developers: ok to apply the patch, which serves to document >> behavior that works and should continue to do so? > > Yes, of course! Installed, thanks. I moved the

Re: mdemo ltdl failure

2007-04-24 Thread Ralf Wildenhues
* Charles Wilson wrote on Fri, Apr 20, 2007 at 12:25:27AM CEST: > Ralf suggests testing this patch on solaris. I can't, but if Ralf does then > I expect that the results will be the same as (4) and (5). Done. AS_CASE was a problem, as it's not in Autoconf-2.59. While fixing, I noted that we can