Patch to make pgindent work cleanly - Mailing list pgsql-hackers

From Gurjeet Singh
Subject Patch to make pgindent work cleanly
Date
Msg-id CABwTF4WJG_QWaGruphXdVbOgysO5br9ad5pNk2bp1eFswHHkbw@mail.gmail.com
Whole thread Raw
Responses Re: Patch to make pgindent work cleanly  (Bruce Momjian <bruce@momjian.us>)
List pgsql-hackers
Please find attached the patch for some cleanup and fix bit rot in pgindent script.

There were a few problems with the script.

.) It failed to use the $ENV{PGENTAB} even if it was set.
.) The file it tries to download from Postgres' ftp site is no longer present.
    ftp://ftp.postgresql.org/pub/dev/indent.netbsd.patched.tgz
.) The tar command extracted the above-mentioned file to a child directory, but did not descend into it before running make on it.
    I think it expected a tarbomb, but clearly the current .tgz file on ftp site is not a tarbomb.

I don't like the fact that it dies with a message "fetching xyz" rather than saying "Could not fetch xyz", but this patch does not address that since it doesn't really affect the output when script does work.

With this patch in place I could very easily run it on any arbitrary file, which seemed like a black-magic before the patch.

src/tools/pgindent/pgindent --build <your file path here>

--
Gurjeet Singh

http://gurjeet.singh.im/

EnterprsieDB Inc.
Attachment

pgsql-hackers by date:

Previous
From: Tomas Vondra
Date:
Subject: Re: PATCH: Split stats file per database WAS: autovacuum stress-testing our system
Next
From: "Erik Rijkers"
Date:
Subject: Re: Materialized views WIP patch