eventscripts; Cleanup up ctdb_check_directories()
authorMartin Schwenke <martin@meltin.net>
Mon, 29 Apr 2013 17:48:51 +0000 (03:48 +1000)
committerAmitay Isaacs <amitay@gmail.com>
Mon, 6 May 2013 05:45:10 +0000 (15:45 +1000)
The documentation comments are wrong... and remove option
$service_name argument.

Signed-off-by: Martin Schwenke <martin@meltin.net>
config/functions

index ba4ca284656ac0a95518a9b85336b657f3fcfa8d..76f94a4de432e3354273fe834441094f7d36d15b 100755 (executable)
@@ -358,9 +358,10 @@ assert_service_name ()
 ######################################################
 # check a set of directories is available
 # return 1 on a missing directory
-# usage: ctdb_check_directories_probe SERVICE_NAME <directories...>
+# directories are read from stdin
 ######################################################
-ctdb_check_directories_probe() {
+ctdb_check_directories_probe()
+{
     while IFS="" read d ; do
        case "$d" in
            *%*)
@@ -374,12 +375,12 @@ ctdb_check_directories_probe() {
 
 ######################################################
 # check a set of directories is available
-# usage: ctdb_check_directories SERVICE_NAME <directories...>
+# directories are read from stdin
 ######################################################
-ctdb_check_directories() {
-    n="${1:-${service_name}}"
+ctdb_check_directories()
+{
     ctdb_check_directories_probe || {
-       echo "ERROR: $n directory \"$d\" not available"
+       echo "ERROR: $service_name directory \"$d\" not available"
        exit 1
     }
 }