Re: Remove useless associativity/precedence from parsers - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: Remove useless associativity/precedence from parsers
Date
Msg-id 30a4f5ff-dd4c-71b4-2708-b213224b76cc@2ndQuadrant.com
Whole thread Raw
In response to Re: Remove useless associativity/precedence from parsers  (Akim Demaille <akim@lrde.epita.fr>)
Responses Re: Remove useless associativity/precedence from parsers
List pgsql-hackers
On 5/21/19 11:49 AM, Akim Demaille wrote:
> Hi Tom!
>
>> Le 20 mai 2019 à 15:54, Tom Lane <tgl@sss.pgh.pa.us> a écrit :
>>
>> Akim Demaille <akim@lrde.epita.fr> writes:
>>> It is for the same reasons that I would recommend not using associativity directives (%left, %right, %nonassoc)
whereassociativity plays no role: %precedence is made for this.  But it was introduced in Bison 2.7.1 (2013-04-15), and
Idon't know if requiring it is acceptable to PostgreSQL.
 
>> 2013?  Certainly not.  We have a lot of buildfarm critters running
>> older platforms than that.
> This I fully understand.  However, Bison is a source generator,
> and it's quite customary to use modern tools on the maintainer
> side, and then deploy the result them on possibly much older
> architectures.
>
> Usually users of Bison build tarballs with the generated parsers
> in them, and ship/test from that.
>


The buildfarm client does not build from tarballs, it builds from git,
meaning it has to run bison. Thus Tom's objection is quite valid, and
your dismissal of it is not.


cheers


andrew


-- 

Andrew Dunstan                https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services




pgsql-hackers by date:

Previous
From: "Jonathan S. Katz"
Date:
Subject: Re: PostgreSQL 12 Beta 1 press release draft
Next
From: Tom Lane
Date:
Subject: Re: Remove useless associativity/precedence from parsers