Re: pgsql: pgbench: Add a real expression syntax to \set - Mailing list pgsql-committers

From Robert Haas
Subject Re: pgsql: pgbench: Add a real expression syntax to \set
Date
Msg-id CA+TgmobvrcD0UsxkhNooVYZZdOvKQgeCJJb+NjNEfyfbvPE1qA@mail.gmail.com
Whole thread Raw
In response to Re: pgsql: pgbench: Add a real expression syntax to \set  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: pgsql: pgbench: Add a real expression syntax to \set  (Fabien COELHO <coelho@cri.ensmp.fr>)
Re: pgsql: pgbench: Add a real expression syntax to \set  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-committers
On Mon, Mar 2, 2015 at 2:38 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Robert Haas <rhaas@postgresql.org> writes:
>> pgbench: Add a real expression syntax to \set
>
> The makefile changes in this do not look right to me.  Files that are
> meant to be shipped in the tarball should be removed by make
> maintainer-clean, not an ordinary "make clean" which is what the committed
> patch appears to do.  Otherwise, a tarball user without bison installed
> would be cut off at the knees if he did "make clean" or "make distclean";
> but those operations are not supposed to remove files that were in the
> tarball.

Hmm, yeah.  Also, it appears that the exprscan.h file doesn't actually
get (or need to be) generated here.  So I think we need something like
the attached.

Thought?

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Attachment

pgsql-committers by date:

Previous
From: Fabien COELHO
Date:
Subject: Re: pgsql: pgbench: Add a real expression syntax to \set
Next
From: Fabien COELHO
Date:
Subject: Re: pgsql: pgbench: Add a real expression syntax to \set