[PATCH] Align check-get with other commands operating on patch IDs
Rob Herring
robh at kernel.org
Thu Mar 31 04:40:24 AEDT 2022
check-get doesn't support specifying the project and doesn't support
more than one patch ID like other commands which take patch ID as an
argument. Align check-get with other commands like view and get.
Cc: Olof Johansson <olof at lixom.net>
Signed-off-by: Rob Herring <robh at kernel.org>
---
man/pwclient.1 | 2 +-
pwclient/parser.py | 5 +----
pwclient/shell.py | 4 ++--
3 files changed, 4 insertions(+), 7 deletions(-)
diff --git a/man/pwclient.1 b/man/pwclient.1
index 63bf78e10442..1ff784f0a3e4 100644
--- a/man/pwclient.1
+++ b/man/pwclient.1
@@ -130,7 +130,7 @@ usage: pwclient projects [-h]
.SH OPTIONS 'pwclient check-get'
-usage: pwclient check-get [-f FORMAT] PATCH_ID
+usage: pwclient check-get [--help] [-h] [-p PROJECT] [-f FORMAT] PATCH_ID [PATCH_ID ...]
.TP
\fBPATCH_ID\fR
diff --git a/pwclient/parser.py b/pwclient/parser.py
index 54c9bdd22ff7..1efedd233766 100644
--- a/pwclient/parser.py
+++ b/pwclient/parser.py
@@ -112,12 +112,9 @@ installed locales.
projects_parser.set_defaults(subcmd='projects')
check_get_parser = subparsers.add_parser(
- 'check-get', add_help=False,
+ 'check-get', parents=[hash_parser], conflict_handler='resolve',
help="get checks for a patch"
)
- check_get_parser.add_argument(
- 'patch_id', metavar='PATCH_ID', action='store', type=int,
- help="patch ID")
check_get_parser.add_argument(
'-f', '--format', metavar='FORMAT',
help=("print output in the given format. You can use tags matching "
diff --git a/pwclient/shell.py b/pwclient/shell.py
index 02165badcfa8..762836c3c417 100644
--- a/pwclient/shell.py
+++ b/pwclient/shell.py
@@ -212,9 +212,9 @@ def main(argv=sys.argv[1:]):
commit=args.commit_ref)
elif action == 'check_get':
- patch_id = args.patch_id
format_str = args.format
- checks.action_get(rpc, patch_id, format_str)
+ for patch_id in patch_ids:
+ checks.action_get(rpc, patch_id, format_str)
elif action == 'check_list':
checks.action_list(rpc)
--
2.32.0
More information about the Patchwork
mailing list