debian/rules is a Makefile with a funny name.
Signed-off-by: Ben Pfaff <[email protected]>
---
utilities/checkpatch.py | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py
index 0f389052c1f9..2827103cd23c 100755
--- a/utilities/checkpatch.py
+++ b/utilities/checkpatch.py
@@ -165,13 +165,12 @@ skip_signoff_check = False
# name, as they may have legitimate reasons to have longer lines.
#
# Python isn't checked as flake8 performs these checks during build.
-line_length_blacklist = ['.am', '.at', 'etc', '.in', '.m4', '.mk', '.patch',
- '.py']
+line_length_blacklist =
re.compile(r'\.(am|at|etc|in|m4|mk|patch|py)$|debian/rules')
# Don't enforce a requirement that leading whitespace be all spaces on
# files that include these characters in their name, since these kinds
# of files need lines with leading tabs.
-leading_whitespace_blacklist = ['.mk', '.am', '.at']
+leading_whitespace_blacklist = re.compile(r'\.(mk|am|at)$|debian/rules')
def is_subtracted_line(line):
@@ -379,13 +378,13 @@ def check_comment_spelling(line):
checks = [
{'regex': None,
'match_name':
- lambda x: not any([fmt in x for fmt in line_length_blacklist]),
+ lambda x: not line_length_blacklist.match(x),
'check': lambda x: line_length_check(x),
'print': lambda: print_warning("Line length is >79-characters long")},
{'regex': None,
'match_name':
- lambda x: not any([fmt in x for fmt in leading_whitespace_blacklist]),
+ lambda x: not leading_whitespace_blacklist.match(x),
'check': lambda x: not leading_whitespace_is_spaces(x),
'print': lambda: print_warning("Line has non-spaces leading whitespace")},
--
2.16.1
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
