From 10fbe42f4fe63fd448bb4289c346aa9614c32c26 Mon Sep 17 00:00:00 2001
From: Hunor Csomortáni <csomh@redhat.com>
Date: Feb 02 2018 10:50:04 +0000
Subject: Make compose-id and repo-url to take one or more values


Using nargs='*' help will show something like:

--repo-url [URL [URL ...]]

which reads like the option taking *zero* or more values.

The above is probably incorrect or at least it does not make
sense from a user point of view, as these options once used
should take one or more space separated values.

Using nargs='+' will fix the above, and help will show the
intended usage:

--repo-url URL [URL ...]

Signed-off-by: Hunor Csomortáni <csomh@redhat.com>

---

diff --git a/pyrpkg/cli.py b/pyrpkg/cli.py
index 279a7c7..b165e64 100644
--- a/pyrpkg/cli.py
+++ b/pyrpkg/cli.py
@@ -1035,7 +1035,7 @@ see API KEY section of copr-cli(1) man page.
                            type=int,
                            help='ODCS composes used. '
                                 'Cannot be used with --signing-intent or --repo-url',
-                           nargs='*')
+                           nargs='+')
         group.add_argument(
                           '--signing-intent',
                           help='Signing intent of the ODCS composes. Cannot be '
@@ -1043,9 +1043,9 @@ see API KEY section of copr-cli(1) man page.
         group.add_argument(
                           '--repo-url',
                           metavar="URL",
-                          help='URL of yum repo file'
+                          help='URLs of yum repo files'
                                'Cannot be used with --signing-intent or --compose-id',
-                          nargs='*')
+                          nargs='+')
 
         self.container_build_parser.add_argument(
             '--target',