Re: Re: cvs: php-src /ext/session/tests 020.phpt

hi Stas, On Mon, Sep 8, 2008 at 7:25 PM, Stanislav Malyshev <stas@zend.com> wrote: > Hi! > >> Just use single quotes then.. where the hell isn't " portable? Windows? >> What in this isn't portable??? > > AFAIK single quotes don't work with tests in --INI-- now. What happens is > that runtest adds some quoting (didn't have time to check it yet) that makes > the resulting value have single quotes inside - i.e. \'foo\' instead of foo > when using something='foo'. Double quotes worked fine for me, and single > quotes did in regular ini file, but not when ini settings are set by tests. As I said earlier, the problem is the cmd line call, not php.ini, not the new parser or something else :-) The easiest way is to drop the double quotes in --INI-- and use escapeshellargs in run-tests.php. The only reason why I did not commit the patch yet is that I did not have the time to fix the tests, help welcome :) Cheers, -- Pierre http://blog.thepimp.net | http://www.libgd.org