| Message ID | 20201030034638.2858999-2-sjg@chromium.org |
|---|---|
| State | Accepted |
| Commit | 4a9e578138c60aa22821910e22a6e6289c884363 |
| Delegated to: | Simon Glass |
| Headers | show |
| Series | patman: Collect review tags and comments from Patchwork | expand |
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>
---
(no changes since v1)
tools/patman/control.py | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)
Applied to u-boot-dm, thanks!
diff --git a/tools/patman/control.py b/tools/patman/control.py index 67e8f397efd..aea9df8c8d1 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)
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> --- (no changes since v1) tools/patman/control.py | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-)