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

2008-08-20 Thread Markus Duft
 
 Hi Ralf,
 
 Ralf Wildenhues skrev:
  Hi Peter,
 
snip
 
 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

Peter Rosin skrev:

Hi Markus,

Markus Duft skrev:

Hi Ralf,

Ralf Wildenhues skrev:

Hi Peter,


snip

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,
 
snip
 
 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