Re: 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
| From: | Johannes Schlüter | Date: | Mon, 20 Jun 2011 22:28:15 +0000 |
| Subject: | Re: 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 |
||
| References: | 1 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-65299@lists.php.net to get a copy of this message | ||
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