Re: pgsql: Use $INDENT rather than indent throughout the pgindent code - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql: Use $INDENT rather than indent throughout the pgindent code
Date
Msg-id 5023.1298000073@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgsql: Use $INDENT rather than indent throughout the pgindent code  (Bruce Momjian <bruce@momjian.us>)
Responses Re: pgsql: Use $INDENT rather than indent throughout the pgindent code  (Alvaro Herrera <alvherre@commandprompt.com>)
List pgsql-committers
Bruce Momjian <bruce@momjian.us> writes:
> Andrew Dunstan wrote:
>> On 02/17/2011 08:23 PM, Alvaro Herrera wrote:
>>> Use $INDENT rather than indent throughout the pgindent code
>>> +INDENT=`which indent`

>> How universally available is "which"?

> I have 'which'.  I am unclear how 'which' helps in this case.

It doesn't.  Anything 'which' would find could perfectly well be invoked
by saying plain old "indent".  What would actually be useful here is a
convenient way to override the script's default value of INDENT, but
this coding doesn't offer that.  I'd suggest something more like

if [ -z "$INDENT" ]; then
  INDENT=indent
fi

            regards, tom lane

pgsql-committers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: pgsql: Use $INDENT rather than indent throughout the pgindent code
Next
From: Itagaki Takahiro
Date:
Subject: pgsql: Add transaction-level advisory locks.