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

[Peter Rosin]

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 was: The specified module could not be found.
>> can't open the module tests/mdemo/libfoo2.la!
>> error was: The specified module could not be found.
>> can't open the module tests/mdemo/libfoo2!
>> error was: The specified module could not be found.
> 
> Yes.
> 
> The problem is in how mdemo_static.exeS.c is constructed;
> lt__PROGRAM__LTX_preloaded_symbols contains "header" entries for each
> -dlpreopen'ed library, but they are:
> 
>   {"cygsub-0.dll", (void *) 0},
> 
> when they should be
> 
>   {"libsub.a", (void *) 0},
> 
> Manually making that change and re-linking fixes the error.  I just need
> to track down *why* the wrong library name is being used and fix it,
> without breaking mdemo.exeS.c (the one that dlpreopens the DLLs, and
> SHOULD be specifying 'cygsub-0.dll' etc).

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]

Maybe you knew that this patch was the cause of the regression, but
I didn't understand that until I bisected the regression. Your comment
a few message back:

Den 2010-09-09 00:14 skrev Charles Wilson:
> (However, there is an unfixed bug here; apparently something has changed
> in libtool between when this patch was created, and when it was
> committed, such that the static lib part no longer works properly; this
> is why on cygwin and mingw the mdemo-exec tests STILL fail.  Very
> frustrating.  I plan to track this down in the next day or so).

came into new light. I mean, "this patch" probably refers to the
above mentioned commit, right?

Cheers,
Peter