From: Martin Schwenke <martin@meltin.net>
Date: Mon, 12 Aug 2019 06:11:13 +0000 (+1000)
Subject: ctdb-tools: Drop 'o' option from getopts command
X-Git-Tag: tdb-1.4.2~147
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=758962a0d435fa595e3917b860a8fd266d122550;p=thirdparty%2Fsamba.git

ctdb-tools: Drop 'o' option from getopts command

Commit 90de5e0594b9180226b9a13293afe31f18576b3d remove the processing
for this option but forgot to remove it from the getopts command.

Versions of ShellCheck >= 0.4.7 warn on this, so it is worth fixing.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14086
RN: Fix onnode test failure with ShellCheck >= 0.4.7
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
---

diff --git a/ctdb/tools/onnode b/ctdb/tools/onnode
index e143ba2d4d4..d6595fff4aa 100755
--- a/ctdb/tools/onnode
+++ b/ctdb/tools/onnode
@@ -72,7 +72,7 @@ parse_options ()
 {
 	local opt
 
-	while getopts "cf:hno:pqvPi?" opt ; do
+	while getopts "cf:hnpqvPi?" opt ; do
 		case "$opt" in
 		c) current=true ;;
 		f) ctdb_nodes_file="$OPTARG" ;;