[coLinux-devel] Re: Mingw patch for CoLinux console/deamon

[Dan A. <da...@co...>]

On Fri, Apr 09, 2004 at 02:46:20PM -0700, Steven Edwards wrote:
> Hello All,
> Well here it is in all of its glory. All 5 lines of it. It works but we
> (ReactOS team) have not tested it beyond booting the kernel.

So the kernel really booted. Can you post a screenshot?

> There are a few changes that will need to be made before this can be
> merged in and we can drop Cygwin I guess. But I beg and plead that we
> do this so we can drop Cygwin totaly if we are not going to need it for
> the Xserver.

I agree, Cygwin should be dropped.

> 1. The FLTK console wont build on the Mingw cross-compiler as we lack
> support for it. I guess we could ship our own copy.

I got it built using the Mingw cross compiler, but I couldn't test the 
library because of some linkage problems.

I've fiddled a few hours with creating a i686-pc-mingw32 cross compiler, 
and found a bunch of annoying problems with gcc's build - actual build 
system bugs. Do you have scripts that automatically create a mingw32 
cross compilation toolkit on Linux using recent versions of gcc 
and binutils? What versions do you use? I'd really benefit from it.

I also tried to use the -mno-cygwin gcc flag. It worked for the daemon, 
but then I found out that libstdc++ depends on a symbol inside cygwin1.dll! 
So I had to specially compile a new libstdc++ that doesn't refer that 
symbol (___getreent. I had to use --disable-threads and pull more defines 
from the hat to get it working).

Anyway, as from the 20040411 snapshot, the daemon and console-nt don't 
depend on Cygwin, so you can test console-nt to see if it works on
ReactOS.

> 2. The user has to go download/install the mxml sources for the
> mingw-crosscompiler. Once again its small we could ship this with the
> coLinux sources.

Or we could ship a patch for it.

> 3. You need to change i*86-cygwin-blah to -*86-mingw32-blah for all of
> the commands in your makefile to support a mingw cross

It only needs to be changed in one location.

> 4. We need to add some configure like checking for these headers I have
> added the ifdefs for in this patch.
> 
> 5. It links to ws2_32 but I have not tested the networking stuff so we
> will have to adapt the rest to use use Winsock but it will still be
> faster than using Cygwin->Winsock.

That networking code in the daemon is not that important (used for 
debugging).

-- 
Dan Aloni
da...@co...