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

[Ralf Wildenhues]

Hi Peter,

* Peter Rosin wrote on Sun, Aug 17, 2008 at 08:47:12AM CEST:
>
> One easy way to avoid cc_basename is to simply leave this for the next
> non-gnu tool to fix, i.e.:
>
> --- a/libltdl/m4/libtool.m4
> +++ b/libltdl/m4/libtool.m4
> @@ -3024,7 +3024,12 @@ case $reload_flag in
>  esac
>  reload_cmds='$LD$reload_flag -o $output$reload_objs'
>  case $host_os in
> +  cygwin* | mingw* | pw32* | cegcc*)
> +    if test "$GCC" != yes; then
> +      reload_cmds=false
> +    fi
> +    ;;

Hmm, a bit better.  What about something like this?
  
case $host_os in
  cygwin* | mingw* | pw32* | cegcc*)
    if test "$GCC" != yes; then
      case $LD in
      link* | LINK* | lib* | LIB* ) reload_cmds=false ;;
      esac
    fi
    ;;
   ...

If you think that is overkill, then please say so.
I'm thinking of two problem cases:
- could be someone would like to use cccl with GNU binutils ld
- could be another non-GCC compiler (we've had reports about
  those on w32 before, I don't remember whether that was Portland or so)


FWIW, did you know that 'w32' to me just means "Windows, 32bit"?  ;-)

Cheers,
Ralf

PS: I'm aware of other loose ends in this thread.  I'll get to them
eventually, hopefully.  ;-)  Feel free to ping me if I haven't by the
end of next (not this) weekend.