On Mon, Jul 7, 2008 at 3:23 AM, Steph Fox <[EMAIL PROTECTED]> wrote:
> sfox            Mon Jul  7 01:23:56 2008 UTC
>
>  Modified files:              (Branch: PHP_5_3)
>    /php-src/win32/build        config.w32 confutils.js
>  Log:
>  - Fix up some bits and pieces.

For examples? Log message helps to see what has been changed, please be verbose.

>  - Head towards making it possible for all platforms to have compiler info in 
> phpinfo() output.

It is already in php info:

phpinfo()
PHP Version => 5.3.0-dev

System => Windows NT PIERREXP64 5.2 build 3790
Build Date => Jul  6 2008 11:50:59
Configure Command => cscript /nologo configure.js
"--enable-snapshot-build" "--with-php-build=..\deps"
"--with-gd=shared" "--disable-phar" "--with-mysqli" "--disable-apc"
"--without-maxdb"
Windows Compiler and Version => MSVC6

What is missing is the non windows platform part. As far as I remember
someone is already working on it and will add this info as well.


>  - Make it possible to switch off configure summary (--without-summary).

Talking about asking before committing, please do it next time (as I
do). The idea is the other way round, to drop the noise or get it if
enabled.

> http://cvs.php.net/viewvc.cgi/php-src/win32/build/config.w32?r1=1.40.2.8.2.10.2.20&r2=1.40.2.8.2.10.2.21&diff_format=u
> Index: php-src/win32/build/config.w32

> +var VC_VERSIONS = new Array();
> +VC_VERSIONS[1200] = 'MSVC6 (Visual C++ 6.0)';
> +VC_VERSIONS[1300] = 'MSVC7 (Visual C++ 2002)';
> +VC_VERSIONS[1310] = 'MSVC7.1 (Visual C++ 2003)';
> +VC_VERSIONS[1400] = 'MSVC8 (Visual C++ 2005)';
> +VC_VERSIONS[1500] = 'MSVC9 (Visual C++ 2008)';
> +
>  // Which version of the compiler do we have?
> -VCVERS = probe_binary(CL);
> -VCVERS = VCVERS.substr(0, 2) + VCVERS.substr(3, 2);
> -STDOUT.WriteLine("Detected MS compiler version " + VCVERS);
> -
> -// 1200 is VC6
> -// 1300 is vs.net 2002
> -// 1310 is vs.net 2003
> -// 1400 is vs.net 2005
> -// 1500 is vs.net 2008
> +VCVERS = probe_binary(CL).substr(0, 5).replace('.', '');
> +STDOUT.WriteLine("  Detected compiler " + VC_VERSIONS[VCVERS]);
> +AC_DEFINE('COMPILER', VC_VERSIONS[VCVERS], "Detected compiler version");

<snip>
> http://cvs.php.net/viewvc.cgi/php-src/win32/build/confutils.js?r1=1.60.2.1.2.8.2.19&r2=1.60.2.1.2.8.2.20&diff_format=u
> Index: php-src/win32/build/confutils.js
> diff -u php-src/win32/build/confutils.js:1.60.2.1.2.8.2.19 
> php-src/win32/build/confutils.js:1.60.2.1.2.8.2.20
> --- php-src/win32/build/confutils.js:1.60.2.1.2.8.2.19  Sat Jul  5 19:53:04 
> 2008
> +++ php-src/win32/build/confutils.js    Mon Jul  7 01:23:56 2008
> @@ -17,7 +17,7 @@
>   +----------------------------------------------------------------------+
>  */
>
> -// $Id: confutils.js,v 1.60.2.1.2.8.2.19 2008/07/05 19:53:04 pajoye Exp $
> +// $Id: confutils.js,v 1.60.2.1.2.8.2.20 2008/07/07 01:23:56 sfox Exp $
>
>  var STDOUT = WScript.StdOut;
>  var STDERR = WScript.StdErr;
> @@ -30,13 +30,6 @@
>  var extensions_enabled = new Array();
>  var sapi_enabled = new Array();
>
> -var VC_VERSIONS = new Array();
> -VC_VERSIONS[1200] = 'VC6';
> -VC_VERSIONS[1300] = 'Visual C++ 2002';
> -VC_VERSIONS[1310] = 'Visual C++ 2003';
> -VC_VERSIONS[1400] = 'Visual C++ 2005';
> -VC_VERSIONS[1500] = 'Visual C++ 2008';

Please leave the declaration here, it is in the top of the configure
script where other globals are declared.


Cheers,
-- 
Pierre

http://blog.thepimp.net | http://www.libgd.org

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to