Re: pgindent vs dtrace on macos
Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
From: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2020-05-22T09:29:25Z
Lists: pgsql-hackers
On 2020-05-20 15:56, Tom Lane wrote: > On a closely related point: I was confused for awhile on Monday > afternoon, wondering why the built tarballs didn't match my local > tree. I eventually realized that when I'd run pgindent on Saturday, > it had reformatted some generated files such as > src/bin/psql/sql_help.h, causing those not to match the freshly-made > ones in the tarball. I wonder if we should make an effort to ensure > that our generated .h and .c files always satisfy pgindent. We should generally try to do that, if only so that they don't appear weird and random when looking at them. I think in the past it would have been very difficult for a generation script to emulate pgindent's weird un-indentation logic on trailing lines, but that shouldn't be a problem anymore. -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Commits
-
Exclude fmgrprotos.h from pgindent processing.
- c4133ec169df 14.0 landed
-
Fix a few more generator scripts to produce pgindent-clean output.
- f859c2ffa01d 14.0 landed
-
Further improve pgindent's list of file exclusions.
- 74d4608f506b 14.0 landed
-
Improve formatting of create_help.pl and plperl_opmask.pl output.
- add105840b67 14.0 landed