#25000 [Opn->Csd]: ext/mbstring/tests/php_gr_jp_16242.phpt uses $_ENV instead of getenv
ID: 25000
Updated by: iliaa@php.net
Reported By: webmaster at ragnarokonline dot de
-Status: Open
+Status: Closed
Bug Type: *Compile Issues
PHP Version: 4CVS-2003-08-09 (stable)
New Comment:
This bug has been fixed in CVS.
In case this was a PHP problem, snapshots of the sources are packaged
every three hours; this change will be in the next snapshot. You can
grab the snapshot at http://snaps.php.net/.
In case this was a documentation problem, the fix will show up soon at
http://www.php.net/manual/.
In case this was a PHP.net website problem, the change will show
up on the PHP.net site and on the mirror sites in short time.
Thank you for the report, and for helping us make PHP better.
Previous Comments:
------------------------------------------------------------------------
[2003-08-09 15:47:42] webmaster at ragnarokonline dot de
ugh, typos
site=side
dump=package
sorry for the spam
------------------------------------------------------------------------
[2003-08-09 15:44:53] webmaster at ragnarokonline dot de
> Congratulation 25000 entry! :)
Thanks ^_^
> Seriously, what difference do you think there is between
> $_ENV["TEST_PHP_EXECUTABLE"] and
> getenv("TEST_PHP_EXECUTABLE") as that
> variable is supposed to be populated before the test-run..
> Works fine here.
Well, the whole array is empty on my site (running SuSE 8.2) and if you
search for 'TEST_PHP_EXECUTABLE' though all phpt-files in the dump, all
resulting testcases use getenv instead of $_ENV
------------------------------------------------------------------------
[2003-08-09 15:32:57] moriyoshi@php.net
Congratulation 25000 entry! :)
Seriously, what difference do you think there is between
$_ENV["TEST_PHP_EXECUTABLE"] and getenv("TEST_PHP_EXECUTABLE") as that
variable is supposed to be populated before the test-run.. Works fine
here.
------------------------------------------------------------------------
[2003-08-09 15:27:04] webmaster at ragnarokonline dot de
Description:
------------
The new testcase located at ext/mbstring/tests/php_gr_jp_16242.phpt
fails, because $_ENV["TEST_PHP_EXECUTABLE"] is used instead of
getenv("TEST_PHP_EXECUTABLE")
To run this test, execute
TEST_PHP_EXECUTABLE=sapi/cli/php \
sapi/cli/php -n run-tests.php \
ext/mbstring/tests/php_gr_jp_16242.phpt
after compiling PHP
The fix is simple:
in ext/mbstring/tests/php_gr_jp_16242.phpt REPLACE
$cmd .= ' ?>" |'.$_ENV["TEST_PHP_EXECUTABLE"].' -c '.$tmpfile;
WITH
$cmd .= ' ?>" |'.getenv("TEST_PHP_EXECUTABLE").' -c
'.$tmpfile;
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=25000&edit=1
Thread (5 messages)