[PATCH v2 01/30] patman: Correct operation of -n

Simon Glass sjg at chromium.org
Mon Oct 26 12:04:13 AEDT 2020


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 at 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)
-- 
2.29.0.rc2.309.g374f81d7ae-goog



More information about the Patchwork mailing list