From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: pgindent (was Re: [COMMITTERS] pgsql: Preventive maintenance in advance of pgindent run.) |
Date: | 2017-05-17 18:24:22 |
Message-ID: | 20170517182422.ps5absh5h7v2sehj@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Tom Lane wrote:
> Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> writes:
> > We have someone who has studied the BSD indent code and even sent us
> > patches to fix quite a few bugs, but we've largely ignored his efforts
> > so far. I propose we take that indent as part of our repo, and patch it
> > to our preferences.
>
> Messing with pgindent didn't seem all that high-priority while we were
> in development mode, and it would also have been pretty painful to have
> a pgindent that wanted to revisit settled decisions when you just wanted
> it to fix new code. Maybe now (ie, over the next few weeks) is a good
> time to pursue it, before we start a new round of code development.
Sounds reasonable.
> Not sure about actually incorporating it into our repo. Doing so would
> make it easier for people to use, for sure, and the license seems to be
> regular 3-clause BSD, so that angle is OK. But do we want to be carrying
> around another 150K of source code?
The alternatives are
1. rely on the dead code we've been using so far (the old BSD indent
patched with our Pg-specific tweaks), or
2. rely on someone else's upstream code -- in this case, FreeBSD's as
patched by Piotr.
Now that Piotr's is about to find a home, perhaps it's okay for us to
rely on that one. I just didn't like the idea of running something from
Piotr's personal repo.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Stephen Frost | 2017-05-17 18:25:10 | Re: [COMMITTERS] pgsql: Preventive maintenance in advance of pgindent run. |
Previous Message | Tom Lane | 2017-05-17 18:22:14 | Re: [COMMITTERS] pgsql: Preventive maintenance in advance of pgindent run. |
From | Date | Subject | |
---|---|---|---|
Next Message | Stephen Frost | 2017-05-17 18:25:10 | Re: [COMMITTERS] pgsql: Preventive maintenance in advance of pgindent run. |
Previous Message | Tom Lane | 2017-05-17 18:22:14 | Re: [COMMITTERS] pgsql: Preventive maintenance in advance of pgindent run. |