Re: [PATCH v1] Add a way to supply stdin to TAP tests - Mailing list pgsql-hackers

From David Fetter
Subject Re: [PATCH v1] Add a way to supply stdin to TAP tests
Date
Msg-id 20190507024242.GH28936@fetter.org
Whole thread Raw
In response to Re: [PATCH v1] Add a way to supply stdin to TAP tests  (Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp>)
Responses Re: [PATCH v1] Add a way to supply stdin to TAP tests  (Andrew Dunstan <andrew.dunstan@2ndquadrant.com>)
List pgsql-hackers
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@fetter.org> wrote in <20190428150716.GP28936@fetter.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?

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



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: reindexdb & clusterdb broken against pre-7.3 servers
Next
From: Andres Freund
Date:
Subject: Re: [HACKERS] Broken O(n^2) avoidance in wal segment recycling.