Use a simpler method to build a glob than splitpath and catpath
authorRichard Levitte <levitte@openssl.org>
Tue, 2 Feb 2016 18:47:05 +0000 (19:47 +0100)
committerRichard Levitte <levitte@openssl.org>
Tue, 2 Feb 2016 19:07:44 +0000 (20:07 +0100)
It turns out that the combination splitpath() could return an empty
string for the directory part.  This doesn't play well with catdir().

Reviewed-by: Rich Salz <rsalz@openssl.org>
Configure

index 06eca504b59b8c35d4ae17141e284e2210a9bdad..44cb40b7c1c6da0f4532141d711428d42f6c3f63 100755 (executable)
--- a/Configure
+++ b/Configure
@@ -10,7 +10,7 @@
 require 5.000;
 use strict;
 use File::Basename;
-use File::Spec::Functions qw/:DEFAULT abs2rel rel2abs catpath splitpath/;
+use File::Spec::Functions qw/:DEFAULT abs2rel rel2abs/;
 use File::Path qw/make_path/;
 use Cwd qw/:DEFAULT realpath/;
 
@@ -175,8 +175,7 @@ die "erroneous version information in opensslv.h: ",
 
 # Collect target configurations
 
-my ($vol, $dir, $dummy) = splitpath($0);
-my $pattern = catpath($vol, catdir($dir, "Configurations"), "*.conf");
+my $pattern = catfile(dirname($0), "Configurations", "*.conf");
 foreach (sort glob($pattern) ) {
     &read_config($_);
 }