Lists: | pgsql-committerspgsql-hackers |
---|
From: | momjian(at)postgresql(dot)org (Bruce Momjian) |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Remove tabs in SGML. |
Date: | 2010-01-26 14:21:29 |
Message-ID: | 20100126142129.B125D7541BC@cvs.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
Log Message:
-----------
Remove tabs in SGML.
Modified Files:
--------------
pgsql/doc/src/sgml:
ecpg.sgml (r1.94 -> r1.95)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ecpg.sgml?r1=1.94&r2=1.95)
From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Bruce Momjian <momjian(at)postgresql(dot)org> |
Cc: | PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Remove tabs in SGML. |
Date: | 2010-01-26 18:20:26 |
Message-ID: | 20100126182026.GP31281@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
On Tue, Jan 26, 2010 at 02:21:29PM +0000, Bruce Momjian wrote:
> Log Message:
> -----------
> Remove tabs in SGML.
Can we see about making a commit hook for CVS that disallows \t in
SGML files? The process in git is pretty simple.
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
From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
---|---|
To: | David Fetter <david(at)fetter(dot)org> |
Cc: | Bruce Momjian <momjian(at)postgresql(dot)org>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Remove tabs in SGML. |
Date: | 2010-01-27 14:52:00 |
Message-ID: | 1264603920.22445.2.camel@fsopti579.F-Secure.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
On tis, 2010-01-26 at 10:20 -0800, David Fetter wrote:
> On Tue, Jan 26, 2010 at 02:21:29PM +0000, Bruce Momjian wrote:
> > Log Message:
> > -----------
> > Remove tabs in SGML.
>
> Can we see about making a commit hook for CVS that disallows \t in
> SGML files? The process in git is pretty simple.
Better write a check in the makefile.
From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
Cc: | Bruce Momjian <momjian(at)postgresql(dot)org>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Remove tabs in SGML. |
Date: | 2010-01-27 17:04:06 |
Message-ID: | 20100127170406.GV31281@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
On Wed, Jan 27, 2010 at 04:52:00PM +0200, Peter Eisentraut wrote:
> On tis, 2010-01-26 at 10:20 -0800, David Fetter wrote:
> > On Tue, Jan 26, 2010 at 02:21:29PM +0000, Bruce Momjian wrote:
> > > Log Message:
> > > -----------
> > > Remove tabs in SGML.
> >
> > Can we see about making a commit hook for CVS that disallows \t in
> > SGML files? The process in git is pretty simple.
>
> Better write a check in the makefile.
That assumes that everyone will always actually run the appropriate
make before committing, which I wouldn't assume, especially for a
"trivial" patch that's going out when someone is pressed for time.
Checking in an unavoidable place seems much more foolproof.
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
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | David Fetter <david(at)fetter(dot)org> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, Bruce Momjian <momjian(at)postgresql(dot)org>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Remove tabs in SGML. |
Date: | 2010-01-27 17:27:08 |
Message-ID: | 9027.1264613228@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
David Fetter <david(at)fetter(dot)org> writes:
> On Wed, Jan 27, 2010 at 04:52:00PM +0200, Peter Eisentraut wrote:
>> Better write a check in the makefile.
> That assumes that everyone will always actually run the appropriate
> make before committing, which I wouldn't assume, especially for a
> "trivial" patch that's going out when someone is pressed for time.
> Checking in an unavoidable place seems much more foolproof.
The rule against tabs in SGML files is only a bit of anal retentivity
anyway. I don't think that enforcing it is worth special hacks that
we do not use anywhere else.
regards, tom lane
From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, Bruce Momjian <momjian(at)postgresql(dot)org>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Remove tabs in SGML. |
Date: | 2010-01-27 17:30:03 |
Message-ID: | 20100127173003.GW31281@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
On Wed, Jan 27, 2010 at 12:27:08PM -0500, Tom Lane wrote:
> David Fetter <david(at)fetter(dot)org> writes:
> > On Wed, Jan 27, 2010 at 04:52:00PM +0200, Peter Eisentraut wrote:
> >> Better write a check in the makefile.
>
> > That assumes that everyone will always actually run the
> > appropriate make before committing, which I wouldn't assume,
> > especially for a "trivial" patch that's going out when someone is
> > pressed for time. Checking in an unavoidable place seems much
> > more foolproof.
>
> The rule against tabs in SGML files is only a bit of anal
> retentivity anyway. I don't think that enforcing it is worth
> special hacks that we do not use anywhere else.
Good point. What other pre-commit hooks do we need?
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
From: | "Joshua D(dot) Drake" <jd(at)commandprompt(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | David Fetter <david(at)fetter(dot)org>, Peter Eisentraut <peter_e(at)gmx(dot)net>, Bruce Momjian <momjian(at)postgresql(dot)org>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Remove tabs in SGML. |
Date: | 2010-01-27 17:33:31 |
Message-ID: | 1264613611.2105.58.camel@jd-desktop.unknown.charter.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
On Wed, 2010-01-27 at 12:27 -0500, Tom Lane wrote:
> David Fetter <david(at)fetter(dot)org> writes:
> > On Wed, Jan 27, 2010 at 04:52:00PM +0200, Peter Eisentraut wrote:
> >> Better write a check in the makefile.
>
> > That assumes that everyone will always actually run the appropriate
> > make before committing, which I wouldn't assume, especially for a
> > "trivial" patch that's going out when someone is pressed for time.
> > Checking in an unavoidable place seems much more foolproof.
>
> The rule against tabs in SGML files is only a bit of anal retentivity
> anyway.
I disagree. SGML suffers the same as code if you are using tabs.
Joshua D. Drake
--
PostgreSQL.org Major Contributor
Command Prompt, Inc: http://www.commandprompt.com/ - 503.667.4564
Consulting, Training, Support, Custom Development, Engineering
Respect is earned, not gained through arbitrary and repetitive use or Mr. or Sir.
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | David Fetter <david(at)fetter(dot)org> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, Bruce Momjian <momjian(at)postgresql(dot)org>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Remove tabs in SGML. |
Date: | 2010-01-27 17:41:28 |
Message-ID: | 9329.1264614088@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
David Fetter <david(at)fetter(dot)org> writes:
> On Wed, Jan 27, 2010 at 12:27:08PM -0500, Tom Lane wrote:
>> The rule against tabs in SGML files is only a bit of anal
>> retentivity anyway. I don't think that enforcing it is worth
>> special hacks that we do not use anywhere else.
> Good point. What other pre-commit hooks do we need?
Hm, aren't you one of the main cheerleaders for moving to git?
Why would you want us to be installing any additional CVS dependencies
right now? Adding commit hooks would just create another must-do-
before-moving task. If we actually desperately *needed* some, maybe
it'd be worth it, but I can't see that this kind of thing is worth it.
regards, tom lane
From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, Bruce Momjian <momjian(at)postgresql(dot)org>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Remove tabs in SGML. |
Date: | 2010-01-27 17:42:48 |
Message-ID: | 20100127174248.GX31281@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
On Wed, Jan 27, 2010 at 12:41:28PM -0500, Tom Lane wrote:
> David Fetter <david(at)fetter(dot)org> writes:
> > On Wed, Jan 27, 2010 at 12:27:08PM -0500, Tom Lane wrote:
> >> The rule against tabs in SGML files is only a bit of anal
> >> retentivity anyway. I don't think that enforcing it is worth
> >> special hacks that we do not use anywhere else.
>
> > Good point. What other pre-commit hooks do we need?
>
> Hm, aren't you one of the main cheerleaders for moving to git?
Yes.
> Why would you want us to be installing any additional CVS dependencies
> right now? Adding commit hooks would just create another must-do-
> before-moving task.
Right now is not the time.
> If we actually desperately *needed* some, maybe it'd be worth it,
> but I can't see that this kind of thing is worth it.
Post-git-move would be a better time to revisit this.
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
From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
---|---|
To: | David Fetter <david(at)fetter(dot)org> |
Cc: | Bruce Momjian <momjian(at)postgresql(dot)org>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Remove tabs in SGML. |
Date: | 2010-01-27 21:23:12 |
Message-ID: | 1264627392.4720.9.camel@vanquo.pezone.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
On ons, 2010-01-27 at 09:04 -0800, David Fetter wrote:
> On Wed, Jan 27, 2010 at 04:52:00PM +0200, Peter Eisentraut wrote:
> > On tis, 2010-01-26 at 10:20 -0800, David Fetter wrote:
> > > On Tue, Jan 26, 2010 at 02:21:29PM +0000, Bruce Momjian wrote:
> > > > Log Message:
> > > > -----------
> > > > Remove tabs in SGML.
> > >
> > > Can we see about making a commit hook for CVS that disallows \t in
> > > SGML files? The process in git is pretty simple.
> >
> > Better write a check in the makefile.
>
> That assumes that everyone will always actually run the appropriate
> make before committing, which I wouldn't assume, especially for a
> "trivial" patch that's going out when someone is pressed for time.
Even if you are pressed for time, you are supposed to check your patch.
And even then, we have the community and automated build services to
pick up after you.
> Checking in an unavoidable place seems much more foolproof.
I'm going to put in an advance vote here against any kind of
code-checking commit hook. Because they live in the repository
metadata, they can only be seen and changed by the system
administrators, they do not travel with the source code, and they are
not under version control themselves. The source code and the scripts
and tests to manage it should be self-consistent and not dependent on
external containers.
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
Cc: | David Fetter <david(at)fetter(dot)org>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Remove tabs in SGML. |
Date: | 2010-02-05 14:52:17 |
Message-ID: | 201002051452.o15EqH516497@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
Peter Eisentraut wrote:
> On tis, 2010-01-26 at 10:20 -0800, David Fetter wrote:
> > On Tue, Jan 26, 2010 at 02:21:29PM +0000, Bruce Momjian wrote:
> > > Log Message:
> > > -----------
> > > Remove tabs in SGML.
> >
> > Can we see about making a commit hook for CVS that disallows \t in
> > SGML files? The process in git is pretty simple.
>
> Better write a check in the makefile.
Done, and attached. The rule is more shell script than makefile, but I
am not sure how else to do it.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
Attachment | Content-Type | Size |
---|---|---|
/rtmp/diff | text/x-diff | 543 bytes |
From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, David Fetter <david(at)fetter(dot)org>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Remove tabs in SGML. |
Date: | 2010-02-05 19:04:22 |
Message-ID: | 20100205190422.GA3736@alvh.no-ip.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
Bruce Momjian wrote:
> Peter Eisentraut wrote:
> > On tis, 2010-01-26 at 10:20 -0800, David Fetter wrote:
> > > On Tue, Jan 26, 2010 at 02:21:29PM +0000, Bruce Momjian wrote:
> > > > Log Message:
> > > > -----------
> > > > Remove tabs in SGML.
> > >
> > > Can we see about making a commit hook for CVS that disallows \t in
> > > SGML files? The process in git is pretty simple.
> >
> > Better write a check in the makefile.
>
> Done, and attached. The rule is more shell script than makefile, but I
> am not sure how else to do it.
Maybe have the check-tabs rule as a dependency of the "check", "html"
and/or "draft" rules?
> + check-tabs:
> + ( ! grep ' ' $(wildcard $(srcdir)/*.sgml $(srcdir)/ref/*.sgml) ) || (echo "Tabs appear in SGML files"; exit 1)
--
Alvaro Herrera http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, David Fetter <david(at)fetter(dot)org>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Remove tabs in SGML. |
Date: | 2010-02-05 19:09:08 |
Message-ID: | 201002051909.o15J98D01315@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
Alvaro Herrera wrote:
> Bruce Momjian wrote:
> > Peter Eisentraut wrote:
> > > On tis, 2010-01-26 at 10:20 -0800, David Fetter wrote:
> > > > On Tue, Jan 26, 2010 at 02:21:29PM +0000, Bruce Momjian wrote:
> > > > > Log Message:
> > > > > -----------
> > > > > Remove tabs in SGML.
> > > >
> > > > Can we see about making a commit hook for CVS that disallows \t in
> > > > SGML files? The process in git is pretty simple.
> > >
> > > Better write a check in the makefile.
> >
> > Done, and attached. The rule is more shell script than makefile, but I
> > am not sure how else to do it.
>
> Maybe have the check-tabs rule as a dependency of the "check", "html"
> and/or "draft" rules?
>
> > + check-tabs:
> > + ( ! grep ' ' $(wildcard $(srcdir)/*.sgml $(srcdir)/ref/*.sgml) ) || (echo "Tabs appear in SGML files"; exit 1)
Uh, I didn't think people wanted to actually have any tab testing in
their automatically. Do they?
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
From: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, David Fetter <david(at)fetter(dot)org>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Remove tabs in SGML. |
Date: | 2010-02-05 19:13:11 |
Message-ID: | 20100205191311.GB3736@alvh.no-ip.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
Bruce Momjian wrote:
> Alvaro Herrera wrote:
> > Maybe have the check-tabs rule as a dependency of the "check", "html"
> > and/or "draft" rules?
> >
> > > + check-tabs:
> > > + ( ! grep ' ' $(wildcard $(srcdir)/*.sgml $(srcdir)/ref/*.sgml) ) || (echo "Tabs appear in SGML files"; exit 1)
>
> Uh, I didn't think people wanted to actually have any tab testing in
> their automatically. Do they?
IMO that's the point of the whole exercise.
--
Alvaro Herrera http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Alvaro Herrera <alvherre(at)commandprompt(dot)com> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, David Fetter <david(at)fetter(dot)org>, PG Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [COMMITTERS] pgsql: Remove tabs in SGML. |
Date: | 2010-02-05 19:32:13 |
Message-ID: | 201002051932.o15JWDw04656@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-committers pgsql-hackers |
Alvaro Herrera wrote:
> Bruce Momjian wrote:
> > Alvaro Herrera wrote:
>
> > > Maybe have the check-tabs rule as a dependency of the "check", "html"
> > > and/or "draft" rules?
> > >
> > > > + check-tabs:
> > > > + ( ! grep ' ' $(wildcard $(srcdir)/*.sgml $(srcdir)/ref/*.sgml) ) || (echo "Tabs appear in SGML files"; exit 1)
> >
> > Uh, I didn't think people wanted to actually have any tab testing in
> > their automatically. Do they?
>
> IMO that's the point of the whole exercise.
OK, done with attached patch, now done as part of 'gmake check'. If
people want it as part of the build process, please let me know.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
Attachment | Content-Type | Size |
---|---|---|
/rtmp/diff | text/x-diff | 1.0 KB |