Re: fairywren is generating bogus BASE_BACKUP commands
Andres Freund <andres@anarazel.de>
From: Andres Freund <andres@anarazel.de>
To: Andrew Dunstan <andrew@dunslane.net>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Robert Haas <robertmhaas@gmail.com>, Thomas Munro <thomas.munro@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2022-01-23T21:31:34Z
Lists: pgsql-hackers
Hi, On 2022-01-23 16:09:01 -0500, Andrew Dunstan wrote: > Msys is a unix-like environment that is useful to build Postgres. It's > not intended as a general runtime environment. We therefore don't build > msys-aware Postgres. We use msys to build standalone Postgres binaries > that don't need or use any msys runtime. There is nothing in the least > bit new about this - that's the way it's been since day one of the > Windows port nearly 20 years ago. > > Speaking as someone who (for my sins) regularly deals with problems on > Windows, I find msys much easier to deal with than VisualStudio, > probably because it's so much like what I use elsewhere. So I think > dropping msys support would be a serious mistake. I agree that msys support is useful (although configure is *so* slow that I find its usefullness reduced substantially). > The most common issues we get are around this issue of virtualized paths > in the TAP tests. If people followed the rule I suggested upthread, 99% > of those problems would go away. I realize it's annoying - I've been > caught by it myself on more than one occasion. Maybe there's a way to > avoid it, but if there is I'm unaware of it. But I don't think it's in > any way a good reason to drop msys support. Needing to sprinkle perl2host and MSYS2_ARG_CONV_EXCL over a good number of tests, getting weird errors when failing, etc IMO isn't a scalable approach, for a platform that most of use never use. Can't we solve this in a generic way? E.g. by insisting that the test run with a native perl and normalizing the few virtual paths we get invoked with centrally? Making the msys initial setup a bit more cumbersome would IMO be an OK price to pay for making it maintainable / predictable from other platforms, as long as we have some decent docs and decent error messages. Greetings, Andres Freund
Commits
-
Remove PostgreSQL::Test::Utils::perl2host completely
- 95d981338b24 15.0 landed
- b30c62bd42eb 10.21 landed
- b4a6ceed52ee 11.16 landed
- 8358eacc954d 12.11 landed
- c27aa21e0dc8 13.7 landed
- 652ff988fbf6 14.3 landed
-
Unbreak pg_basebackup/t/010_pg_basebackup.pl on msys
- 4f0bcc735038 15.0 landed
-
On Windows, also call shutdown() while closing the client socket.
- ed52c3707bcf 15.0 cited
-
On Windows, close the client socket explicitly during backend shutdown.
- 6051857fc953 15.0 cited