PATCH - Define needed flags when building for Mingw

[Steven Edwards]

Dont know if this is right or not as you all know I think configure is a little whore. =P

Maybe this should go somewhere else. I have about 12 or 13 other things that need to be fixed when
building WINE for Mingw and MSYS and I am tired of doing ./configure CFLAGS="-D__MINGW__
-D_WINDOWS -DWINE_NOWINSOCK" Maybe someone can suggest a better way rather than -DWINE_NOWINSOCK?

Thanks
Steven

Changelog: add defines for Mingw

cvs diff -u configure 
Index: configure
===================================================================
RCS file: /home/wine/wine/configure,v
retrieving revision 1.387
diff -u -r1.387 configure
--- configure	9 Jan 2003 06:04:33 -0000	1.387
+++ configure	10 Jan 2003 04:55:54 -0000
@@ -11973,7 +11973,8 @@

case $host_os in
   mingw32*)
-    CRTLIBS="-lmsvcrt" ;;
+    CRTLIBS="-lmsvcrt" ;
+    CFLAGS="=$CFLAGS -D__MINGW__ -D_WINDOWS -DWINE_NOWINSOCK" ;
 esac

cvs diff -u configure.ac 
Index: configure.ac
===================================================================
RCS file: /home/wine/wine/configure.ac,v
retrieving revision 1.119
diff -u -r1.119 configure.ac
--- configure.ac	9 Jan 2003 06:04:33 -0000	1.119
+++ configure.ac	10 Jan 2003 04:56:29 -0000
@@ -857,7 +857,8 @@
 AC_SUBST(CRTLIBS,"")
 case $host_os in
   mingw32*)
-    CRTLIBS="-lmsvcrt" ;;
+    CRTLIBS="-lmsvcrt" ;
+    CFLAGS="=$CFLAGS -D__MINGW__ -D_WINDOWS -DWINE_NOWINSOCK" ;
 esac

dnl **** Get the soname for libraries that we load dynamically ****

__________________________________________________
Do you Yahoo!?
Yahoo! Mail Plus - Powerful. Affordable. Sign up now.
http://mailplus.yahoo.com

[Dimitrie O. Paun]

On January 10, 2003 12:01 am, Steven Edwards wrote:
> - CFLAGS="=$CFLAGS -D__MINGW__ -D_WINDOWS -DWINE_NOWINSOCK" ;

Why do you need these defines? In particular, isn't the compiler
defining __MINGW__ and _WINDOWS?

[Steven Edwards]

--- "Dimitrie O. Paun"  wrote:
> On January 10, 2003 12:01 am, Steven Edwards wrote:
> > - CFLAGS="=$CFLAGS -D__MINGW__ -D_WINDOWS -DWINE_NOWINSOCK" ;
>
> Why do you need these defines? In particular, isn't the compiler
> defining __MINGW__ and _WINDOWS?

I started using them a while back on one of the mingw releases and have just always kept them out
of habit. I thought it was supposed to define them but around the mingw-gcc 2.95.3 release It
didnt do one of them (_WINDOWS) I think. When I first started wine development and was playing
with the stuff Patrik did for MS_VC anything that both MS_VC and Mingw could use I always #ifdef'd
as _WINDOWS. I guess I should have checked to see if this was still needed so yeah its way wrong.

BRAIN FART.

Anyway disregard this anyway because I forgot to test this patch after my ./configure was done.
Now I'm ending up with

gcc -c -I. -I. -I../include -I../include  =-g -O2 -Wall -mpreferred-stack-bounda
ry=2 -gstabs+ -D__MINGW__ -D_WINDOWS -DWINE_NOWINSOCK  -D__WINESRC__ -D_REENTRAN
T -o cptable.o cptable.c
gcc.exe: cannot specify -o with -c or -S and multiple compilations

I really only need a patch that adds "-DWINE_NOWINSOCK" to the CFLAGS if Mingw isnt broken. If I
need anything else it can be added later.

Thanks
Steven

__________________________________________________
Do you Yahoo!?
Yahoo! Mail Plus - Powerful. Affordable. Sign up now.
http://mailplus.yahoo.com