Re: patch src/bin/psql/help.c

From: alex avriette <alex(at)posixnap(dot)net>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: patch src/bin/psql/help.c
Date: 2003-05-12 23:11:34
Message-ID: 12E7C7D8-84CF-11D7-B653-003065BDE8A6@posixnap.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches


>> Per the discussion earlier re: sockets and psql
>
> It would be clearer if you list the environment variables in a separate
> list below the options, not in the middle of the options without
> explanation. It seems, though, that if you list them in psql, you
> need to
> list them everywhere they apply, and that might get a little verbose.

Yeah. I would be happy to wander through and add a quick blurb about
the environment variables to the rest of the bin/ tree. However, I
think you're right, that it is a little too verbose. I couldn't come up
with a better way to phrase it. I suppose it might be better to leave
it out. My original aim was to make sure that the --help output
included the use of a directory for -h.

Hm. Alternatively, something like mpg123's --longhelp comes to mind.
Does anyone else see a need for something like that?

> However, the name of the environment variable is PGUSER, not $PGUSER.

Against my better judgment, I'll bite. 'Scuze me?

Alex

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Stephan Szabo 2003-05-12 23:19:25 Re: How are null's stored?
Previous Message Þórhallur Hálfdánarson 2003-05-12 23:03:20 Re: Scheduled jobs

Browse pgsql-patches by date

  From Date Subject
Next Message Alvaro Herrera 2003-05-13 03:55:31 Re: [PATCHES] Static snapshot data
Previous Message Andrew Dunstan 2003-05-12 21:13:22 CIDR addresses in pg_hba.conf