Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:74445
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.128.173 as permitted sender)
MIME-Version: 1.0
Sender: julienpauli@gmail.com
In-Reply-To: <CAEZPtU7p-RiamfkdmBGGg2ADpqbsmev_SBxT7t60dJ7xWN+5AA@mail.gmail.com>
References: <2DC459EE-AE98-4CAE-977A-6FB918FDEF54@bafford.com>
 <537EA42B.4000000@lerdorf.com> <B20CF9D3-B61C-462E-825A-DA421F351694@bafford.com>
 <CAEZPtU7p-RiamfkdmBGGg2ADpqbsmev_SBxT7t60dJ7xWN+5AA@mail.gmail.com>
Date: Fri, 23 May 2014 13:58:15 +0200
Message-ID: <CAMUwpuQ+yy5ZJx_wEgt3S_ZRG2rBOAOciz7YMdYw97Gbqhud9g@mail.gmail.com>
To: Pierre Joye <pierre.php@gmail.com>
Cc: John Bafford <john@bafford.com>, Rasmus Lerdorf <rasmus@lerdorf.com>, 
	PHP internals <internals@lists.php.net>, Ferenc Kovacs <tyrael@php.net>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] Bison 3 support for PHP 5.5
From: jpauli@php.net (Julien Pauli)

On Fri, May 23, 2014 at 8:40 AM, Pierre Joye <pierre.php@gmail.com> wrote:
> Hi John,
>
> On Fri, May 23, 2014 at 3:44 AM, John Bafford <john@bafford.com> wrote:
>> Hi Rasmus,
>>
>> On May 22, 2014, at 20:28, Rasmus Lerdorf <rasmus@lerdorf.com> wrote:
>>
>>> On 5/22/14, 6:01 PM, John Bafford wrote:
>>>> Hi,
>>>>
>>>> I=E2=80=99ve submitted a PR (https://github.com/php/php-src/pull/683) =
to add support for Bison 3.0 to PHP 5.5.
>>>>
>>>> I=E2=80=99ve run the tests on this patch, with no unexpected errors, w=
ith Bison 2.4 (minimum version for PHP 5.5 and 5.6), 2.7, and 3.0, with and=
 without --enable-maintainer-zts against the current PHP-5.5 and (with mino=
r changes) master branches.
>>>
>>> Could you explain your changes a bit? You removed %pure-parser, for
>>> example, and you shuffled globals around. This looks like more than jus=
t
>>> adding support for Bison 3.
>>>
>>> -Rasmus
>>
>> I replaced %pure-parser (which was deprecated) with %pure_parser (its re=
placement). (Also, YYERROR_VERBOSE, which also was deprecated, with %error-=
verbose.)
>>
>> Most of the changes in the PR revolve around this change in Zend/zend_la=
nguage_parser.y:
>> -%code requires {
>> -#ifdef ZTS
>> -# define YYPARSE_PARAM tsrm_ls
>> -# define YYLEX_PARAM tsrm_ls
>> -#endif
>> -}
>> +%parse-param { void *tsrm_ls }
>> +%lex-param { void *tsrm_ls }
>>
>> YYPARSE_PARAM and YYLEX_PARAM were deprecated, and then removed, and rep=
laced with %parse-param and %lex-param. However, you cannot (or I did not s=
ee a way to) conditionally define parameters, as PHP was previously doing h=
ere.
>>
>> So instead, I created the TSRMLS_DN and TSRMLS_CN macros which, in the Z=
TS case, work like TSRMLS_D/TSRMLS_C, but when ZTS is off, still resolve to=
 a parameter (that must always be present), which is passed NULL.
>
> Thanks for your work, always good to support less archaic version of biso=
n.
>
> Some comments:
>
> - does it still work with previous versions? If yes, do you know which
> or which minimum version we should then require?
> - 5.5 is stable, I am not sure it is a good idea to do these changes
> there. 5.6 may be a good candidate (but has to be done quickly, RMs
> can confirm if it is still possible)

That's a no-go for 5.5.
5.5 is stable, 1year old (at this email date), and we should not
change such a thing into this stable branch.

For 5.6, we are actually in RC stage (with first RC to come soon).
Seeing the comments saying that it's all compatible and tests have
passed, I'm not against requiring bison 3.x for PHP 5.6.
Waiting for Ferenc's answer ;-)

Thank you for your patch and your support to make PHP better.

Julien Pauli