ctdb-scripts: Do not replace commas with spaces in "smb ports" list

The list changed back to space-separated in commit
93448f4be92d4e018aaf2f9705f0351360b2ed0f, so simplify the code a
little.

Signed-off-by: Martin Schwenke <mschwenke@ddn.com>
Reviewed-by: Volker Lendecke <vl@samba.org>

diff --git a/ctdb/config/events/legacy/50.samba.script b/ctdb/config/events/legacy/50.samba.script
index 11b22abac67dcad7d05e9a44b17772bb576848a3..44ae0a4e63b91b73761c59627943d90ec2eba2c1 100755 (executable)
--- a/ctdb/config/events/legacy/50.samba.script
+++ b/ctdb/config/events/legacy/50.samba.script
@@ -123,8 +123,7 @@ list_samba_shares()
 
 list_samba_ports()
 {
-       testparm_cat --parameter-name="smb ports" |
-               sed -e 's@,@ @g'
+       testparm_cat --parameter-name="smb ports"
 }
 
 ###########################

diff --git a/ctdb/tests/UNIT/eventscripts/stubs/testparm b/ctdb/tests/UNIT/eventscripts/stubs/testparm
index 480326e63ffe3be76085f1f11bce3395c40981f1..0f8216ff8b377179a78354041b48a6eccab8d31a 100755 (executable)
--- a/ctdb/tests/UNIT/eventscripts/stubs/testparm
+++ b/ctdb/tests/UNIT/eventscripts/stubs/testparm
@@ -61,7 +61,7 @@ security)
        exit 0
        ;;
 smb*ports)
-       echo "445, 139"
+       echo "445 139"
        exit 0
        ;;
 ?*) not_implemented "--parameter-name=$parameter" ;;