This narrows down spelling errors that are in the commit
subject. In v2, it also provides a subject if the subject
line is missing. The provisional subject is the name of the
patch file, which should provide some context about the patch.

Signed-off-by: Chandan Somani <csom...@redhat.com>
---
 utilities/checkpatch.py | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py
index e5d5029f2..f34e4bf2a 100755
--- a/utilities/checkpatch.py
+++ b/utilities/checkpatch.py
@@ -1024,6 +1024,14 @@ def ovs_checkpatch_file(filename):
     result = ovs_checkpatch_parse(part.get_payload(decode=False), filename,
                                   mail.get('Author', mail['From']),
                                   mail['Commit'])
+    if spellcheck:
+        if not mail['Subject'].strip():
+            mail.replace_header('Subject', sys.argv[-1])
+            print("Subject missing! Your provisional subject is", 
+                  mail['Subject'])
+        if check_spelling(mail['Subject'], False):
+            print("Subject: %s" % mail['Subject'])
+
     ovs_checkpatch_print_result()
     return result
 
-- 
2.26.3

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to