Fix vcregress.pl's ancient misspelling of --max-connections.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 11 May 2021 23:17:07 +0000 (19:17 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 11 May 2021 23:17:07 +0000 (19:17 -0400)
I copied the existing spelling of "--max_connections", but
that's just wrong :-(.  Evidently setting $ENV{MAX_CONNECTIONS}
has never actually worked in this script.  Given the lack of
complaints, it's probably not worth back-patching a fix.

Per buildfarm.

Discussion: https://postgr.es/m/899209.1620759506@sss.pgh.pa.us

src/tools/msvc/vcregress.pl

index 0c2d56a45bea96c17814bbc032de5908f0202a9b..a09b60fc858e56ef16a49630db91f42464ce6d79 100644 (file)
@@ -71,7 +71,7 @@ else
 }
 
 my $maxconn = "";
-$maxconn = "--max_connections=$ENV{MAX_CONNECTIONS}"
+$maxconn = "--max-connections=$ENV{MAX_CONNECTIONS}"
   if $ENV{MAX_CONNECTIONS};
 
 my $temp_config = "";
@@ -109,7 +109,7 @@ sub installcheck_internal
        # for backwards compatibility, "serial" runs the tests in
        # parallel_schedule one by one.
        my $maxconn = $maxconn;
-       $maxconn = "--max_connections=1" if $schedule eq 'serial';
+       $maxconn = "--max-connections=1" if $schedule eq 'serial';
        $schedule = 'parallel' if $schedule eq 'serial';
 
        my @args = (
@@ -141,7 +141,7 @@ sub check
        # for backwards compatibility, "serial" runs the tests in
        # parallel_schedule one by one.
        my $maxconn = $maxconn;
-       $maxconn = "--max_connections=1" if $schedule eq 'serial';
+       $maxconn = "--max-connections=1" if $schedule eq 'serial';
        $schedule = 'parallel' if $schedule eq 'serial';
 
        InstallTemp();