Robert Haas <robertmhaas@gmail.com> writes:
> On Mon, Mar 2, 2015 at 2:38 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> 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.
Looks OK to me.
regards, tom lane