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

[Peter Rosin]

Ralf Wildenhues skrev:
> Hi Peter,
>
> * Peter Rosin wrote on Thu, Aug 07, 2008 at 02:10:41PM CEST:
> > Peter Rosin skrev:
> > > To fix the test failure on the MinGW cross compile, it might be enough
> > > to disable the nocase stuff in libtool.m4 for cross compiles (take the
> > > default branch in the case $host_os statement, near the end of
> > > _LT_CHECK_MAGIC_METHOD). This should cause a skip of the offending
> > > test.
> > Like this patch. Can you verify if it works for i586-mingw32msvc-gcc?
>
> Yes it works but ...
>
> > 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?

Hi Ralf,

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.

Are you saying we should aim for more?

Cheers,
Peter