Lists: | pgsql-hackers |
---|
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-hackers(at)postgreSQL(dot)org |
Subject: | Modest proposal: run check_keywords.pl on every build |
Date: | 2012-09-27 01:16:21 |
Message-ID: | 10396.1348708581@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
I've had it with mopping up after oversights like this one:
http://archives.postgresql.org/pgsql-hackers/2012-09/msg01057.php
We made a script (src/tools/check_keywords.pl) to check for this type of
error years ago, and even added it to the top-level "maintainer-check"
target awhile back, but nonetheless people continue to screw it up on
a regular basis (twice already this year alone). I think we should move
the script into src/backend/parser and run it as part of the make gram.c
target. (Doing it that way will not cause perl to become required for
building from tarballs, since gram.c is supplied prebuilt in tarballs.
And we already require perl for builds from raw git pulls.)
Any objections?
regards, tom lane
From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)postgreSQL(dot)org |
Subject: | Re: Modest proposal: run check_keywords.pl on every build |
Date: | 2012-09-27 02:00:52 |
Message-ID: | 5063B354.7030604@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On 09/26/2012 09:16 PM, Tom Lane wrote:
> I've had it with mopping up after oversights like this one:
> http://archives.postgresql.org/pgsql-hackers/2012-09/msg01057.php
>
> We made a script (src/tools/check_keywords.pl) to check for this type of
> error years ago, and even added it to the top-level "maintainer-check"
> target awhile back, but nonetheless people continue to screw it up on
> a regular basis (twice already this year alone). I think we should move
> the script into src/backend/parser and run it as part of the make gram.c
> target. (Doing it that way will not cause perl to become required for
> building from tarballs, since gram.c is supplied prebuilt in tarballs.
> And we already require perl for builds from raw git pulls.)
>
> Any objections?
>
>
works for me.
cheers
andrew
From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Modest proposal: run check_keywords.pl on every build |
Date: | 2012-09-27 13:29:50 |
Message-ID: | CA+Tgmoa8xDW50Nsn1BNhX+nZXQ3g6+sik_L99Zj5=pV60PzpNw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Wed, Sep 26, 2012 at 9:16 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Any objections?
None, and my apologies for the oversight.
I would really like to have a cross-check for duplicated OIDs, too.
After a couple of years I think I have learned most of the rules for
what stuff I need to cross-check before committing, but it is a
nuisance to remember to do it each time and, despite my best efforts,
I have fouled things up a few times.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company