Hello Jani,

Monday, July 21, 2008, 6:08:15 PM, you wrote:

> http://cvs.php.net/viewvc.cgi/php-src/run-tests.php?r1=1.226.2.37.2.35.2.34&r2=1.226.2.37.2.35.2.35&diff_format=u
> Index: php-src/run-tests.php
> diff -u php-src/run-tests.php:1.226.2.37.2.35.2.34
> php-src/run-tests.php:1.226.2.37.2.35.2.35
> --- php-src/run-tests.php:1.226.2.37.2.35.2.34  Mon Jul 21 11:05:23 2008
> +++ php-src/run-tests.php       Mon Jul 21 16:08:15 2008
> @@ -857,16 +863,16 @@
>                 $failed_tests_data .= "Bundled Libtool:\n$libtool\n";
>                 $failed_tests_data .= "System Libtool:\n$sys_libtool\n";
>                 $failed_tests_data .= "Compiler:\n$compiler\n";
> -               $failed_tests_data .= "Bison:\n". @shell_exec('bison 
> --version 2>/dev/null'). "\n";
> +               $failed_tests_data .= "Bison:\n". shell_exec('bison --version 
> 2>/dev/null') . "\n";

Seeing this again, is it really a good idea to list those? Shouldn't we
instead check whether we have the Makefile for the php we test and show the
versions from the respective programs specified in it instead?

Other than that, thanks for cleaning this up. I still thing we should
rewrite larger chunks of it though :-)

Best regards,
 Marcus


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

Reply via email to