running run-tests.php without make test can still cause some tests to fail (even ext/standard/tests/php_ini_loaded_file.phpt), as there will be no ini file loaded (at least this is why I'm seeing on linux) so adding the option to create the tmp-php.ini through run-tests.php would be useful for me.
On Tue, Sep 6, 2011 at 11:42 AM, Pierre Joye <pierre....@gmail.com> wrote: > it is generated by make test, not by run-tests.php, which is wrong as > it then fails when not called using run-tests.php. All testing > environment testing for regression will fail here as they don't call > make test but run-tests.php. run-tests.php needs an option to create > it then and it should be done only there. > > On Tue, Sep 6, 2011 at 11:32 AM, Hannes Magnusson > <hannes.magnus...@gmail.com> wrote: >> tmp-php.ini is generated by the test suite. How is that not portable >> and why would that change? >> >> -Hannes >> >> On Tue, Sep 6, 2011 at 11:30, Pierre Joye <pierre....@gmail.com> wrote: >>> no, as it is not portable and can change. >>> >>> On Tue, Sep 6, 2011 at 11:25 AM, Hannes Magnusson >>> <hannes.magnus...@gmail.com> wrote: >>>> On Tue, Sep 6, 2011 at 09:47, Pierre Joye <paj...@php.net> wrote: >>>>> pajoye Tue, 06 Sep 2011 07:47:20 +0000 >>>>> >>>>> Revision: http://svn.php.net/viewvc?view=revision&revision=316229 >>>>> >>>>> Log: >>>>> - fix test >>>>> >>>>> Changed paths: >>>>> U >>>>> php/php-src/branches/PHP_5_3/ext/standard/tests/php_ini_loaded_file.phpt >>>>> U >>>>> php/php-src/branches/PHP_5_4/ext/standard/tests/php_ini_loaded_file.phpt >>>>> U php/php-src/trunk/ext/standard/tests/php_ini_loaded_file.phpt >>>>> >>>>> Modified: >>>>> php/php-src/branches/PHP_5_3/ext/standard/tests/php_ini_loaded_file.phpt >>>>> =================================================================== >>>>> --- >>>>> php/php-src/branches/PHP_5_3/ext/standard/tests/php_ini_loaded_file.phpt >>>>> 2011-09-06 07:44:43 UTC (rev 316228) >>>>> +++ >>>>> php/php-src/branches/PHP_5_3/ext/standard/tests/php_ini_loaded_file.phpt >>>>> 2011-09-06 07:47:20 UTC (rev 316229) >>>>> @@ -11,4 +11,4 @@ >>>>> var_dump(php_ini_loaded_file()); >>>>> ?> >>>>> --EXPECTF-- >>>>> -string(%d) "%s/tmp-php.ini" >>>>> +string(%d) "%sphp.ini" >>>> >>>> The tmp- part here is quite intentional, please leave it in. >>>> >>>> -Hanes >>>> >>>> -- >>>> PHP CVS Mailing List (http://www.php.net/) >>>> To unsubscribe, visit: http://www.php.net/unsub.php >>>> >>>> >>> >>> >>> >>> -- >>> Pierre >>> >>> @pierrejoye | http://blog.thepimp.net | http://www.libgd.org >>> >> > > > > -- > Pierre > > @pierrejoye | 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 > > -- Ferenc Kovács @Tyr43l - http://tyrael.hu -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
