summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rwxr-xr-xutils/checkstyle.py12
1 files changed, 9 insertions, 3 deletions
diff --git a/utils/checkstyle.py b/utils/checkstyle.py
index 7edea25f..4a14309f 100755
--- a/utils/checkstyle.py
+++ b/utils/checkstyle.py
@@ -541,7 +541,7 @@ def check_style(top_level, commit):
files = [f for f in files if len([p for p in patterns if fnmatch.fnmatch(os.path.basename(f), p)])]
if len(files) == 0:
print("Commit doesn't touch source files, skipping")
- return
+ return 0
issues = 0
for f in files:
@@ -554,6 +554,8 @@ def check_style(top_level, commit):
print("%u potential style %s detected, please review" % \
(issues, 'issue' if issues == 1 else 'issues'))
+ return issues
+
def extract_revlist(revs):
"""Extract a list of commits on which to operate from a revision or revision
@@ -632,11 +634,15 @@ def main(argv):
revlist = extract_revlist(args.revision_range)
+ issues = 0
for commit in revlist:
- check_style(top_level, commit)
+ issues += check_style(top_level, commit)
print('')
- return 0
+ if issues:
+ return 1
+ else:
+ return 0
if __name__ == '__main__':