Fix obsolete config-module-exclusion logic in vcregress.pl.

The recent addition of regression tests to uuid-ossp exposed the fact
that the MSVC build system wasn't being consistent about whether it was
building/testing that contrib module, ie, it would try to test the module
even when it hadn't built it.  The same hazard was latent for sslinfo.

For the moment I just copied the more up-to-date logic from point A to
point B, but this is screaming for refactoring.

Per buildfarm results.

diff --git a/src/tools/msvc/Install.pm b/src/tools/msvc/Install.pm
index 55de9e55e6b9e8b6476ada4d5c7622af9c7e0039..ce17f72ec5a5befe3334ff09f95d2c8e271e62e9 100644 (file)
--- a/src/tools/msvc/Install.pm
+++ b/src/tools/msvc/Install.pm
@@ -408,6 +408,7 @@ sub CopyContribFiles
        next
          if ($insttype eq "client" && !grep { $_ eq $d } @client_contribs);
 
+       # these configuration-based exclusions must match vcregress.pl
        next if ($d eq "uuid-ossp" && !defined($config->{uuid}));
        next if ($d eq "sslinfo"   && !defined($config->{openssl}));
        next if ($d eq "xml2"      && !defined($config->{xml}));

diff --git a/src/tools/msvc/vcregress.pl b/src/tools/msvc/vcregress.pl
index 1b0c9d4c0f72a8f5432174358d613ac526e60a23..2c8cd50a2f7af8a1e6968266ada4913ed356c6f2 100644 (file)
--- a/src/tools/msvc/vcregress.pl
+++ b/src/tools/msvc/vcregress.pl
@@ -219,8 +219,12 @@ sub contribcheck
    my $mstat = 0;
    foreach my $module (glob("*"))
    {
-       next if ($module eq 'sepgsql');
-       next if ($module eq 'xml2' && !$config->{xml});
+       # these configuration-based exclusions must match Install.pm
+       next if ($module eq "uuid-ossp" && !defined($config->{uuid}));
+       next if ($module eq "sslinfo"   && !defined($config->{openssl}));
+       next if ($module eq "xml2"      && !defined($config->{xml}));
+       next if ($module eq "sepgsql");
+
        next
          unless -d "$module/sql"
              && -d "$module/expected"