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

2008-08-20 Thread Markus Duft
> 
> Hi Ralf,
> 
> Ralf Wildenhues skrev:
> > Hi Peter,
> >

> 
> So, I guess I'm saying that I'd prefer sticking to:
>  if test "$GCC" != yes; then
>reload_cmds=false
>  fi
> 
> Ok to push?

Could this break parity support? I know It's not in the tree yet, but I
still hope, that ralf comes to looking into my patch some day

> 
> > I'm thinking of two problem cases:
> > - could be someone would like to use cccl with GNU binutils ld

I guess neither cccl nor parity will work with GNU ld right now, BUT: M$ own
cc and cc89 scripts (which are used by M$ to build nearly the whole system
with MSVC as backend) use GNU ld, maybe those aren't too important, since
not many people except M$ use it, but still ...

> 
> I have wasted quite a bit of time trying to make cccl work. Can someone
> pleasepleaseprettypleasewithsugerontop point me to the *exact* cccl
> script I should use, what environment I should have and how I should
> configure to make it work?

Use parity instead :)

Cheers, Markus

> 
> > - 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"?  ;-)
> 
> Err, yes, what makes you make that statement? The same goes for me, but
> cygwin is also w32 but most of the time you are best off if you forget
> that when you are using cygwin, and that's one of the reasons the name
> w32 is not all that useful. When I'm aware of what I'm doing I try to
> hold back any use of w32, it's generally too unspecific. Any time I see
> w32, I think: "Ok, what is not included this time?"
> 
> > 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.
> 
> I will, thanks in advance!
> 
> Cheers,
> Peter
> 






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

2008-08-20 Thread Peter Rosin

Hi Markus,

Markus Duft skrev:

Hi Ralf,

Ralf Wildenhues skrev:

Hi Peter,




So, I guess I'm saying that I'd prefer sticking to:
 if test "$GCC" != yes; then
   reload_cmds=false
 fi

Ok to push?


Could this break parity support? I know It's not in the tree yet, but I
still hope, that ralf comes to looking into my patch some day


Well, my guess is that you would know if you have implemented reloadable
objects (the -r switch) in parity. If you don't know, I expect reloadable
objects are not supported and that this patch will help parity.

BTW, did you file a copyright assignment?


I'm thinking of two problem cases:
- could be someone would like to use cccl with GNU binutils ld


I guess neither cccl nor parity will work with GNU ld right now, BUT: M$ own
cc and cc89 scripts (which are used by M$ to build nearly the whole system
with MSVC as backend) use GNU ld, maybe those aren't too important, since
not many people except M$ use it, but still ...


This is exactly my point, the only thing that currently works with libtool
and gnu ld is gcc (at least I think so). So, why make exceptions at this
time for something that doesn't work anyway? It can be added when (if ever)
all the other bits needed for cross toolchain support are added.

Regarding cc and cc89, that would be with interix as host, so not affected
by this. Right?


I have wasted quite a bit of time trying to make cccl work. Can someone
pleasepleaseprettypleasewithsugerontop point me to the *exact* cccl
script I should use, what environment I should have and how I should
configure to make it work?


Use parity instead :)


That's just silly, as the whole point of me getting cccl to fly is to not
break any existing cccl support. And the whole point of my current efforts
is to require as little 3rd party stuff as possible to use libtool with MS
tools.

Cheers,
Peter




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

2008-08-20 Thread Peter Rosin

Peter Rosin skrev:

Hi Markus,

Markus Duft skrev:

Hi Ralf,

Ralf Wildenhues skrev:

Hi Peter,




So, I guess I'm saying that I'd prefer sticking to:
 if test "$GCC" != yes; then
   reload_cmds=false
 fi

Ok to push?


Could this break parity support? I know It's not in the tree yet, but I
still hope, that ralf comes to looking into my patch some day


Well, my guess is that you would know if you have implemented reloadable
objects (the -r switch) in parity. If you don't know, I expect reloadable
objects are not supported and that this patch will help parity.


Hold on! Since parity is using a $host matching *winnt*, this patch doesn't 
affect parity at all.



BTW, did you file a copyright assignment?


I'm thinking of two problem cases:
- could be someone would like to use cccl with GNU binutils ld


I guess neither cccl nor parity will work with GNU ld right now, BUT: 
M$ own
cc and cc89 scripts (which are used by M$ to build nearly the whole 
system

with MSVC as backend) use GNU ld, maybe those aren't too important, since
not many people except M$ use it, but still ...


This is exactly my point, the only thing that currently works with libtool
and gnu ld is gcc (at least I think so).


I.e. for these cases:

case $host_os in
  cygwin* | mingw* | pw32* | cegcc*)


 So, why make exceptions at this
time for something that doesn't work anyway? It can be added when (if ever)
all the other bits needed for cross toolchain support are added.

Regarding cc and cc89, that would be with interix as host, so not affected
by this. Right?


Cheers,
Peter




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

2008-08-20 Thread Markus Duft
> 
> Peter Rosin skrev:
> > Hi Markus,
> >

> 
> Hold on! Since parity is using a $host matching *winnt*, this patch
> doesn't
> affect parity at all.

Yeah, right, i oversaw that :)

> 
> > BTW, did you file a copyright assignment?

Sure, now 2 years ago, or so...

Cheers, Markus

> >
>  I'm thinking of two problem cases:
>  - could be someone would like to use cccl with GNU binutils ld
> >>
> >> I guess neither cccl nor parity will work with GNU ld right now,
> BUT:
> >> M$ own
> >> cc and cc89 scripts (which are used by M$ to build nearly the whole
> >> system
> >> with MSVC as backend) use GNU ld, maybe those aren't too important,
> since
> >> not many people except M$ use it, but still ...
> >
> > This is exactly my point, the only thing that currently works with
> libtool
> > and gnu ld is gcc (at least I think so).
> 
> I.e. for these cases:
> 
> case $host_os in
>cygwin* | mingw* | pw32* | cegcc*)
> 
> >  So, why make exceptions at
> this
> > time for something that doesn't work anyway? It can be added when (if
> ever)
> > all the other bits needed for cross toolchain support are added.
> >
> > Regarding cc and cc89, that would be with interix as host, so not
> affected
> > by this. Right?
> 
> Cheers,
> Peter
> 






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

2008-08-20 Thread Ralf Wildenhues
Hello Markus,

* Markus Duft wrote on Wed, Aug 20, 2008 at 09:26:10AM CEST:
> 
> Could this break parity support?

We try to ensure that it doesn't happen.

> I know It's not in the tree yet, but I
> still hope, that ralf comes to looking into my patch some day

Yes.  My plan is to get Peter's stuff fully through once, then merge,
then sigh and probably take a couple of weeks off Libtool, then do the
same game with your parity patches.  I took Peter's patches first since
they have been waiting for longer.  I'll go crazy if I have to do both
at once.

Cheers,
Ralf