[fedfs-utils] [PATCH 2/6] fedfsc: Remove vestigial dash switches for position arguments

Chuck Lever chuck.lever at oracle.com
Fri Oct 23 12:07:17 PDT 2015


Commit 54713f0416e6 missed a couple of places where dash switches
were replaced with the use of positional command line arguments.

Coverity: 126105

Fixes: 54713f0416e6 ("fedfsc: Specify manadatory arguments . . .")
Signed-off-by: Chuck Lever <chuck.lever at oracle.com>
---
 doc/man/fedfs-lookup-replication.8.in |    5 -----
 src/fedfsc/fedfs-lookup-replication.c |    4 ----
 2 files changed, 9 deletions(-)

diff --git a/doc/man/fedfs-lookup-replication.8.in b/doc/man/fedfs-lookup-replication.8.in
index 710e257..0cc3482 100644
--- a/doc/man/fedfs-lookup-replication.8.in
+++ b/doc/man/fedfs-lookup-replication.8.in
@@ -124,11 +124,6 @@ If this option is not specified, the default value is
 See
 .BR rpc (3t)
 for details.
-.IP "\fB\-p, \-\-path=\fIpathname\fP"
-Specifies the location on the remote server where the target FedFS replication
-resides.
-This pathname is relative to the remote server's physical root directory,
-not the remote server's NFS pseudoroot.
 .IP "\fB\-t, \-\-resolvetype=\fItype\fP"
 Specifies the desired type of resolution.  Valid values for
 .I type
diff --git a/src/fedfsc/fedfs-lookup-replication.c b/src/fedfsc/fedfs-lookup-replication.c
index c6f0b2c..c8a1ebf 100644
--- a/src/fedfsc/fedfs-lookup-replication.c
+++ b/src/fedfsc/fedfs-lookup-replication.c
@@ -282,7 +282,6 @@ main(int argc, char **argv)
 	nettype = "netpath";
 	security = "unix";
 	resolvetype = "none";
-	path = NULL;
 	while ((arg = getopt_long(argc, argv, fedfs_lookup_replication_opts,
 				fedfs_lookup_replication_longopts, NULL)) != -1) {
 		switch (arg) {
@@ -295,9 +294,6 @@ main(int argc, char **argv)
 		case 'n':
 			nettype = optarg;
 			break;
-		case 'p':
-			path = optarg;
-			break;
 		case 's':
 			security = optarg;
 			break;




More information about the fedfs-utils-devel mailing list