commit d2d136cace7127d8c3b8319f001db660e749ef35
author Simon Glass <sjg@chromium.org> Wed May 06 16:29:08 2020 -0600
committer Simon Glass <sjg@chromium.org> Thu Jun 11 20:52:11 2020 -0600
tree f8831cbe56bb915665892b6d982122042e60defb
parent 9a690c47fa185021110f749e7428ebf67725d9ad

patman: Support warnings in the patch subject

Sometimes checkpatch outputs problems in the patch subject. Add support
for parsing this output and reporting it correctly.

Signed-off-by: Simon Glass <sjg@chromium.org>

tools/patman/checkpatch.py

diff --git a/tools/patman/checkpatch.py b/tools/patman/checkpatch.py
index 5426bb9..5ae450d 100644
--- a/tools/patman/checkpatch.py
+++ b/tools/patman/checkpatch.py
@@ -127,6 +127,7 @@
         warn_match = re_warning.match(line)
         file_match = re_file.match(line)
         check_match = re_check.match(line)
+        subject_match = line.startswith('Subject:')
         if err_match:
             item['msg'] = err_match.group(1)
             item['type'] = 'error'
@@ -139,6 +140,9 @@
         elif file_match:
             item['file'] = file_match.group(1)
             item['line'] = int(file_match.group(2))
+        elif subject_match:
+            item['file'] = '<patch subject>'
+            item['line'] = None
 
     return result
 
@@ -157,7 +161,8 @@
         msg_type = col.Color(col.RED, msg_type)
     elif msg_type == 'check':
         msg_type = col.Color(col.MAGENTA, msg_type)
-    return '%s:%d: %s: %s\n' % (fname, line, msg_type, msg)
+    line_str = '' if line is None else '%d' % line
+    return '%s:%s: %s: %s\n' % (fname, line_str, msg_type, msg)
 
 def CheckPatches(verbose, args):
     '''Run the checkpatch.pl script on each patch'''