Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:102891
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.223.171 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAJHtznyR9u7i97JY=A6ZOmvis08SSbOzkxrn5FdZn7BUm-Le_w@mail.gmail.com>
References: <BY1PR02MB1260706DBB8F5E3A36A2AEF2BF580@BY1PR02MB1260.namprd02.prod.outlook.com>
 <CAF+90c_BGHBmUr1MjSXhJGhc8zJGQ67ynsn4QJhcsxo6iDWzNQ@mail.gmail.com>
 <BL0PR02MB4435EBFFEAE3BC137224BE7FAC580@BL0PR02MB4435.namprd02.prod.outlook.com>
 <27a9854c-d779-366a-5cc8-70722487139c@fedoraproject.org> <CAESVnVpvvSkevpBHdza34fnxK23ZvMfD25-o4cNTtduNAEQPig@mail.gmail.com>
 <CAJHtznyR9u7i97JY=A6ZOmvis08SSbOzkxrn5FdZn7BUm-Le_w@mail.gmail.com>
Date: Tue, 17 Jul 2018 17:05:22 +0200
Message-ID: <CAF+90c9=a+4J7UxDWPjJKE8E0RPixfy3JLPpOdeWVZi-7yUMnA@mail.gmail.com>
To: Zeev Suraski <vsuraski@gmail.com>
Cc: Sara Golemon <pollita@php.net>, Remi Collet <remi@fedoraproject.org>, 
	Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="0000000000007775a705713346df"
Subject: Re: [PHP-DEV] re2c version(s)
From: nikita.ppv@gmail.com (Nikita Popov)

--0000000000007775a705713346df
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, Jul 17, 2018 at 1:51 PM, Zeev Suraski <vsuraski@gmail.com> wrote:

> On Tue, Jul 17, 2018 at 2:01 PM Sara Golemon <pollita@php.net> wrote:
>
> > On Tue, Jul 17, 2018 at 1:04 AM, Remi Collet <remi@fedoraproject.org>
> > wrote:
> > > Le 13/07/2018 =C3=A0 23:48, Zeev Suraski a =C3=A9crit :
> > > Perhaps we can also add all the generated files (including configure)
> in
> > > the tagged versions, so the tag will have same content than the
> official
> > > archive.
> > >
> > Ick, no.  That's the worst outcome IMO.  I don't think they need to be
> > in git, but if we're going to have them there during tags, then they
> > should always be there.  One way or the other, not some
> > middle-of-the-road thing.
>
>
> I can explain why I think that what I propose is the best outcome:
> - It ensures that the correct versions of re2c are always used.
> - It doesn't track generated files in source control.
> - It allows users to build PHP from source on platforms where re2c is
> unavailable.
>
> Correct me if I'm wrong, but isn't that exactly what we do with
> zend_language_parser.c?  makedist is responsible for generating it so tha=
t
> it's available in distributions, but it isn't tracked.  Why would
> zend_language_scanner.c be any different?  My guess is that it's probably
> because at the time we moved to re2c it wasn't nearly as ubiquitous as it
> is today and even most developers didn't have access to it, but now that'=
s
> changed.
>
> I'm also fine with what Remi proposed which is adding these files
> specifically to the source control at the time of tagging, in the spirit =
of
> tracking everything that we actually end up releasing in source control
> (and perhaps do that for zend_language_parser.c if we decide that this is
> the right thing to do).
>
> Either way - the first step (normalizing re2c versions - updating our r2e=
c
> requirements) seems to make sense.  We can decide about whether or not we
> track the generated files in git as we do today, only during tagging or n=
ot
> at all independently of that.
>
> Zeev
>

I feel like we are all really in violent agreement that these files should
be dropped from git, and at this point I'm not even sure what the
discussion is about anymore. Let's wait until after PHP-7.3 branching in
two weeks and drop them at that point.

Normalizing the version numbers seems unnecessary after they are dropped --
at least Dmitry's original motivation for that was related exclusively to
the spurious diffs caused by different versions, which will no longer be an
issue.

Nikita

--0000000000007775a705713346df--