Michael Paquier wrote: > On Mon, Feb 27, 2017 at 11:28 AM, Craig Ringer <cr...@2ndquadrant.com> wrote:
> > Instead of > > > > $exc_save !~ /^$timeout_exception.*/ > > > > I've updated to: > > > > $exc_save !~ /^\Q$timeout_exception\E/ > > > > i.e. don't do an unnecessary wildcard match at the end, and disable > > metachar interpretation in the substituted range. > > > > Still needs applying to pg9.6 and pg10. > > I did not understand at first what you meant, but after looking at the > commit message of the patch things are clear: > Newer Perl or IPC::Run versions default to appending the filename to string > exceptions, e.g. the exception > psql timed out > is thrown as > psql timed out at /usr/share/perl5/vendor_perl/IPC/Run.pm line 2961. Hmm, I think this is really a bugfix that we should backpatch all the way back to where we introduced PostgresNode. Lately I've been wondering about backpatching the whole TAP test infrastructure, all the way back. As we notice bugs, it's really useful to use newly added tests in all branches; but currently PostgresNode doesn't work with old branches, particularly since the '-w' switch was removed from pg_ctl invokations in PostgresNode->start and ->restart methods -- (the test just fail without any indication of what is going on). -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
