From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
---|---|
To: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | pg_rewind tests |
Date: | 2015-03-31 00:59:21 |
Message-ID: | 5519F169.8030406@gmx.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
There are some small issues with the pg_rewind tests.
This technique
check: all
$(prove_check) :: local
$(prove_check) :: remote
for passing arguments to "prove" does not work with the tools included
in Perl 5.8.
While sorting out the portability issues in the TAP framework during the
9.4 release cycle, we had set 5.8 as the oldest Perl version that is
supported. (It's the Perl version in RHEL 5.) I suggest using
environment variables instead, unless we want to change that.
Moreover,
if ($test_mode == "local")
...
elsif ($test_mode == "remote")
don't work, because those are numerical comparisons, not string
comparisons. So the remote branch is never actually run.
Finally, RewindTest.pm should use
use strict;
use warnings;
and the warnings caused by that should be addressed.
From | Date | Subject | |
---|---|---|---|
Next Message | Tomas Vondra | 2015-03-31 01:20:31 | Re: PATCH: pgbench - merging transaction logs |
Previous Message | Peter Eisentraut | 2015-03-31 00:40:14 | Re: Exposing PG_VERSION_NUM in pg_config |