Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:126419
Message-ID: <51e86950-d88c-4913-96b9-af4119e620b8@gmail.com>
Date: Sun, 16 Feb 2025 00:20:37 +0100
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PHP-DEV] CLI/CGI -z command line options is broken
To: internals@lists.php.net
References: <298861df-b65f-4708-8a26-a377fd69e319@gmail.com>
 <5478e83c-e2ca-44e1-a93d-43a559cbe0be@app.fastmail.com>
Content-Language: en-US
In-Reply-To: <5478e83c-e2ca-44e1-a93d-43a559cbe0be@app.fastmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
From: dossche.niels@gmail.com (Niels Dossche)

On 2/10/25 11:14 PM, Larry Garfield wrote:
> On Mon, Feb 10, 2025, at 3:24 PM, Niels Dossche wrote:
>> Hi internals
>>
>> We received a bug report (https://github.com/php/php-src/issues/17740) 
>> that the `-z` command line option for CLI/CGI doesn't actually work.
>> This option is supposed to load a zend extension using a command line 
>> argument.
>> However, it turns out that the extension doesn't actually load: PHP 
>> will claim that the extension's loaded (in some places), but in reality 
>> it never loaded properly.
>> I did an analysis of why this is happening in that GitHub thread, and a 
>> clean solution isn't straight-forward.
>> Christoph figured out that this option has been broken since at least 
>> PHP 5.5, a long time.
>> As passing `-d zend_extension=foo` on the command line is already a 
>> suitable replacement, we'd like to just remove the `-z` option in PHP 
>> 8.5 instead of fixing it.
>>
>> Are there any objections against this? Please let us know.
>>
>> Kind regards
>> Niels
> 
> Seems reasonable to me.  Perhaps update the -d help text accordingly, if needed?
> 
> --Larry Garfield

Hi

Just to clarify: the -d option just sets an ini option.
One idea could be to make -z print out a message telling the user to use "-d zend_extension=...". That would make the transition more "gradual".
OTOH this didn't work for ages so nobody realistically relies on this option.

Kind regards
Niels