Re: pg_regress cleans up tablespace twice.
Thomas Munro <thomas.munro@gmail.com>
From: Thomas Munro <thomas.munro@gmail.com>
To: Michael Paquier <michael@paquier.xyz>
Cc: Kyotaro Horiguchi <horikyota.ntt@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2020-06-20T03:01:36Z
Lists: pgsql-hackers
On Sat, Jun 20, 2020 at 2:42 PM Michael Paquier <michael@paquier.xyz> wrote: > > +ERROR: could not set permissions on directory > > "C:/projects/postgresql/src/test/regress/testtablespace": Permission > > denied > > > > Any ideas? Here's what it does: > > > > https://github.com/macdice/cfbot/tree/master/appveyor > > I am not sure, and I am not really familiar with this stuff. Your > code does a simple vcregress check, and that should take care of > automatically cleaning up the testtablespace path. The buildfarm uses > this code for MSVC builds and does not complain, nor do my own VMs > complain. A difference in the processing after 2b2a070d is that the > tablespace cleanup/creation does not happen while holding a restricted > token [1] anymore because it got out of pg_regress.c. Are there any > kind of restrictions applied to the user running appveyor on Windows? Thanks for the clue. Appveyor runs your build script as a privileged user (unlike, I assume, the build farm animals), and that has caused a problem with this test in the past, though I don't know the details. I might go and teach it to skip that test until a fix can be found.
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