From: Nick Mathewson Date: Mon, 22 Dec 2014 14:34:55 +0000 (-0500) Subject: Short python script to lint the changes files X-Git-Tag: tor-0.2.6.2-alpha~45 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=8b532a8c81577701c1af75d4641f7acb1ce5fc91;p=thirdparty%2Ftor.git Short python script to lint the changes files --- diff --git a/scripts/maint/lintChanges.py b/scripts/maint/lintChanges.py new file mode 100755 index 0000000000..43f2f21685 --- /dev/null +++ b/scripts/maint/lintChanges.py @@ -0,0 +1,46 @@ +#!/usr/bin/python + +import sys +import re + + + +def lintfile(fname): + have_warned = [] + def warn(s): + if not have_warned: + have_warned.append(1) + print fname,":" + print "\t",s + + m = re.search(r'(\d{3,})', fname) + if m: + bugnum = m.group(1) + else: + bugnum = None + + with open(fname) as f: + contents = f.read() + + if bugnum and bugnum not in contents: + warn("bug number %s does not appear"%bugnum) + + lines = contents.split("\n") + isBug = ("bug" in lines[0] or "fix" in lines[0]) + + contents = " ".join(contents.split()) + + if isBug and not re.search(r'(\d+)', contents): + warn("bugfix does not mention a number") + elif isBug and not re.search(r'Fixes bug (\d+)', contents): + warn("bugfix does not say 'Fixes bug XXX'") + + if re.search(r'[bB]ug (\d+)', contents) and not re.search(r'Bugfix on ', contents): + warn("bugfix does not say 'bugfix on X.Y.Z'") + + +if __name__=='__main__': + for fname in sys.argv[1:]: + if fname.endswith("~"): + continue + lintfile(fname)