[ovs-dev] [PATCH] utilities/ovs-vsctl.c: honor 'help' as ovs-ofctl and dpctl

Aaron Conole aconole at redhat.com
Tue Jan 5 18:58:50 UTC 2016


Currently, ovs-ofctl and ovs-dpctl allow a 'help' keyword, in addition to -h
and --help. However, ovs-vsctl does not honor the same 'help' keyword. This
change adds a 'help' which redirects to usage(), bringing ovs-vsctl in line
with ovs-ofctl and ovs-dpctl.

Signed-off-by: Aaron Conole <aconole at redhat.com>
---
 utilities/ovs-vsctl.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/utilities/ovs-vsctl.c b/utilities/ovs-vsctl.c
index 36290db..5d5983d 100644
--- a/utilities/ovs-vsctl.c
+++ b/utilities/ovs-vsctl.c
@@ -92,7 +92,7 @@ static struct ovsdb_idl *the_idl;
 static struct ovsdb_idl_txn *the_idl_txn;
 OVS_NO_RETURN static void vsctl_exit(int status);
 
-OVS_NO_RETURN static void usage(void);
+OVS_NO_RETURN static void usage(struct ctl_context *);
 static void parse_options(int argc, char *argv[], struct shash *local_options);
 static void run_prerequisites(struct ctl_command[], size_t n_commands,
                               struct ovsdb_idl *);
@@ -292,7 +292,7 @@ parse_options(int argc, char *argv[], struct shash *local_options)
             break;
 
         case 'h':
-            usage();
+            usage(NULL);
 
         case OPT_COMMANDS:
             ctl_print_commands();
@@ -350,7 +350,7 @@ parse_options(int argc, char *argv[], struct shash *local_options)
 }
 
 static void
-usage(void)
+usage(struct ctl_context *ctx OVS_UNUSED)
 {
     printf("\
 %s: ovs-vswitchd management utility\n\
@@ -2701,6 +2701,7 @@ vsctl_exit(int status)
 static const struct ctl_command_syntax vsctl_commands[] = {
     /* Open vSwitch commands. */
     {"init", 0, 0, "", NULL, cmd_init, NULL, "", RW},
+    {"help", 0, 0, "", NULL, usage, NULL, "", RO},
 
     /* Bridge commands. */
     {"add-br", 1, 3, "NEW-BRIDGE [PARENT] [NEW-VLAN]", pre_get_info,
-- 
2.6.1.133.gf5b6079




More information about the dev mailing list