summaryrefslogtreecommitdiff
path: root/selftest/target
diff options
context:
space:
mode:
authorRalph Boehme <slow@samba.org>2018-01-09 10:40:41 +0100
committerRalph Boehme <slow@samba.org>2018-01-10 01:01:24 +0100
commit584a8ac4aa90707cf353975be0f2ddfe65fb065a (patch)
treef29465f56f6768d85bdeb429401a674fe1ca1510 /selftest/target
parent686fc4126dc5b69d34e71f7d014c3c17ba0f649e (diff)
downloadsamba-584a8ac4aa90707cf353975be0f2ddfe65fb065a.tar.gz
selftest: split a large system invocation line
Small cleanup for better code readability, no change in behaviour. Signed-off-by: Ralph Boehme <slow@samba.org> Reviewed-by: Andreas Schneider <asn@samba.org>
Diffstat (limited to 'selftest/target')
-rwxr-xr-xselftest/target/Samba3.pm5
1 files changed, 4 insertions, 1 deletions
diff --git a/selftest/target/Samba3.pm b/selftest/target/Samba3.pm
index df96fd18f55..73ac1c3a17e 100755
--- a/selftest/target/Samba3.pm
+++ b/selftest/target/Samba3.pm
@@ -2454,7 +2454,10 @@ sub wait_for_start($$$$$)
}
}
if ($create_builtin_users eq "yes") {
- $ret = system("SELFTEST_WINBINDD_SOCKET_DIR=" . $envvars->{SELFTEST_WINBINDD_SOCKET_DIR} . " " . Samba::bindir_path($self, "net") ." $envvars->{CONFIGURATION} sam createbuiltingroup Users");
+ $cmd = "SELFTEST_WINBINDD_SOCKET_DIR='$envvars->{SELFTEST_WINBINDD_SOCKET_DIR}' ";
+ $cmd .= Samba::bindir_path($self, "net") . " $envvars->{CONFIGURATION} ";
+ $cmd .= "sam createbuiltingroup Users";
+ $ret = system($cmd);
if ($ret != 0) {
print "Failed to create BUILTIN\\Users group\n";
teardown_env($self, $envvars);