diff options
author | Simon Glass <sjg@chromium.org> | 2021-01-23 08:56:15 -0700 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2021-03-22 19:23:26 +1300 |
commit | 0fb560d9a71e9bd3c36944dda9b6122df63f49a3 (patch) | |
tree | c41f8c64e154129da347962644a01b7a649c11e5 /tools/patman/main.py | |
parent | c7e42cabed3d761f4aa6007135fd126db82a2ae1 (diff) |
patman: Quieten down the alias checking
When a tag is used in a patch subject (e.g. "tag: rest of message") and
it cannot be found as an alias, patman currently reports a fatal error,
unless -t is provided, in which case it reports a warning.
Experience suggest that the fatal error is not very useful. Instead,
default to reporting a warning, with -t tell patman to ignore it
altogether.
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'tools/patman/main.py')
-rwxr-xr-x | tools/patman/main.py | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/tools/patman/main.py b/tools/patman/main.py index c4e4d80d425..4e0a3533e9f 100755 --- a/tools/patman/main.py +++ b/tools/patman/main.py @@ -68,7 +68,8 @@ send.add_argument('-n', '--dry-run', action='store_true', dest='dry_run', send.add_argument('-r', '--in-reply-to', type=str, action='store', help="Message ID that this series is in reply to") send.add_argument('-t', '--ignore-bad-tags', action='store_true', - default=False, help='Ignore bad tags / aliases') + default=False, + help='Ignore bad tags / aliases (default=warn)') send.add_argument('-T', '--thread', action='store_true', dest='thread', default=False, help='Create patches as a single thread') send.add_argument('--cc-cmd', dest='cc_cmd', type=str, action='store', @@ -176,6 +177,9 @@ elif args.cmd == 'send': command.Run(pager, fname) else: + # If we are not processing tags, no need to warning about bad ones + if not args.process_tags: + args.ignore_bad_tags = True control.send(args) # Check status of patches in patchwork |