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

2010-09-13 Thread Peter Rosin
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.

2010-09-10 Thread Peter Rosin
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, func_cygming_gnu_implib_p

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

2010-09-10 Thread 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 back this patch. func_cygming_gnu_implib_p and

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

2010-09-10 Thread Peter Rosin
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 back this

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

2010-09-10 Thread 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 stopped breathing, but it's nothing a little CPR won't

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

2010-09-10 Thread Peter Rosin
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 stopped

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

2010-09-10 Thread Ralf Wildenhues
* 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.

2010-09-10 Thread Peter Rosin
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 testsuite.

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

2010-09-09 Thread Peter Rosin
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 import

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

2010-09-09 Thread 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 was: The specified module could not be found. can't

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

2010-09-09 Thread Charles Wilson
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 works

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

2010-09-09 Thread Charles Wilson
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 to

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

2010-09-09 Thread Peter Rosin
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.

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

2010-09-09 Thread 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. I agree. If it makes things easier for you, then

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

2010-09-09 Thread Peter Rosin
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. I

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

2010-09-09 Thread 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.

2010-09-08 Thread Peter Rosin
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, func_cygming_gnu_implib_p

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

2010-09-08 Thread 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 import library. $sharedlib_from_linklib

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

2010-09-08 Thread Charles Wilson
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. * libltdl/config/ltmain.m4sh (func_generate_dlsyms

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

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

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

2010-09-05 Thread Peter Rosin
From 6b925d5364311ad57a0df041f59b34623e8b3cb2 Mon Sep 17 00:00:00 2001 From: Peter Rosin p...@lysator.liu.se 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_libid

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

2010-09-05 Thread 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, func_cygming_gnu_implib_p) (func_cygming_ms_implib_p): When using the name lister

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

2010-09-01 Thread Peter Rosin
From acf4b65fb20f86e0757f13a0dce11c1fb27411d3 Mon Sep 17 00:00:00 2001 From: Peter Rosin p...@lysator.liu.se 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_libid