I've gone through some testing with my app, as well as some others
using wxCurl. The whole winsock2 thing is becomming an issue. Even
defining _WINSOCKAPI_ in the derivative apps causes issues. Weird
compile errors and such for anyone using Win32.

I think if it isn't too big a deal, we should go back to Guenter's
suggestion of:

--- multi.h.orig Wed Mar 24 23:53:42 2004
+++ multi.h Fri Sep 03 19:26:22 2004
@@ -52,7 +52,9 @@
#endif

#if defined(WIN32) && !defined(__GNUC__) || defined(__MINGW32__)
+#if !(defined(_WINSOCKAPI_) || defined(_WINSOCK_H))
#include <winsock2.h>
+#endif
#else

/* HP-UX systems version 9, 10 and 11 lack sys/select.h and so does oldish

Which seemed to work for me and for others...I don't know how well it
fits into the larger CURL app though. I'd like to get this fixed
though, because the Win32 folks are big wxWidgets users, and right now
it's toughest for them.

Thanks for the help.

Cheers.
Casey

On Mon, 20 Sep 2004 14:34:01 -0400, Casey ODonnell
<caseyodonnell_at_gmail.com> wrote:
> I added the:
>
> /D "_WINSOCKAPI_" to my CFLAGs, which seems to work...unfortunately I get:
> warning C4005: '_WINSOCKAPI_' : macro redefinition
>
> All over the place.
>
> Compiles and runs fine though. I'm ok with the warnings...but a lot
> of people get nervous about that kind of stuff.
>
> CKO
>
>
>
> On Sun, 5 Sep 2004 12:36:14 -0400, Casey ODonnell
> <caseyodonnell_at_gmail.com> wrote:
> > I have not tried this method yet. I'll try it on Monday and let you
> > know the results. Certainly something that is easy, and I can
> > document if it does indeed fix the problem.
> >
> > Casey
> >
> >
> >
> >
> > On Fri, 3 Sep 2004 19:32:33 +0200, Guenter Knauf <eflash_at_gmx.net> wrote:
> > > Hi Casey,
> > > have you also tried what Gisle suggested below?
> > > I've somehow missed to read Gisle's post until now, but I think that's a better approach than my suggestion since you dont have to define an additional var - if that works for you...
> > >
> > > Guenter.
> > >
> > > --- multi.h.orig Wed Mar 24 23:53:42 2004
> > > +++ multi.h Fri Sep 03 19:26:22 2004
> > > @@ -52,7 +52,9 @@
> > > #endif
> > >
> > > #if defined(WIN32) && !defined(__GNUC__) || defined(__MINGW32__)
> > > +#if !(defined(_WINSOCKAPI_) || defined(_WINSOCK_H))
> > > #include <winsock2.h>
> > > +#endif
> > > #else
> > >
> > > /* HP-UX systems version 9, 10 and 11 lack sys/select.h and so does oldish
> > >
> > >
> > > > "Casey ODonnell" <caseyodonnell_at_gmail.com> said:
> > >
> > > >> Let me know what your thoughts are, and if there are good ways to
> > > >> reconcile Winsock and Winsock2...
> > >
> > > > Carefully reorder your headers and/or define _WINSOCKAPI_ in
> > > > your CFLAGS.
> > >
> > > > MingW's <winsock2.h> has this at top:
> > > > #define _WINSOCK_H /* to prevent later inclusion of winsock.h */
> > >
> > > > Microsoft in their SDK where not so smart to foresee this problem.
> > > > <multi.h> could probably do something like:
> > >
> > > > #if !defined(_WINSOCKAPI_) && !defined(_WINSOCK_H)
> > > > #include <winsock2.h>
> > > > #endif
> > >
> > > > Do you provide a patch?
> > >
> > > > But, winsock2.h is only needed for the definition of fd_set.
> > > > So those args could just as well be 'void *' to avoid this problem.
> > > > Anything can be converted to 'void *' without a cast in C, right?
> > >
> > > > --gv
> > >
> > >
> >
> >
> > --
> > Casey O'Donnell
> > RPI STS Department - Graduate Student
> >
> > http://homepage.mac.com/codonnell/
> > http://homepage.mac.com/codonnell/wxsync/
> >
> >
> > http://homepage.mac.com/codonnell/wxblogger/
> >
>
>
>
>
> --
> Casey O'Donnell
> RPI STS Department - Graduate Student
>
> http://homepage.mac.com/codonnell/
> http://homepage.mac.com/codonnell/wxsync/
> http://homepage.mac.com/codonnell/wxblogger/
>

-- 
Casey O'Donnell
RPI STS Department - Graduate Student
http://homepage.mac.com/codonnell/
http://homepage.mac.com/codonnell/wxsync/
http://homepage.mac.com/codonnell/wxblogger/