Patch to make pgindent work cleanly

Поиск
Список
Период
Сортировка
От Gurjeet Singh
Тема Patch to make pgindent work cleanly
Дата
Msg-id CABwTF4WJG_QWaGruphXdVbOgysO5br9ad5pNk2bp1eFswHHkbw@mail.gmail.com
обсуждение исходный текст
Ответы Re: Patch to make pgindent work cleanly  (Bruce Momjian <bruce@momjian.us>)
Список 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.
Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Tomas Vondra
Дата:
Сообщение: Re: PATCH: Split stats file per database WAS: autovacuum stress-testing our system
Следующее
От: "Erik Rijkers"
Дата:
Сообщение: Re: Materialized views WIP patch