#45698 [WFx->Fbk]: Doesn't compile with VC6 - Include problem

ID: 45698 Updated by: pajoye@php.net Reported By: Keisial at gmail dot com -Status: Wont fix +Status: Feedback Bug Type: Compile Failure Operating System: Windows - VC6 PHP Version: 5.3CVS-2008-08-03 (snap) Assigned To: pajoye New Comment: I finally got some more time to investigate. Can you try the following: - open a normal console - "c:\program files\Microsoft Visual Studio\VC98\bin\vcvars32.bat" - Do not register the SDK or VS vars, type "export<enter>" to be sure that no LIB or INCLUDE are defined (may be set to VC98 include or lib) Then run: configure.bat --with-extra-includes=c:\path\to\sdk200302\include --with-extra-libs=c:\path\to\sdk200302\lib and finally nmake. It may fix the problem as the correct winsock2 will be included. The important thing to remember is that winsock2.h is a special case, it has to be included before windows.h (msdn or google about it). Previous Comments: ------------------------------------------------------------------------ [2008-08-03 13:35:50] pajoye@php.net Err no. Please do not commit this patch as it does break VC6 builds now. Please use the following compilers SDK pairs: http://wiki.php.net/internals/windows/compiler http://wiki.php.net/internals/windows/windowssdk This is what we use for http://snaps.php.net ------------------------------------------------------------------------ [2008-08-03 13:25:31] pajoye@php.net Stas, can you commit the patch please? ------------------------------------------------------------------------ [2008-08-03 12:27:13] derick@php.net This will probably fix the compile failure that Steph and I were getting... ------------------------------------------------------------------------ [2008-08-03 12:23:45] Keisial at gmail dot com Description: ------------ 5.3 doesn't compile with VC6. When including winsock2.h at zend_config.w32.h it includes windows.h which in turn includes winsock stuff before a SOCKET is defined. Compilation fails with error on mswsock.h (This doesn't happen including only those two headers, so some defines are modifying the defualt behaviour) Changing the order to include windows.h before winsock2.h fixes it: --- zend_config.w32.h 2008-07-17 20:33:46.000000000 +0200 +++ zend_config.w32.h 2008-08-03 14:19:56.593750000 +0200 @@ -35,8 +35,8 @@ #ifndef ZEND_INCLUDE_FULL_WINDOWS_HEADERS #define WIN32_LEAN_AND_MEAN #endif -#include <winsock2.h> #include <windows.h> +#include <winsock2.h> #include <float.h> ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=45698&edit=1