Expand properly list of TAP tests used for prove in vcregress.pl
authorMichael Paquier <michael@paquier.xyz>
Wed, 11 Sep 2019 02:07:25 +0000 (11:07 +0900)
committerMichael Paquier <michael@paquier.xyz>
Wed, 11 Sep 2019 02:07:25 +0000 (11:07 +0900)
Depending on the system used, t/*.pl may not be expanded into a list of
tests which can be consumed by prove when attempting to run TAP tests on
a given path.  Fix that by using glob() directly in the script, to make
sure that a complete list of tests is provided.  This has not proved to
be an issue with MSVC as the list was properly expanded, but it is on
Linux with perl's system().

This is extracted from a larger patch.

Author: Tom Lane
Discussion: https://postgr.es/m/6628.1567958876@sss.pgh.pa.us
Backpatch-through: 9.4

src/tools/msvc/vcregress.pl

index 5495066b4d98639d07276ec80097c0ab1f322d49..a71300176599d7f381f180b513de4fa047c01880 100644 (file)
@@ -207,7 +207,7 @@ sub tap_check
    my $dir = shift;
    chdir $dir;
 
-   my @args = ("prove", @flags, "t/*.pl");
+   my @args = ("prove", @flags, glob("t/*.pl"));
 
    # adjust the environment for just this test
    local %ENV = %ENV;