Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:102797
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.214.54 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <BY1PR02MB1260706DBB8F5E3A36A2AEF2BF580@BY1PR02MB1260.namprd02.prod.outlook.com>
References: <BY1PR02MB1260706DBB8F5E3A36A2AEF2BF580@BY1PR02MB1260.namprd02.prod.outlook.com>
Date: Fri, 13 Jul 2018 11:26:22 +0200
Message-ID: <CAF+90c_BGHBmUr1MjSXhJGhc8zJGQ67ynsn4QJhcsxo6iDWzNQ@mail.gmail.com>
To: Dmitry Stogov <dmitry@zend.com>
Cc: PHP internals list <internals@lists.php.net>, Stanislav Malyshev <smalyshev@gmail.com>, 
	"derick@derickrethans.nl" <derick@derickrethans.nl>, "Christoph M. Becker" <cmbecker69@gmx.de>
Content-Type: multipart/alternative; boundary="000000000000b845190570de1296"
Subject: Re: [PHP-DEV] re2c version(s)
From: nikita.ppv@gmail.com (Nikita Popov)

--000000000000b845190570de1296
Content-Type: text/plain; charset="UTF-8"

On Fri, Jul 13, 2018 at 11:13 AM, Dmitry Stogov <dmitry@zend.com> wrote:

> Hi,
>
>
> I think, many core developers saw unexpected changes in
> "zend_labguages_scanner.c" or "var_unserializer.c" after rebuilds.
>
> This occurs, because we use different versions of re2c, and some of them
> produce really different code.
>
> They also embed version number into the generate source. Currently
> different files in PHP source tree generated by different re2c versions:
>
>
> ext/json/json_scanner.c 0.16
>
> ext/date/lib/parse_date.c 0.15.3
>
> ext/date/lib/parse_iso_intervals.c 0.15.3
>
> ext/pdo/pdo_sql_parser.c 0.16
>
> ext/phar/phar_path_check.c 1.0.3
>
> sapi/phpdbg/phpdbg_lexer.c 0.16
>
> ext/standard/url_scanner_ex.c 0.16
>
> ext/standard/var_unserializer.c 1.0.1
>
> Zend/zend_ini_scanner.c 0.15
>
> Zend/zend_language_scanner.c 1.0.1
>
>
> I propose, to change build scripts (in master and PHP-7.3) to require at
> least re2c version 1.0.0 (it seems 1.0.0-1.0.3 produce the same result) and
> suppress version output into the generated files.
>
>
> I'm not sure about timelib files.
>
>
> Thanks. Dmitry.
>

I don't think normalizing the version really solves anything. These files
should be dropped from version control entirely instead. Generated files do
not belong in version control.

re2c is widely available on Linux distros nowadays (probably specifically
because PHP uses it) and while there might have been historical ground to
bundle these generated files, there no longer is one.

Nikita

--000000000000b845190570de1296--