]> granicus.if.org Git - postgresql/commitdiff
Simplify Perl chmod calls
authorPeter Eisentraut <peter_e@gmx.net>
Tue, 25 Aug 2015 13:58:49 +0000 (09:58 -0400)
committerPeter Eisentraut <peter_e@gmx.net>
Fri, 28 Aug 2015 02:48:38 +0000 (22:48 -0400)
The Perl chmod function already takes multiple file arguments, so we
don't need a separate looping function.

src/test/ssl/ServerSetup.pm

index 8c1b517575500f08476f049d021414dc8d4c40a0..a8228b04c0ffe01b5d03c03ffae3d02138111ff6 100644 (file)
@@ -43,20 +43,6 @@ sub copy_files
        }
 }
 
-# Perform chmod on a set of files, taking into account wildcards
-sub chmod_files
-{
-       my $mode = shift;
-       my $file_expr = shift;
-
-       my @all_files = glob $file_expr;
-       foreach my $file_entry (@all_files)
-       {
-               chmod $mode, $file_entry
-                 or die "Could not run chmod with mode $mode on $file_entry";
-       }
-}
-
 sub configure_test_server_for_ssl
 {
        my $tempdir = $_[0];
@@ -82,7 +68,7 @@ sub configure_test_server_for_ssl
 # Copy all server certificates and keys, and client root cert, to the data dir
        copy_files("ssl/server-*.crt", "$tempdir/pgdata");
        copy_files("ssl/server-*.key", "$tempdir/pgdata");
-       chmod_files(0600, "$tempdir/pgdata/server-*.key");
+       chmod(0600, glob "$tempdir/pgdata/server-*.key") or die $!;
        copy_files("ssl/root+client_ca.crt", "$tempdir/pgdata");
        copy_files("ssl/root+client.crl",    "$tempdir/pgdata");