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
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 |
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 |