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:38:05 +0000 (22:38 -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");