Re: pg_regress cleans up tablespace twice.
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Cc: pgsql-hackers@lists.postgresql.org
Date: 2020-02-19T21:06:33Z
Lists: pgsql-hackers
Kyotaro Horiguchi <horikyota.ntt@gmail.com> writes: > But in the first place it comes from [1] and the comment says: >> * XXX it would be better if pg_regress.c had nothing at all to do with >> * testtablespace, and this were handled by a .BAT file or similar on >> * Windows. See pgsql-hackers discussion of 2008-01-18. > Is there any reason not to do that in vcregress.pl? I think the > commands other than 'check' don't needs this. I think the existing coding dates from before we had a Perl driver for this, or else we had it but there were other less-functional ways to replace "make check" on Windows. +1 for taking the code out of pg_regress.c --- but I'm not in a position to say whether the other part of your patch is sufficient. regards, tom lane
Commits
-
Revert "Remove reset of testtablespace from pg_regress on Windows"
- 61be85afabba 14.0 landed
-
Remove reset of testtablespace from pg_regress on Windows
- 2b2a070d98b2 14.0 landed
-
Lower privilege level of programs calling regression_main
- ce5d3424d641 12.0 cited