From 4a9e578138c60aa22821910e22a6e6289c884363 Mon Sep 17 00:00:00 2001
From: Simon Glass <sjg@chromium.org>
Date: Thu, 29 Oct 2020 21:46:10 -0600
Subject: [PATCH] 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>
---
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 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)
--
2.39.5