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(-)
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)