Re: [COMMITTERS] pgsql: Remove more SGML tabs.

From: David Fetter <david(at)fetter(dot)org>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: Bruce Momjian <bruce(at)momjian(dot)us>, PG Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [COMMITTERS] pgsql: Remove more SGML tabs.
Date: 2011-02-10 22:08:10
Message-ID: 20110210220810.GA13504@fetter.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Thu, Feb 10, 2011 at 09:16:09PM +0200, Peter Eisentraut wrote:
> On tor, 2011-02-10 at 10:40 -0800, David Fetter wrote:
> > I think all such checks belong in .git/hooks/pre-commit, and need
> > to be as cross-platform as needed for committers. Would a
> > *n*x-based version do for a start?
>
> I think as a matter of principle, the only things that belongs into
> git hooks are things that relate to the repository itself, such as
> commit metadata (author checks, commit message spell checks, etc.).
> Anything that relates to the source belongs into the source.

So on that principle, should "make check" do these checks?

Just to put another viewpoint out there, the reason that such hooks
exist is precisely to enable people to do deeper checks than commit
metadata.

Cheers,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778 AIM: dfetter666 Yahoo!: dfetter
Skype: davidfetter XMPP: david(dot)fetter(at)gmail(dot)com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message David Fetter 2011-02-10 22:22:27 Re: [COMMITTERS] pgsql: Remove more SGML tabs.
Previous Message Bruce Momjian 2011-02-10 21:58:09 pgsql: Fix "variable not used" warnings when USE_WIDE_UPPER_LOWER is no

Browse pgsql-hackers by date

  From Date Subject
Next Message David Christensen 2011-02-10 22:17:11 Re: ALTER EXTENSION UPGRADE, v3
Previous Message Tom Lane 2011-02-10 21:49:49 Re: ALTER EXTENSION UPGRADE, v3