#25000 [Fbk->Opn]: ext/mbstring/tests/php_gr_jp_16242.phpt uses $_ENV instead of getenv

ID: 25000 User updated by: webmaster at ragnarokonline dot de Reported By: webmaster at ragnarokonline dot de -Status: Feedback +Status: Open Bug Type: *Compile Issues PHP Version: 4CVS-2003-08-09 (stable) New Comment: > 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 Previous Comments: ------------------------------------------------------------------------ [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