From 4375c3fd7a8c77267b10ccfc2038c30ffd182f3c Mon Sep 17 00:00:00 2001 From: Wez Furlong Date: Fri, 12 Aug 2005 20:05:19 +0000 Subject: [PATCH] /GB option is the default; remove it. Add compiler version checks, as vs.net 2005 has dropped support for /YX and /GB options, and deprecated ANSI stdio (in favour of more secure, non-standard versions). --- win32/build/config.w32 | 37 ++++++++++++++++++++++++++++++++++--- 1 file changed, 34 insertions(+), 3 deletions(-) diff --git a/win32/build/config.w32 b/win32/build/config.w32 index ec36cc60bc..dece210b82 100644 --- a/win32/build/config.w32 +++ b/win32/build/config.w32 @@ -4,7 +4,27 @@ // equivalent. ARG_WITH('cygwin', 'Path to cygwin utilities on your system', '\\cygwin'); -PATH_PROG('cl'); +CL = PATH_PROG('cl'); +if (!CL) { + ERROR("MS C++ compiler is required"); +} +// Which version of the compiler do we have ? +function probe_msvc_compiler_version(CL) +{ + // tricky escapes to get stderr redirection to work + var banner = execute('cmd /c ""' + CL + '" 2>&1"'); + if (banner.match(/(\d+)\.(\d+)\.(\d+)(\.(\d+))?/)) { + return RegExp.$1; + } + return 0; +} + +VCVERS = probe_msvc_compiler_version(CL); +STDOUT.WriteLine("Detected MS compiler version " + VCVERS); + +// 12 is VC6 +// 13 is vs.net 2003 +// 14 is vs.net 2005 // cygwin now ships with link.exe. Avoid searching the cygwin path // for this, as we want the MS linker, not the fileutil @@ -61,9 +81,20 @@ DEFINE("CFLAGS_PHP", "/D _USRDLL /D PHP5DLLTS_EXPORTS /D PHP_EXPORTS \ DEFINE('CFLAGS_PHP_OBJ', '$(CFLAGS_PHP) $(STATIC_EXT_CFLAGS)'); // General CFLAGS for building objects -DEFINE("CFLAGS", "/nologo /YX /FD $(BASE_INCLUDES) /D _WINDOWS \ +DEFINE("CFLAGS", "/nologo /FD $(BASE_INCLUDES) /D _WINDOWS \ /D ZEND_WIN32=1 /D PHP_WIN32=1 /D WIN32 /D _MBCS"); +if (VCVERS < 14) { + // Enable automatic precompiled headers + ADD_FLAG('CFLAGS', ' /YX '); +} + +if (VCVERS >= 14) { + // fun stuff: MS deprecated ANSI stdio and similar functions + // disable annoying warnings + ADD_FLAG('CFLAGS', ' /wd4996 '); +} + // General link flags DEFINE("LDFLAGS", "/nologo /version:" + PHP_VERSION + "." + PHP_MINOR_VERSION + "." + PHP_RELEASE_VERSION); @@ -102,7 +133,7 @@ if (PHP_DEBUG == "yes") { } // Equivalent to Release_TSInline build -> best optimization ADD_FLAG("CFLAGS", "/LD /MD /W3 /Ox /D NDebug /D NDEBUG \ -/D ZEND_WIN32_FORCE_INLINE /GB /GF /D ZEND_DEBUG=0"); +/D ZEND_WIN32_FORCE_INLINE /GF /D ZEND_DEBUG=0"); // if you have VS.Net /GS hardens the binary against buffer overruns // ADD_FLAG("CFLAGS", "/GS"); } -- 2.50.1