diff options
| author | Simon Glass <sjg@chromium.org> | 2020-10-29 21:46:10 -0600 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2020-11-05 09:11:31 -0700 |
| commit | 4a9e578138c60aa22821910e22a6e6289c884363 (patch) | |
| tree | 895bbd531ea17770a32e3bc62827a55079d3565e | |
| parent | 08594d49860c335f39d4f7797cac766ac8b57c56 (diff) | |
| download | u-boot-4a9e578138c60aa22821910e22a6e6289c884363.tar.gz | |
patman: Correct operation of -n
This operation was unfortunately broken by a recent change. It is now
necessary to use -i in addition to -n, if there are errors or warnings in
the patches.
Correct this by always showing the summary information.
Fixes: f3653759758 ("patman: Move main code out to a control module")
Signed-off-by: Simon Glass <sjg@chromium.org>
| -rw-r--r-- | tools/patman/control.py | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/tools/patman/control.py b/tools/patman/control.py index 67e8f397ef..aea9df8c8d 100644 --- a/tools/patman/control.py +++ b/tools/patman/control.py @@ -170,9 +170,8 @@ def send(args): ok = ok and gitutil.CheckSuppressCCConfig() its_a_go = ok or args.ignore_errors - if its_a_go: - email_patches( - col, series, cover_fname, patch_files, args.process_tags, - its_a_go, args.ignore_bad_tags, args.add_maintainers, - args.limit, args.dry_run, args.in_reply_to, args.thread, - args.smtp_server) + email_patches( + col, series, cover_fname, patch_files, args.process_tags, + its_a_go, args.ignore_bad_tags, args.add_maintainers, + args.limit, args.dry_run, args.in_reply_to, args.thread, + args.smtp_server) |