diff options
author | Ralph Boehme <slow@samba.org> | 2018-01-09 10:40:41 +0100 |
---|---|---|
committer | Ralph Boehme <slow@samba.org> | 2018-01-10 01:01:24 +0100 |
commit | 584a8ac4aa90707cf353975be0f2ddfe65fb065a (patch) | |
tree | f29465f56f6768d85bdeb429401a674fe1ca1510 /selftest/target | |
parent | 686fc4126dc5b69d34e71f7d014c3c17ba0f649e (diff) | |
download | samba-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-x | selftest/target/Samba3.pm | 5 |
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); |