public inbox for [email protected]
help / color / mirror / Atom feedFrom: Fabien COELHO <[email protected]>
To: Pavel Stehule <[email protected]>
Cc: Peter Eisentraut <[email protected]>
Cc: [email protected] <[email protected]>
Subject: Re: psql - add SHOW_ALL_RESULTS option
Date: Thu, 22 Jul 2021 11:00:41 +0200 (CEST)
Message-ID: <alpine.DEB.2.22.394.2107221051140.25741@pseudo> (raw)
In-Reply-To: <CAFj8pRD6XuTgbLCaHra+t1PJBEm8SL+y-Qmhax3-5A=0Q9xO0g@mail.gmail.com>
References: <[email protected]>
<[email protected]>
<[email protected]>
<[email protected]>
<alpine.DEB.2.22.394.2104151336060.191178@pseudo>
<[email protected]>
<alpine.DEB.2.22.394.2106121130400.4105466@pseudo>
<CALDaNm0+pzGgp-M1mAoUVYRtjadbWYuB96J8RaV9iH3+HA8FTA@mail.gmail.com>
<alpine.DEB.2.22.394.2107151742300.1705285@pseudo>
<[email protected]>
<alpine.DEB.2.22.394.2107220744430.25741@pseudo>
<CAFj8pRD6XuTgbLCaHra+t1PJBEm8SL+y-Qmhax3-5A=0Q9xO0g@mail.gmail.com>
Hello Pavel,
>> The newly added PSQL_WATCH_PAGER feature which broke the patch does not
>> seem to be tested anywhere, this is tiring:-(
>
> Do you have any idea how this can be tested?
The TAP patch sent by Peter on this thread is a very good start.
> It requires some pager that doesn't use blocking reading, and you need
> to do remote control of this pager. So it requires a really especially
> written pager just for this purpose. It is solvable, but I am not sure
> if it is adequate to this patch.
Not really: The point would not be to test the pager itself (that's for
the people who develop the pager, not for psql), but just to test that the
pager is actually started or not started by psql depending on conditions
(eg pset pager…) and that it does *something* when started. See for
instance the simplistic pager.pl script attached, the output of which
could be tested. Note that PSQL_PAGER is not tested at all either.
Basically "psql" is not tested, which is a pain when developing a non
trivial patch.
--
Fabien.
Attachments:
[text/x-perl] pager.pl (61B, ../alpine.DEB.2.22.394.2107221051140.25741@pseudo/2-pager.pl)
download | inline:
#! /usr/bin/perl -wn
print "$.\t$_";
exit(0) if $. == 30;
view thread (104+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected]
Subject: Re: psql - add SHOW_ALL_RESULTS option
In-Reply-To: <alpine.DEB.2.22.394.2107221051140.25741@pseudo>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox