When a TAP file has non-zero exit status, retain temporary directories. PostgresNode already retained base directories in such cases. Stop using $SIG{__DIE__}, which is redundant with the exit status check, in lieu of proliferating it to TestLib. Back-patch to 9.6, where commit 88802e068017bee8cea7a5502a712794e761c7b5 introduced retention on failure.
Reviewed by Daniel Gustafsson. Discussion: https://postgr.es/m/20200202170155.ga3264...@rfd.leadboat.com Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/bf989aaf35613ed2c2a66e120ca600e7b9da4f82 Modified Files -------------- src/test/perl/PostgresNode.pm | 17 +---------------- src/test/perl/TestLib.pm | 9 +++++++-- 2 files changed, 8 insertions(+), 18 deletions(-)