Re: [COMMITTERS] pgsql: Refactor Perl test code

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [COMMITTERS] pgsql: Refactor Perl test code
Date: 2015-12-03 03:40:51
Message-ID: CAB7nPqTEgZM9jOfjp-wBLVWJW1f_qFGVrDZOyA3JHqiDH=ZYHw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Thu, Dec 3, 2015 at 12:19 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> BTW, not the fault of this patch in particular, but this example points
> up the complaint I've had right along about how opaque TAP test failures
> are. How did you dig down to see that error message?

Well, it showed up on my terminal...

> Is it even possible to diagnose such a failure from what the buildfarm logs?

Yes. If you look at TestLib.pm, stderr/stdout redirection is done in
an INIT block, not BEGIN block, and INIT gets executed *after* the
code is compiled. FWIW, I recall arguing in favor of adding this
redirection logic in BEGIN so as we could get compilation errors
directly in the log files... The reason why it is done this way is
that it has been argued as well that we should not change the FS in
the BEGIN block, but in the INIT block when TestLib.pm is loaded.
--
Michael

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2015-12-03 03:50:24 Re: [COMMITTERS] pgsql: Refactor Perl test code
Previous Message Tom Lane 2015-12-03 03:19:49 Re: [COMMITTERS] pgsql: Refactor Perl test code

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2015-12-03 03:50:24 Re: [COMMITTERS] pgsql: Refactor Perl test code
Previous Message Josh Berkus 2015-12-03 03:36:56 Re: Fwd: Another little thing about psql wrapped expanded output