pgsql: Make upgradecheck a no-op in MSVC's vcregress.pl - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: Make upgradecheck a no-op in MSVC's vcregress.pl
Date
Msg-id E1naU6P-000OSi-Ra@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Make upgradecheck a no-op in MSVC's vcregress.pl

322becb has changed upgradecheck to use the TAP tests, discarding
pg_upgrade's tests in bincheck.  However, this is proving to be a bad
idea for the Windows buildfarm clients that use MSVC when TAP tests are
disabled as this causes a hard failure at the pg_upgrade step.

This commit disables upgradecheck, moving the execution of the tests of
pg_upgrade to bincheck, as per an initial suggestion from Andres
Freund, so as the buildfarm is able to live happily with those changes.

While on it, remove the routine that was used by upgradecheck to
create databases whose names are generated with a range of ASCII
characters as it is not used since 322becb.  upgradecheck is removed
from the CI script for Windows, as bincheck takes care of that now.

Per report from buildfarm member hamerkop (MSVC 2017 without a TAP
setup).

Reviewed-by: Justin Pryzby
Discussion: https://postgr.es/m/YkbnpriYEAagZ2wH@paquier.xyz

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/d2a2ce4184b0038adb1d6d292f12685056e2ab7a

Modified Files
--------------
.cirrus.yml                       |  2 --
doc/src/sgml/install-windows.sgml |  1 -
src/tools/msvc/vcregress.pl       | 33 +++++----------------------------
3 files changed, 5 insertions(+), 31 deletions(-)


pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: pageinspect: Use better macros to get special page area for GIN
Next
From: Michael Paquier
Date:
Subject: Re: pgsql: SQL/JSON query functions