perl: use the 'if' module to conditionally load File::Glob
authorRichard Levitte <levitte@openssl.org>
Mon, 30 May 2016 09:20:37 +0000 (11:20 +0200)
committerRichard Levitte <levitte@openssl.org>
Mon, 30 May 2016 09:55:46 +0000 (11:55 +0200)
Trying to use normal perl conditions to conditionally 'use' a perl
module didn't quite work.  Using the 'if' module to do so does work.

Reviewed-by: Andy Polyakov <appro@openssl.org>
Configure
test/recipes/40-test_rehash.t
test/recipes/80-test_ssl_new.t
test/run_tests.pl
util/process_docs.pl

index 9e58fc1..020f2a1 100755 (executable)
--- a/Configure
+++ b/Configure
@@ -14,9 +14,7 @@ use strict;
 use File::Basename;
 use File::Spec::Functions qw/:DEFAULT abs2rel rel2abs/;
 use File::Path qw/mkpath/;
-if ($^O ne "VMS") {
-    use File::Glob qw/glob/;
-}
+use if $^O ne "VMS", 'File::Glob' => qw/glob/;
 
 # see INSTALL for instructions.
 
index 201d1ad..fa05ca0 100644 (file)
@@ -13,9 +13,7 @@ use warnings;
 use File::Spec::Functions;
 use File::Copy;
 use File::Basename;
-if ($^O ne "VMS") {
-    use File::Glob qw/glob/;
-}
+use if $^O ne "VMS", 'File::Glob' => qw/glob/;
 use OpenSSL::Test qw/:DEFAULT bldtop_file/;
 
 setup("test_rehash");
index b9f3fa8..2bce02a 100644 (file)
@@ -12,9 +12,7 @@ use warnings;
 
 use File::Basename;
 use File::Compare qw/compare_text/;
-if ($^O ne "VMS") {
-    use File::Glob qw/glob/;
-}
+use if $^O ne "VMS", 'File::Glob' => qw/glob/;
 
 use OpenSSL::Test qw/:DEFAULT srctop_dir srctop_file/;
 use OpenSSL::Test::Utils qw/disabled alldisabled available_protocols/;
index b108413..6550e0b 100644 (file)
@@ -16,9 +16,7 @@ BEGIN {
 
 use File::Spec::Functions qw/catdir catfile curdir abs2rel rel2abs/;
 use File::Basename;
-if ($^O ne "VMS") {
-    use File::Glob qw/glob/;
-}
+use if $^O ne "VMS", 'File::Glob' => qw/glob/;
 use Test::Harness qw/runtests $switches/;
 
 my $srctop = $ENV{SRCTOP} || $ENV{TOP};
index cf1d326..fe8589b 100644 (file)
@@ -13,9 +13,7 @@ use File::Spec::Functions;
 use File::Basename;
 use File::Copy;
 use File::Path;
-if ($^O ne "VMS") {
-    use File::Glob qw/glob/;
-}
+use if $^O ne "VMS", 'File::Glob' => qw/glob/;
 use Getopt::Long;
 use Pod::Usage;