Re: [cygwin] cwrapper emits wrapper script

2007-04-23 Thread Ralf Wildenhues
* Charles Wilson wrote on Tue, Apr 24, 2007 at 04:34:41AM CEST: > Ralf Wildenhues wrote: > >> (note I'm not asking you to do this work here; actually, I'd like to ask >> you not to fix even more different things with one patch. Merely noting >> it in case you're interested.) > > Ok, but it my defe

Re: [cygwin] cwrapper emits wrapper script

2007-04-23 Thread Charles Wilson
Ralf Wildenhues wrote: * Charles Wilson wrote on Sat, Apr 21, 2007 at 03:03:02AM CEST: When the wrapper foo.exe is launched, it generates a new wrapper script .libs/foo_ltshwrapper Hmm, I'm wondering whether we should keep prefixing within .libs. Maybe .libs/ltsh-foo ? WDYT? Meh, I do

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

2007-04-23 Thread Bob Friesenhahn
On Mon, 23 Apr 2007, Ralf Wildenhues wrote: Both of those two tests work with Debian's current libtool, but break with your patch. Note that they also break if -static is used for all libraries (add LDFLAGS=-static to TESTSUITEFLAGS to try out), rather than only to the final ones. Libtool deve

Re: [cygwin] cwrapper emits wrapper script

2007-04-23 Thread Ralf Wildenhues
Hello Charles, Thanks for the patch again. First review: * Charles Wilson wrote on Sat, Apr 21, 2007 at 03:03:02AM CEST: > This patch depends on this one: > http://lists.gnu.org/archive/html/libtool-patches/2007-04/msg00048.html (unfortunately, due to idiocy on my part, that patch will have to

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

2007-04-23 Thread Kurt Roeckx
On Mon, Apr 23, 2007 at 07:31:38PM +0200, Ralf Wildenhues wrote: > Hello Kurt, all, > > Thanks for the patch. > > * Kurt Roeckx wrote on Sun, Apr 22, 2007 at 01:22:30AM CEST: > > > > So would the attached patch be acceptable for now? I'm thinking about > > adding that to the Debian patch. > >

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

2007-04-23 Thread Ralf Wildenhues
Hello Kurt, all, Thanks for the patch. * Kurt Roeckx wrote on Sun, Apr 22, 2007 at 01:22:30AM CEST: > > So would the attached patch be acceptable for now? I'm thinking about > adding that to the Debian patch. Hmm. It breaks when -static is used. Here's how I tested your patch, as a new test

Re: HEAD: fix the inherited_linker_flags regression

2007-04-23 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Thu, Apr 12, 2007 at 12:09:07AM CEST: > Prior to this patch, inherite_linker_flags from all dependent libraries > would accumulate [...] No comments in 11 days. I've applied this patch now. Cheers, Ralf > 2007-04-11 Ralf Wildenhues <[EMAIL PROTECTED]> > > * l

Re: [cygwin, libtool] use shell function to emit wrapper scripts and wrapper.exe source

2007-04-23 Thread Ralf Wildenhues
Hello Charles, * Charles Wilson wrote on Thu, Apr 19, 2007 at 11:40:58PM CEST: > > 2007-04-19 Charles Wilson <[EMAIL PROTECTED]> > > * libltdl/config/ltmain.m4sh (func_mode_link): move wrapper > script generation from here... > (func_emit_libtool_wrapper_script): to this new fu

ltmain not rebuilt

2007-04-23 Thread Ralf Wildenhues
Just to let everyone know again: current Makefile.am of CVS HEAD does NOT ensure that the libtool script is rebuilt if you change ltmain.m4sh and run `make'. The 2007-03-26 change broke this, and any testing of patches to ltmain.m4sh will be affected. I've posted a partial workaround here: