Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/sapi/cli/config.m4 branches/PHP_5_4/sapi/cli/config.w32 branches/PHP_5_4/sapi/cli/php_cli.c branches/PHP_5_4/sapi/cli/php_cli_server.c branches/PHP_5_4/sapi/cli/php_cli_server.h branches/PHP_5_4/sapi/cli/php_http_parser.c branches/PHP_5_4/sapi/cli/php_http_parser.h trunk/sapi/cli/config.m4 trunk/sapi/cli/config.w32 trunk/sapi/cli/php_cli.c trunk/sapi/cli/php_cli_server.c trunk/sapi/cli/php_cli_server.h trunk/sapi/cli/php_http_parser.c trunk/sapi/cli/php_http_p

[Johannes Schlüter]

Hi,

On Mon, 2011-06-20 at 20:27 +0000, Moriyoshi Koizumi wrote:
> moriyoshi                                Mon, 20 Jun 2011 20:27:39 +0000
> 
> Revision: http://svn.php.net/viewvc?view=revision&revision=312344
> 
> Log:
> - Add built-in web server to CLI SAPI. See the RFC for detail.

This broke the -d option. Causing this error on make:

| Generating phar.phar
| PEAR package PHP_Archive not installed: generated phar will require
| PHP's phar extension be enabled.
| Creating phar files is disabled by ini setting 'phar.readonly'.
| make: *** [ext/phar/phar.phar] Error 1

which can be reduced to this simple test case:

$ sapi/cli/php -n -d phar.readonly=0 -r 'echo ini_get("phar.readonly");'
1

I see that php_cli.c now handles parsing -d options before calling
sapi_startup() whic seems to cause this.


johannes



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