Re: [PATCH 6/7] Convert file name to toolchain format when invoking $NM.

Hi Peter, * Peter Rosin wrote on Fri, Sep 10, 2010 at 12:35:59AM CEST: > Den 2010-09-09 22:05 skrev Charles Wilson: > > On 9/9/2010 3:56 PM, Ralf Wildenhues wrote: > >> Secondly, I can help with unsupervised git bisect if you need. In > >>

Re: [PATCH 6/7] Convert file name to toolchain format when invoking $NM.

Den 2010-09-09 22:05 skrev Charles Wilson: > On 9/9/2010 3:56 PM, Ralf Wildenhues wrote: >> I understand that you're doing a difficult bug hunt here, and 6/7 is the >> only unapplied patch of this series (right?). I've looked at 6/7 again, >> and conclude that it has a low chance of regressing. >

Re: [PATCH 6/7] Convert file name to toolchain format when invoking $NM.

On 9/9/2010 3:56 PM, Ralf Wildenhues wrote: > I understand that you're doing a difficult bug hunt here, and 6/7 is the > only unapplied patch of this series (right?). I've looked at 6/7 again, > and conclude that it has a low chance of regressing. I agree. > If it makes things easier for you, th

Re: [PATCH 6/7] Convert file name to toolchain format when invoking $NM.

Peter, Charles, I understand that you're doing a difficult bug hunt here, and 6/7 is the only unapplied patch of this series (right?). I've looked at 6/7 again, and conclude that it has a low chance of regressing. If it makes things easier for you, then I'm fine with you going ahead and applying

Re: [PATCH] Fix dependency tracking for MSYS/MinGW.

On 9/9/2010 12:57 PM, Charles Wilson wrote: > On 9/9/2010 4:59 AM, Peter Rosin wrote: >> As discussed in that other thread, namely >> http://lists.gnu.org/archive/html/libtool-patches/2010-09/msg00105.html >> I accidentally broke MSYS/MinGW. Here's an improved version of the patch >> shown in that

Re: [PATCH 6/7] Convert file name to toolchain format when invoking $NM.

Den 2010-09-09 18:12 skrev Charles Wilson: > Sorry, I just lost track of the thread of the various conversations. > When you presented a chain of 7 patches, and then they weren't all > committed simultaneously, I started to wonder (1-4, then later 5, then > even later 6, and 7 still hasn't been).

Re: [PATCH] Fix dependency tracking for MSYS/MinGW.

Den 2010-09-09 19:00 skrev Ralf Wildenhues: > * Peter Rosin wrote on Thu, Sep 09, 2010 at 10:59:30AM CEST: >> + func_stripname 'func_convert_file_' '' "$to_tool_file_cmd" > > func_stripname is fast with modern shells, but may fork with others. > Let's not introduce a number of forks here, just be

Re: [PATCH 6/7] Convert file name to toolchain format when invoking $NM.

Den 2010-09-09 18:02 skrev Charles Wilson: > After all, the point of that patch was to correct the FAILURE of that > particular test! > > So, somewhere between 2009-01 and e83da49a, some OTHER change to libtool > fixed (?) the mdemo test on mingw/msys. > > Sigh. Doesn't look like it. 4efe88d335

Re: [PATCH] Fix dependency tracking for MSYS/MinGW.

* Peter Rosin wrote on Thu, Sep 09, 2010 at 10:59:30AM CEST: > As discussed in that other thread, namely > http://lists.gnu.org/archive/html/libtool-patches/2010-09/msg00105.html > I accidentally broke MSYS/MinGW. Here's an improved version of the patch > shown in that message to fix the build issu

Re: [PATCH] Fix dependency tracking for MSYS/MinGW.

On 9/9/2010 4:59 AM, Peter Rosin wrote: > As discussed in that other thread, namely > http://lists.gnu.org/archive/html/libtool-patches/2010-09/msg00105.html > I accidentally broke MSYS/MinGW. Here's an improved version of the patch > shown in that message to fix the build issue. Sorry again. > >

Re: [PATCH 6/7] Convert file name to toolchain format when invoking $NM.

On 9/9/2010 11:33 AM, Peter Rosin wrote: > Ok, I bisected the failure to e83da49a1faf9df1c7e351df9e9b175 > "[cygwin|mingw] fix dlpreopen with --disable-static" > > You know, just fix the broken stuff...without[couldn't resist] Oh, I know. This was *precisely* what that patch was supposed

Re: [PATCH 6/7] Convert file name to toolchain format when invoking $NM.

On 9/9/2010 3:19 AM, Peter Rosin wrote: > Den 2010-09-09 06:18 skrev Charles Wilson: >> Peter, a question about your current patch series: with it only >> partially committed, do you expect errors? Are we waiting for some >> other change upon which the current series depends, before it all "just >

Re: [PATCH 6/7] Convert file name to toolchain format when invoking $NM.

Den 2010-09-09 15:25 skrev Charles Wilson: > On 9/9/2010 5:47 AM, Peter Rosin wrote: >> Anyway, both fail in pretty much the same way for me: >> >> can't open the module tests/mdemo/foo1.la! >> error was: The specified module could not be found. >> can't open the module tests/mdemo/foo1! >> error w

Re: [PATCH 6/7] Convert file name to toolchain format when invoking $NM.

On 9/9/2010 5:47 AM, Peter Rosin wrote: > Anyway, both fail in pretty much the same way for me: > > can't open the module tests/mdemo/foo1.la! > error was: The specified module could not be found. > can't open the module tests/mdemo/foo1! > error was: The specified module could not be found. > can

Re: [PATCH 6/7] Convert file name to toolchain format when invoking $NM.

Den 2010-09-09 00:19 skrev Charles Wilson: > On 9/8/2010 6:14 PM, Charles Wilson wrote: >> On 9/8/2010 5:52 PM, Peter Rosin wrote: > >>> sharedlib_from_linklib_cmd, which is not used anywhere according > >> # no lafile. user explicitly requested -dlpreopen . >

[PATCH] Fix dependency tracking for MSYS/MinGW.

Hi! As discussed in that other thread, namely http://lists.gnu.org/archive/html/libtool-patches/2010-09/msg00105.html I accidentally broke MSYS/MinGW. Here's an improved version of the patch shown in that message to fix the build issue. Sorry again. Ok to push? Cheers, Peter >From 747d3b2a27abe

Re: [PATCH 6/7] Convert file name to toolchain format when invoking $NM.

Den 2010-09-09 06:18 skrev Charles Wilson: > Peter, a question about your current patch series: with it only > partially committed, do you expect errors? Are we waiting for some > other change upon which the current series depends, before it all "just > works" again...or are things fubared now? I