Testsuite issues (was: [PATCH 6/7] Convert file name to toolchain format when invoking $NM.)

Den 2010-09-10 07:38 skrev Ralf Wildenhues: > 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-10 19:19 skrev Ralf Wildenhues: > * Peter Rosin wrote on Fri, Sep 10, 2010 at 03:11:39PM CEST: >> 1. Not covered, you'd need to link a program with -dlopen self, >> but also with a non-libtool object (.o) and/or an non-libtooled old >> archive (.a). Neither appears to happen in the test

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

* Peter Rosin wrote on Fri, Sep 10, 2010 at 03:11:39PM CEST: > Den 2010-09-05 23:29 skrev Ralf Wildenhues: > > You're gonna hate me for this, I already know, but: does this patch fix > > testsuite failures, are all code paths covered? If not, we need to > > improve the test suite. If yes, please

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

Den 2010-09-10 16:09 skrev Charles Wilson: > On 9/10/2010 10:07 AM, Peter Rosin wrote: >> It's dead, you need the below patch to animate it. There is no sane >> way you can influence the sharedlib_from_linklib variable. > > Well, that's just a bug with a known but uncommitted patch. So he's > sto

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

On 9/10/2010 10:07 AM, Peter Rosin wrote: > It's dead, you need the below patch to animate it. There is no sane > way you can influence the sharedlib_from_linklib variable. Well, that's just a bug with a known but uncommitted patch. So he's stopped breathing, but it's nothing a little CPR won't f

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

Den 2010-09-10 15:54 skrev Charles Wilson: > On 9/10/2010 9:11 AM, Peter Rosin wrote: >> func_cygming_gnu_implib_p >> - >> >> 6. Dead code. Needs the sharedlib_from_linklib -> sharedlib_from_linklib_cmd >> typo fix. So, a previous testsuite deficiency that should not hold ba

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

On 9/10/2010 9:11 AM, Peter Rosin wrote: > func_cygming_gnu_implib_p > - > > 6. Dead code. Needs the sharedlib_from_linklib -> sharedlib_from_linklib_cmd > typo fix. So, a previous testsuite deficiency that should not hold back this > patch. > > func_cygming_gnu_implib_p a

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

Den 2010-09-05 23:29 skrev Ralf Wildenhues: > * Peter Rosin wrote on Sun, Sep 05, 2010 at 10:02:11PM CEST: >> Subject: [PATCH 6/7] Convert file name to toolchain format when invoking $NM. >> >> * libltdl/config/ltmain.m4sh (func_generate_dlsyms) >> (func_win32_libid

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

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

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

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? Right now, I'm seeing a regression just building lib

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

On 9/8/2010 5:52 PM, Peter Rosin wrote: > Den 2010-09-05 23:29 skrev Ralf Wildenhues: >> * Peter Rosin wrote on Sun, Sep 05, 2010 at 10:02:11PM CEST: >>> Subject: [PATCH 6/7] Convert file name to toolchain format when invoking >>> $NM. >>> >>> * lib

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

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 . > $sharedlib_from_linklib "$dlprefile"

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

Den 2010-09-05 23:29 skrev Ralf Wildenhues: > * Peter Rosin wrote on Sun, Sep 05, 2010 at 10:02:11PM CEST: >> Subject: [PATCH 6/7] Convert file name to toolchain format when invoking $NM. >> >> * libltdl/config/ltmain.m4sh (func_generate_dlsyms) >> (func_win32_libid

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

* Peter Rosin wrote on Sun, Sep 05, 2010 at 10:02:11PM CEST: > Subject: [PATCH 6/7] Convert file name to toolchain format when invoking $NM. > > * libltdl/config/ltmain.m4sh (func_generate_dlsyms) > (func_win32_libid, func_cygming_gnu_implib_p) > (func_cygming_ms_implib_p): When

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

>From 6b925d5364311ad57a0df041f59b34623e8b3cb2 Mon Sep 17 00:00:00 2001 From: Peter Rosin Date: Sun, 5 Sep 2010 17:54:21 +0200 Subject: [PATCH 6/7] Convert file name to toolchain format when invoking $NM. * libltdl/config/ltmain.m4sh (func_generate_dlsyms) (func_win32_li

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

>From acf4b65fb20f86e0757f13a0dce11c1fb27411d3 Mon Sep 17 00:00:00 2001 From: Peter Rosin Date: Wed, 1 Sep 2010 22:03:23 +0200 Subject: [PATCH 6/7] Convert file name to toolchain format when invoking $NM. * libltdl/config/ltmain.m4sh (func_generate_dlsyms) (func_win32_li