Re: [Fwd: Re: proposal: new long psql parameter --on-error-stop]

Hello,

From: "Pavel Stehule" <pavel.stehule@gmail.com>

fixed

Thank you.  All is fine.

should be ""Environment variables".  And this section lacks description
for Windows, such as:

+ printf(_("  NAME=VALUE [NAME=VALUE] psql ...\n  or \\setenv NAME [VALUE]
in interactive mode\n\n"));

+ printf(_("  PGPASSFILE         password file (default ~/.pgpass)\n"));

??? -

I meant that how to set environment variables on Windows command prompt is different from on UNIX/Linux, and the default password file path is also different on Windows.  Do we describe them in this help?

Lastly, to follow most of your descriptions, "s" at the end of the first verb in these messages should be removed.  For example, use "set" instead of "sets".

+ printf(_("  COMP_KEYWORD_CASE  determines which letter case to use when completing an SQL key word\n"));

+ printf(_("  columns            sets the target width for the wrapped format\n"));
+ printf(_("  linestyle          sets the border line drawing style [ascii, old-ascii, unicode]\n"));
+ printf(_("  recordsep          specifies the record (line) separator to use in unaligned output format\n"));
+ printf(_("  title              sets the table title for any subsequently printed tables\n"));


This is all I noticed in the review.


Regards
MauMau