Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:102907
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: softfail (pb1.pair.com: domain golemon.com does not designate 209.85.208.48 as permitted sender)
MIME-Version: 1.0
Sender: php@golemon.com
In-Reply-To: <CAJHtznwP35HZBbuv-4gmw4kjrQD6LvqPFKW0jPtVzA8GyhZbZg@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>
 <CAF+90c9=a+4J7UxDWPjJKE8E0RPixfy3JLPpOdeWVZi-7yUMnA@mail.gmail.com>
 <CAJHtznyeGxy9QQzjPRoqFdhT2xxrtosP9dgQQu63xn0gc4wxfg@mail.gmail.com>
 <CAF+90c-EZ-N+GLmXP5OO19zKVtmiZvxiv_vgaY=iiho1mFa7fw@mail.gmail.com> <CAJHtznwP35HZBbuv-4gmw4kjrQD6LvqPFKW0jPtVzA8GyhZbZg@mail.gmail.com>
Date: Tue, 17 Jul 2018 21:53:24 -0400
Message-ID: <CAESVnVp8JSmioJzY2LdkP8saPTmZ2xAi17p=cUV-jYsriTqY=g@mail.gmail.com>
To: Zeev Suraski <vsuraski@gmail.com>
Cc: Nikita Popov <nikita.ppv@gmail.com>, Remi Collet <remi@fedoraproject.org>, 
	Internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [PHP-DEV] re2c version(s)
From: pollita@php.net (Sara Golemon)

On Tue, Jul 17, 2018 at 1:06 PM, Zeev Suraski <vsuraski@gmail.com> wrote:
> You know, you made me go back to Sara's email where she disagreed with me,
> only to find she actually was disagreeing with Remi's proposal to track the
> generated files for releases in git.  That's definitely not a hill to die on
> for me :)
>
Correct. I'm in agreement with you that we DO include the generated
files in tarballs (using a standardized version of re2c, or at least,
a consistent-per-branch version), but that we do NOT ever check those
generated files into git.

> So all in all I think you're right, we agree on the important things:
> 1. Remove generated files from git
> 2. Keep them in source packages
>
+1

> What we seem to disagree on is that we should have a narrower list of
> acceptable re2c versions determined by configure/makedist.  This isn't a
> hill to die on for me either, although I think that narrowing it down is
> better in terms of our ability to deliver a source package with confidence,
> and be sure that everyone who's testing/using it is testing the same thing
> (similar to how we do it with bison).  Arguably it's more important to be on
> the safe wide with makedist than it is with configure.
>
I think devs should be able to use flexible versions of re2c (and
other tools, e.g. bison), BUT that we should declare formally what
versions of these build tools will be used on what branches so that
those working on features can predictably know what their changes will
generate.

For example, my builder (which Remi and I both use for 7.2 builds)
currently uses debian:jessie (and therefore re2c 0.13.5 and bison
3.0.2).  Barring any pressing need, we'll plan to keep them at these
versions until 7.2 goes EOL.

-Sara