From: | Christoph Berg <cb(at)df7cb(dot)de> |
---|---|
To: | Noah Misch <noah(at)leadboat(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-hackers(at)postgreSQL(dot)org |
Subject: | Re: Portability issues in TAP tests |
Date: | 2014-07-21 14:06:52 |
Message-ID: | 20140721140652.GH3669@msg.df7cb.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Re: Noah Misch 2014-07-18 <20140718052625(dot)GA2231360(at)tornado(dot)leadboat(dot)com>
> Installing a new version of one Perl module is well within the capabilities of
> buildfarm owners. Saving them the trouble, which in turn means more of them
> actually activating the TAP tests, might justify the loss. I'd be sad to see
> the "subtest" use go away, but I lean toward thinking it's for the best.
>
> From a technical standpoint, it would be nicest to bundle copies of Test::More
> and IPC::Run for the test suites to use.
Please not. If no OS packages are available, there's still "cpan IPC::Run"
to get it installed to $HOME automatically.
> A minor point to add to your list:
>
> 5. The TAP suites don't work when $PWD contains spaces.
Here's yet another thing that I think is pretty major:
6. The tests fail if your $LANG isn't en_something:
ok 1 - vacuumdb -z postgres exit code 0
not ok 2 - vacuumdb -z: SQL found in server log
# Failed test 'vacuumdb -z: SQL found in server log'
# at /home/cbe/projects/postgresql/postgresql/9.4/build/../src/test/perl/TestLib.pm line 221.
# 'LOG: Anweisung: VACUUM (ANALYZE);
# '
# doesn't match '(?^:statement: VACUUM \(ANALYZE\);)'
# Looks like you failed 1 test of 2.
... repeated a gazillion times. pg_regress unsets LANG and LC_*, the
perl tests should do also.
Christoph
--
cb(at)df7cb(dot)de | http://www.df7cb.de/
From | Date | Subject | |
---|---|---|---|
Next Message | Emre Hasegeli | 2014-07-21 14:52:05 | Shapes on the regression test for polygon |
Previous Message | Andres Freund | 2014-07-21 12:51:50 | Re: [GSoC2014] Patch ALTER TABLE ... SET LOGGED |