Re: pgindent run? - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: pgindent run?
Date
Msg-id 200103220510.AAA14910@candle.pha.pa.us
Whole thread Raw
In response to Re: pgindent run?  (The Hermit Hacker <scrappy@hub.org>)
Responses Re: pgindent run?  (The Hermit Hacker <scrappy@hub.org>)
List pgsql-hackers
> > > and most times, those have to be merged into the source tree due to
> > > extensive changes anyway ... maybe we should just get rid of the use of
> > > pgindent altogether?  its not something that I've ever seen required on
> > > other projects I've worked on ... in general, most projects seem to
> > > require that a submit'd patch from an older release be at least tested on
> > > the newest CVS, and with nightly snapshots being created as it is, I
> > > really don't see why such a requirement is a bad thing ...
> >
> > In an ideal world, people would test on CVS but in reality, the patches
> > are usually pretty small and if they fix the problem, we apply them.
> >
> > Seems like a lot of work just to avoid pgindent.
> 
> If they are small, then why is pgindent required?  And if they are large,
> is it too much to ask that the person submitting tests the patch to make
> sure its even applicable in the newest snapshot?

The problem is that the small ones don't apply cleanly if they don't
match the indenting in the source.

--  Bruce Momjian                        |  http://candle.pha.pa.us pgman@candle.pha.pa.us               |  (610)
853-3000+  If your life is a hard drive,     |  830 Blythe Avenue +  Christ can be your backup.        |  Drexel Hill,
Pennsylvania19026
 


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: pgindent run?
Next
From: Bruce Momjian
Date:
Subject: Re: pgindent run?