cli: Make 'info' options mutually exclusive

crobinso · crobinso · commit 447202e3063e · 2018-03-06T14:33:15.000-05:00
Doing otherwise doesn't really make much sense
diff --git a/bugzilla.1 b/bugzilla.1
@@ -213,7 +213,7 @@ Get a list of products
 List the components in the given product
 .IP "--component_owners=PRODUCT, -o PRODUCT"
 List components (and their owners)
-.IP "--versions=VERSION, -v VERSION"
+.IP "--versions=PRODUCT, -v PRODUCT"
 List the versions for the given product
 
 
diff --git a/bugzilla/_cli.py b/bugzilla/_cli.py
@@ -346,13 +346,14 @@ def _setup_action_info_parser(subparsers):
         "bugzilla server.")
     p = subparsers.add_parser("info", description=description)
 
-    p.add_argument('-p', '--products', action='store_true',
+    x = p.add_mutually_exclusive_group()
+    x.add_argument('-p', '--products', action='store_true',
             help='Get a list of products')
-    p.add_argument('-c', '--components', metavar="PRODUCT",
+    x.add_argument('-c', '--components', metavar="PRODUCT",
             help='List the components in the given product')
-    p.add_argument('-o', '--component_owners', metavar="PRODUCT",
+    x.add_argument('-o', '--component_owners', metavar="PRODUCT",
             help='List components (and their owners)')
-    p.add_argument('-v', '--versions', metavar="VERSION",
+    x.add_argument('-v', '--versions', metavar="PRODUCT",
             help='List the versions for the given product')
 
 
