Date: Thu, 13 Aug 2020 16:16:28 -0400
From: Stephen Frost <sfrost@snowman.net>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Magnus Hagander <magnus@hagander.net>, Tom Lane <tgl@sss.pgh.pa.us>,
	Noah Misch <noah@leadboat.com>, Jesse Zhang <sbjesse@gmail.com>,
	Andres Freund <andres@anarazel.de>,
	PostgreSQL-development <pgsql-hackers@postgresql.org>
Subject: Re: run pgindent on a regular basis / scripted manner
Message-ID: <20200813201628.GA29590@tamriel.snowman.net>
References: <20200812223409.6di3y2qsnvynao7a@alap3.anarazel.de>
 <CAGf+fX71Oa2vzg0xdj+vgrjmdVrm=kRLc7H3SEGtCHkGhoWDCg@mail.gmail.com>
 <328084.1597276021@sss.pgh.pa.us>
 <20200813035308.GA1137170@rfd.leadboat.com>
 <397020.1597291716@sss.pgh.pa.us>
 <CABUevEwg3LmBk-+ZiAYZW71dsB=boS-8sdYBux7M6qZp46avtA@mail.gmail.com>
 <20200813163043.GZ29590@tamriel.snowman.net>
 <CA+TgmoYNGNucnPnaEKDbhQQJL8wETmZcnBzpjiKNzaTzbqV9BA@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="Kf9dPzkPP9+q7vtQ"
Content-Disposition: inline
In-Reply-To: 
 <CA+TgmoYNGNucnPnaEKDbhQQJL8wETmZcnBzpjiKNzaTzbqV9BA@mail.gmail.com>
User-Agent: Mutt/1.5.24 (2015-08-30)
Precedence: bulk


--Kf9dPzkPP9+q7vtQ
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Greetings,

* Robert Haas (robertmhaas@gmail.com) wrote:
> On Thu, Aug 13, 2020 at 12:30 PM Stephen Frost <sfrost@snowman.net> wrote:
> > So, in our world, wouldn't this translate to 'make cfbot complain'?
>=20
> This seems like it would be useful, but we'd have to figure out what
> to do about typedefs.list. If the patch is indented with the current
> one (which is auto-generated by the entire build farm, remember) it's
> likely to mess up a patch that's otherwise properly formatted. We'd
> either need to insist that people include updates to typedefs.list in
> the patch, or else have the cfbot take a stab at doing those updates
> itself.

For my 2c, anyway, I like the idea of having folks update the typedefs
themselves when they've got a patch that needs a new typedef to be
indented correctly.  Having cfbot try to do that seems unlikely to work
well.

I also didn't mean to imply that we'd push back and ask for a rebase due
to indentation changes, but at the same time, I question if it's really
that realistic a concern- either whomever posted the patch ran pgindent
on it, or they didn't, and I doubt cfbot's check of that would change
without there being a conflict between the patch and something that got
committed anyway.

I also disagree that it's that much of a burden to ask people who are
already hacking on PG to install pgindent.

All that said, seems that others feel differently and while I still
think it's a pretty reasonable idea to have cfbot check, if no one
agrees with me, that's fine too.  Having the pre-commit hook would help
with the downstream issue of pgindent pain from unrelated incorrect
indentation, so at least dealing with the patch author not properly
indenting to start with would be just on the bits the patch is already
modifying, which is a lot better.

Thanks,

Stephen

--Kf9dPzkPP9+q7vtQ
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQIcBAEBCgAGBQJfNZ+cAAoJEO1sijiDR2RVZSEQAKv76YXCYZembi2I7NPMGMvS
8jl0+4n+/ldwFFgx7eWV6/VJKP6sJOQVl76aH+fxbYjZgH/Y9q7YUhzmjjxrZDEJ
RMnJiew5Yr58QybPDqcfrJhVZJnpaZw3yS/qAXewVdD4+ZkA70F79GDp3MpL/LGS
ztJ6LtchbZS4OzpTG53+n6NKMwco/RWD4w2vto+EJHV0WpV7hFNw0NFHomXDLynv
C37AV8eu7u1CTzWXK0k7Bs4gOMyRKC9adiYDn2+CTr2ZUtrSOW26IVurhfWBi86u
2FRsR44O2G5yl7F43EOK4IIevjZELzULopKQHk2XlirkIKozHjm1Zzx2dDrAXH1E
GWIxAZFJ8HQjtVIjsnqVIn3xvEQbGC3gvD6lPH+IpLVvEy3/iG7BM7A40CZdOaI1
ZLhG9bkts4qqwrDIt/gwkW7kglyBQNVUXqXkDa5akKdajrrewCyyKsCX9TES2m8X
RMgdV/dBFy6i/yji/1W1LaMb3gpxS+PLSBtQ2vVnwreyggEZ1fgabjcLNCFy5uEN
mrPHifOeB4j5nB4E0jN8wBDRrgwp8e+BMIhBa+CqIOtVh/rzVC97I+1uzkEu7eIz
SWd+mnjAkOLlohB1zLjOKg0+mP70+O/nXcon3ZPDCbsXBPR7fnMDa1zhooHKboaC
RNR/SvYadZY8QzrqqSzI
=d8Fi
-----END PGP SIGNATURE-----

--Kf9dPzkPP9+q7vtQ--