From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Make flex/bison checks stricter in Git trees |
Date: | 2016-09-27 20:41:10 |
Message-ID: | B3986385-00AB-487E-8CDA-95DE9B2A0C47@yesql.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> On 27 Sep 2016, at 15:49, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Daniel Gustafsson <daniel(at)yesql(dot)se> writes:
>> When running ./configure on a system without Flex/Bison it’s easy to miss the
>> warning that flies past and then run into compilation error instead. When it
>> happened to a colleague yesterday a brief discussion came to the conclusion
>> that it would be neat it the flex and bison checks took the existence of the
>> generated files into consideration.
>
>> Attached patch scans for the generated files and iff flex or bison isn’t found
>> in a non-cross compilation build, errors out in case the generated filed don’t
>> exist while retaining the warning in case they do.
>
> Not exactly convinced this is a good idea. What if the files exist but
> are out of date?
Wouldn’t that be the same as today if so?
> More generally, how much advantage is there really in
> failing at configure rather than build time?
The error reporting is clearer IMO but it’s a matter of taste.
> The subtext here is that I'm disinclined to load more behavior into
> configure while we're waiting to see if cmake conversion happens.
> That job is tough enough without the autoconf sources being more of
> a moving target than they have to be.
Fair enough, that’s a very valid argument.
Thanks!
cheers ./daniel
From | Date | Subject | |
---|---|---|---|
Next Message | Vitaly Burovoy | 2016-09-27 20:48:25 | Re: Detect supported SET parameters when pg_restore is run |
Previous Message | Robert Haas | 2016-09-27 20:32:51 | Re: Partition-wise join for join between (declaratively) partitioned tables |