Dmitry,

I am sorry to say but I think this patch as a tad pre-mature, as a consequence of this patch a number of tests break. The problem is that INI parser disallows usage of a whole range of characters such as ; & =, etc... without being encapsulated inside double quotes. Something the old php-cli parser allowed because it was not using the parse_ini() function. We "could" fix it by relaxing the INI scanner, but I am not sure of the consequences it would have on ini file parsing. I think at the very minimum we need to restore the old style php-cli INI parsing mechanism.


On 19-Sep-06, at 4:33 PM, Dmitry Stogov wrote:

dmitry          Tue Sep 19 20:33:12 2006 UTC

  Modified files:              (Branch: PHP_5_2)
    /php-src/main       SAPI.h SAPI.c php_ini.c
    /php-src    NEWS
    /ZendEngine2        zend_ini.h zend_ini_parser.y zend_ini_scanner.h
                        zend_ini_scanner.l
    /php-src/sapi/cgi   cgi_main.c
    /php-src/sapi/cli   php_cli.c
  Log:
MFH: Fixed mess with CGI/CLI -d command line option (now it works with cgi; constants are working exactly like in php.ini; with FastCGI -d affects all requests).

<dmitry-20060919203312.txt>
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Ilia Alshanetsky

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

Reply via email to