Re: [patch #6448] [MSVC 7/7] Add MSVC Support

[Ralf Wildenhues]

* Peter Rosin wrote on Sat, Aug 09, 2008 at 10:43:26AM CEST:
> Ralf Wildenhues skrev:
>> * Peter Rosin wrote on Thu, Aug 07, 2008 at 02:10:41PM CEST:
>>> 2008-08-07  Peter Rosin  <address@hidden>
>>>
>>>     * libltdl/m4/libtool.m4 (_LT_CHECK_MAGIC_METHOD): Disable nocase
>>>     handling for cross compiles.
>>
>>> +file_magic_glob=
>>> +want_nocaseglob=no
>>> +if test "$build" = "$host"; then
>>> +  case $host_os in
>>> +  cygwin* | mingw* | pw32*)
>>
>> What if build is cygwin* and host is mingw*, for example?

> I expect that you will not need the nocase search in that case. Just use
> the exact names as they appear in the w32api. I.e. we define that from a
> libtool p.o.w. the canonical name of the ddraw import library is -lddraw.
> The nocase stuff is then *only* needed for MSVC, and using MSVC from
> cygwin isn't supported atm anyway, so nothing is lost.

Oh, I didn't remember that MSVC wasn't supported from Cygwin anyway.
Hmm, that makes me wonder why the above enables caseglobbing for the
cygwin non-cross case.

> Are you saying we should aim for more?

Naah, just checking to make sure we haven't forgotten anything
inadvertently.

Thanks, and I guess you can go ahead and apply this,
Ralf