From: | David Fetter <david(at)fetter(dot)org> |
---|---|
To: | Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com> |
Cc: | Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [PATCH v1] Add a way to supply stdin to TAP tests |
Date: | 2019-05-07 16:47:59 |
Message-ID: | 20190507164759.GN28936@fetter.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, May 07, 2019 at 09:39:57AM -0400, Andrew Dunstan wrote:
>
> On 5/6/19 10:42 PM, David Fetter wrote:
> > On Tue, May 07, 2019 at 11:05:32AM +0900, Kyotaro HORIGUCHI wrote:
> >> Hi.
> >>
> >> At Sun, 28 Apr 2019 17:07:16 +0200, David Fetter <david(at)fetter(dot)org> wrote in <20190428150716(dot)GP28936(at)fetter(dot)org>
> >>> Our test coverage needs all the help it can get.
> >>>
> >>> This patch, extracted from another by Fabian Coelho, helps move things
> >>> in that direction.
> >>>
> >>> I'd like to argue that it's not a new feature, and that it should be
> >>> back-patched as far as possible.
> >> The comment for the parameter "in".
> >>
> >> +# - in: standard input
> >>
> >> Perhaps this is "string to be fed to standard input". This also
> >> can be a I/O reference but we don't care that?
> > OK
> >
> >> + $in = '' if not defined $in;
> >>
> >> run($cmd, '<', \undef) seems to work, maybe assuming "<
> >> /dev/null", which might be better?
> > Is /dev/null a thing on Windows?
>
> However, I don't think we should be faking anything here. I think it
> would be better to avoid setting $in if not supplied and then have this:
>
> if (defined($in))
>
> {
>
> IPC::Run::run($cmd, '<', \$in, '>', \$stdout, '2>', \$stderr);
>
> }
>
> else
>
> {
>
> IPC::Run::run($cmd, >', \$stdout, '2>', \$stderr);
>
> }
Done that way.
Best,
David.
--
David Fetter <david(at)fetter(dot)org> http://fetter.org/
Phone: +1 415 235 3778
Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate
Attachment | Content-Type | Size |
---|---|---|
v2-0001-Add-a-way-to-supply-stdin-to-TAP-tests.patch | text/x-diff | 1.7 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Geoghegan | 2019-05-07 16:59:56 | Re: VACUUM can finish an interrupted nbtree page split -- is that okay? |
Previous Message | Andres Freund | 2019-05-07 16:44:30 | Re: New EXPLAIN option: ALL |