Re: pgsql: Work around Msys weakness in Testlib.pm's command_like()

From: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>, pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Work around Msys weakness in Testlib.pm's command_like()
Date: 2017-07-31 19:30:35
Message-ID: 8123c31c-5e70-caf7-76ed-6d9d5cd5b8a3@2ndquadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On 7/26/17 22:49, Andrew Dunstan wrote:
> Work around Msys weakness in Testlib.pm's command_like()
>
> When output of IPC::Run::run () is redirected to scalar references, in
> certain circumstances the Msys perl does not correctly detect that the
> end of file has been seen, making the test hang indefinitely. One such
> circumstance is when the command is 'pg_ctl start', and such a change
> was made in commit f13ea95f9e. The workaround, which only applies on
> MSys, is to redirect the output to temporary files and then read them in
> when the process has finished.

This commit message contains interesting information that would be
useful to record in a code comment. Otherwise it's not really clear
when or why to use the "safe" variant.

--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Heikki Linnakangas 2017-07-31 19:41:09 pgsql: Always use 2048 bit DH parameters for OpenSSL ephemeral DH ciphe
Previous Message Tom Lane 2017-07-31 17:43:06 pgsql: Doc: specify that the minimum supported version of Perl is 5.8.3