Tighten usage of PSQL_WATCH_PAGER.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 12 May 2023 20:11:14 +0000 (16:11 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 12 May 2023 20:11:14 +0000 (16:11 -0400)
commit51b2c08798867cb9788090704b37c4698b456240
tree2a92b72f73b17f7550bcba99677a4a112a4e771d
parente32701b8d285cd402f75fc9059d083e89ee8166b
Tighten usage of PSQL_WATCH_PAGER.

Don't use PSQL_WATCH_PAGER when stdin/stdout are not a terminal.
This corresponds to the restrictions on when other commands will
use [PSQL_]PAGER.  There isn't a lot of sense in trying to use a
pager in non-interactive cases, and doing so allows an environment
setting to break our tests.

Also, ignore PSQL_WATCH_PAGER if it is set but empty or all-blank,
for the same reasons we ignore such settings of [PSQL_]PAGER (see
commit 18f8f784c).

No documentation change is really needed, since there is nothing
suggesting that these constraints on [PSQL_]PAGER didn't already
apply to PSQL_WATCH_PAGER too.  But I rearranged the text
a little to make it read more naturally (IMHO anyway).

Per report from Pavel Stehule.  Back-patch to v15 where
PSQL_WATCH_PAGER was introduced.

Discussion: https://postgr.es/m/CAFj8pRDTwFzmEWdA-gdAcUh0ZnxUioSfTMre71WyB_wNJy-8gw@mail.gmail.com
doc/src/sgml/ref/psql-ref.sgml
src/bin/psql/command.c