Re: v7.2.4 bundled ...

From: Kevin Brown <kevin(at)sysexperts(dot)com>
To: PostgreSQL Development <pgsql-hackers(at)postgresql(dot)org>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: v7.2.4 bundled ...
Date: 2003-01-30 06:51:13
Message-ID: 20030130065113.GE12957@filer
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Tom Lane wrote:
> Rod Taylor <rbt(at)rbt(dot)ca> writes:
> > Updated to tag REL7_2_4 on FreeBSD 4.7 and cannot compile it. gram.y
> > errors complaining: invalid character: ','.
>
> > bash-2.05b$ bison --version
> > bison (GNU Bison) 1.75
>
> We just had that discussion on pgcore. The 7.2 grammar was developed
> against bison 1.28; it works with warnings against bison 1.35, but bison
> 1.75 just flat rejects it (not for any significant reason, but just
> because they decided to get anal-retentive about whether they'd allow
> commas in keyword lists).
>
> We could update the 7.2 grammar and compile it with the latest bison,
> but we were worried about whether we might introduce any subtle problems
> if we did. The 7.2 branch has received zero testing with bison 1.75.
>
> ISTM that the eve of what'll probably be our last dot-release for 7.2
> is not the time to drop a new bison into its toolchain.

For what it's worth, I've fixed all the errors in the 7.2.4 gram.y
file that bison 1.75 complained and then re-ran bison 1.35 against it,
then compared that with the output that the same version of bison
generated from the original grammar file. The only differences were
references to line numbers -- everything else is identical.

So if any problems occur from using bison 1.75, they will be either
due to bugs in that version of bison or due to our dependence on a bug
in earlier versions, or something like that.

I'm attaching a patch for 7.2.4's parser/gram.y that fixes all of
bison 1.75's complaints. Since the output of bison 1.35 is
essentially identical between the original and this, I don't see any
reason we shouldn't include the fix in the 7.2.4 release, as long as
we include a warning in the release notes that we haven't done any
real testing with a build against bison 1.75 (or later).

--
Kevin Brown kevin(at)sysexperts(dot)com

Attachment Content-Type Size
psql-7.2.4-gram.y.diff text/plain 15.5 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Kevin Brown 2003-01-30 06:56:31 Re: v7.2.4 bundled ...
Previous Message Tom Lane 2003-01-30 06:28:13 Re: v7.2.4 bundled ...

Browse pgsql-patches by date

  From Date Subject
Next Message Kevin Brown 2003-01-30 06:56:31 Re: v7.2.4 bundled ...
Previous Message Tom Lane 2003-01-30 06:28:13 Re: v7.2.4 bundled ...