]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
selftest:Samba4: avoid File::Path 'make_path' in setup_dns_hub_internal()
authorStefan Metzmacher <metze@samba.org>
Thu, 17 Dec 2020 09:42:03 +0000 (10:42 +0100)
committerAndreas Schneider <asn@cryptomilk.org>
Wed, 27 Jan 2021 09:56:29 +0000 (09:56 +0000)
While spliting the build and test stages I hit strange permission
problems, when a parent directory is missing,
which can be avoided by using plain mkdir() on each level.

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
selftest/target/Samba4.pm

index 1ebdf2a548419dc1cd23f332b565bb14f37eabcf..c743de0a981d782e3c57618a1949b7babcd38323 100755 (executable)
@@ -17,7 +17,6 @@ use SocketWrapper;
 use target::Samba;
 use target::Samba3;
 use Archive::Tar;
-use File::Path 'make_path';
 
 sub new($$$$$) {
        my ($classname, $SambaCtx, $bindir, $srcdir, $server_maxtime) = @_;
@@ -281,7 +280,7 @@ sub setup_dns_hub_internal($$$)
        my ($self, $hostname, $prefix) = @_;
        my $STDIN_READER;
 
-       unless(-d $prefix or make_path($prefix, 0777)) {
+       unless(-d $prefix or mkdir($prefix, 0777)) {
                warn("Unable to create $prefix");
                return undef;
        }
@@ -356,6 +355,10 @@ sub setup_dns_hub
 
        my $hostname = "rootdnsforwarder";
 
+       unless(-d $prefix or mkdir($prefix, 0777)) {
+               warn("Unable to create $prefix");
+               return undef;
+       }
        my $env = $self->setup_dns_hub_internal("$hostname", "$prefix/$hostname");
 
        $self->{dns_hub_env} = $env;