Mercurial > cpython-lru_cache-weakref
annotate Tools/scripts/patchcheck.py @ 71672:bea11ce24bb0 2.7
Let “make patchcheck” work for out-of-dir builds (#9860)
| author | Éric Araujo <merwok@netwok.org> |
|---|---|
| date | Sat, 30 Jul 2011 21:34:04 +0200 |
| parents | 7fb0abf928e0 |
| children | 11662a6e34df |
| rev | line source |
|---|---|
|
71588
7fb0abf928e0
Fix missing or shebangs and executable bits for scripts (#10318)
Éric Araujo <merwok@netwok.org>
parents:
66460
diff
changeset
|
1 #!/usr/bin/env python |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
2 import re |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
3 import sys |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
4 import shutil |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
5 import os.path |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
6 import subprocess |
|
71672
bea11ce24bb0
Let “make patchcheck” work for out-of-dir builds (#9860)
Éric Araujo <merwok@netwok.org>
parents:
71588
diff
changeset
|
7 import sysconfig |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
8 |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
9 import reindent |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
10 import untabify |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
11 |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
12 |
|
71672
bea11ce24bb0
Let “make patchcheck” work for out-of-dir builds (#9860)
Éric Araujo <merwok@netwok.org>
parents:
71588
diff
changeset
|
13 SRCDIR = sysconfig.get_config_var('srcdir') |
|
bea11ce24bb0
Let “make patchcheck” work for out-of-dir builds (#9860)
Éric Araujo <merwok@netwok.org>
parents:
71588
diff
changeset
|
14 |
|
bea11ce24bb0
Let “make patchcheck” work for out-of-dir builds (#9860)
Éric Araujo <merwok@netwok.org>
parents:
71588
diff
changeset
|
15 |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
16 def n_files_str(count): |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
17 """Return 'N file(s)' with the proper plurality on 'file'.""" |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
18 return "{} file{}".format(count, "s" if count != 1 else "") |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
19 |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
20 |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
21 def status(message, modal=False, info=None): |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
22 """Decorator to output status info to stdout.""" |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
23 def decorated_fxn(fxn): |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
24 def call_fxn(*args, **kwargs): |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
25 sys.stdout.write(message + ' ... ') |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
26 sys.stdout.flush() |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
27 result = fxn(*args, **kwargs) |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
28 if not modal and not info: |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
29 print "done" |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
30 elif info: |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
31 print info(result) |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
32 else: |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
33 print "yes" if result else "NO" |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
34 return result |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
35 return call_fxn |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
36 return decorated_fxn |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
37 |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
38 |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
39 @status("Getting the list of files that have been added/changed", |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
40 info=lambda x: n_files_str(len(x))) |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
41 def changed_files(): |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
42 """Get the list of changed or added files from the VCS.""" |
|
71672
bea11ce24bb0
Let “make patchcheck” work for out-of-dir builds (#9860)
Éric Araujo <merwok@netwok.org>
parents:
71588
diff
changeset
|
43 if os.path.isdir(os.path.join(SRCDIR, '.hg')): |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
44 vcs = 'hg' |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
45 cmd = 'hg status --added --modified --no-status' |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
46 elif os.path.isdir('.svn'): |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
47 vcs = 'svn' |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
48 cmd = 'svn status --quiet --non-interactive --ignore-externals' |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
49 else: |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
50 sys.exit('need a checkout to get modified files') |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
51 |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
52 st = subprocess.Popen(cmd.split(), stdout=subprocess.PIPE) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
53 try: |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
54 st.wait() |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
55 if vcs == 'hg': |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
56 return [x.decode().rstrip() for x in st.stdout] |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
57 else: |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
58 output = (x.decode().rstrip().rsplit(None, 1)[-1] |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
59 for x in st.stdout if x[0] in 'AM') |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
60 return set(path for path in output if os.path.isfile(path)) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
61 finally: |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
62 st.stdout.close() |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
63 |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
64 |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
65 def report_modified_files(file_paths): |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
66 count = len(file_paths) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
67 if count == 0: |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
68 return n_files_str(count) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
69 else: |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
70 lines = ["{}:".format(n_files_str(count))] |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
71 for path in file_paths: |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
72 lines.append(" {}".format(path)) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
73 return "\n".join(lines) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
74 |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
75 |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
76 @status("Fixing whitespace", info=report_modified_files) |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
77 def normalize_whitespace(file_paths): |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
78 """Make sure that the whitespace for .py files have been normalized.""" |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
79 reindent.makebackup = False # No need to create backups. |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
80 fixed = [] |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
81 for path in (x for x in file_paths if x.endswith('.py')): |
|
71672
bea11ce24bb0
Let “make patchcheck” work for out-of-dir builds (#9860)
Éric Araujo <merwok@netwok.org>
parents:
71588
diff
changeset
|
82 if reindent.check(os.path.join(SRCDIR, path)): |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
83 fixed.append(path) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
84 return fixed |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
85 |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
86 |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
87 @status("Fixing C file whitespace", info=report_modified_files) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
88 def normalize_c_whitespace(file_paths): |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
89 """Report if any C files """ |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
90 fixed = [] |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
91 for path in file_paths: |
|
71672
bea11ce24bb0
Let “make patchcheck” work for out-of-dir builds (#9860)
Éric Araujo <merwok@netwok.org>
parents:
71588
diff
changeset
|
92 abspath = os.path.join(SRCDIR, path) |
|
bea11ce24bb0
Let “make patchcheck” work for out-of-dir builds (#9860)
Éric Araujo <merwok@netwok.org>
parents:
71588
diff
changeset
|
93 with open(abspath, 'r') as f: |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
94 if '\t' not in f.read(): |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
95 continue |
|
71672
bea11ce24bb0
Let “make patchcheck” work for out-of-dir builds (#9860)
Éric Araujo <merwok@netwok.org>
parents:
71588
diff
changeset
|
96 untabify.process(abspath, 8, verbose=False) |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
97 fixed.append(path) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
98 return fixed |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
99 |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
100 |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
101 ws_re = re.compile(br'\s+(\r?\n)$') |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
102 |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
103 @status("Fixing docs whitespace", info=report_modified_files) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
104 def normalize_docs_whitespace(file_paths): |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
105 fixed = [] |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
106 for path in file_paths: |
|
71672
bea11ce24bb0
Let “make patchcheck” work for out-of-dir builds (#9860)
Éric Araujo <merwok@netwok.org>
parents:
71588
diff
changeset
|
107 abspath = os.path.join(SRCDIR, path) |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
108 try: |
|
71672
bea11ce24bb0
Let “make patchcheck” work for out-of-dir builds (#9860)
Éric Araujo <merwok@netwok.org>
parents:
71588
diff
changeset
|
109 with open(abspath, 'rb') as f: |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
110 lines = f.readlines() |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
111 new_lines = [ws_re.sub(br'\1', line) for line in lines] |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
112 if new_lines != lines: |
|
71672
bea11ce24bb0
Let “make patchcheck” work for out-of-dir builds (#9860)
Éric Araujo <merwok@netwok.org>
parents:
71588
diff
changeset
|
113 shutil.copyfile(abspath, abspath + '.bak') |
|
bea11ce24bb0
Let “make patchcheck” work for out-of-dir builds (#9860)
Éric Araujo <merwok@netwok.org>
parents:
71588
diff
changeset
|
114 with open(abspath, 'wb') as f: |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
115 f.writelines(new_lines) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
116 fixed.append(path) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
117 except Exception as err: |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
118 print 'Cannot fix %s: %s' % (path, err) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
119 return fixed |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
120 |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
121 |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
122 @status("Docs modified", modal=True) |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
123 def docs_modified(file_paths): |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
124 """Report if any file in the Doc directory has been changed.""" |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
125 return bool(file_paths) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
126 |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
127 |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
128 @status("Misc/ACKS updated", modal=True) |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
129 def credit_given(file_paths): |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
130 """Check if Misc/ACKS has been changed.""" |
|
50786
2d453b221bec
Merged revisions 68292,68344,68361,68378,68424,68426,68429-68430,68450,68457,68480-68481,68493,68495,68499,68501,68512,68514-68515 via svnmerge from
Georg Brandl <georg@python.org>
parents:
45859
diff
changeset
|
131 return 'Misc/ACKS' in file_paths |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
132 |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
133 |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
134 @status("Misc/NEWS updated", modal=True) |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
135 def reported_news(file_paths): |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
136 """Check if Misc/NEWS has been changed.""" |
|
50786
2d453b221bec
Merged revisions 68292,68344,68361,68378,68424,68426,68429-68430,68450,68457,68480-68481,68493,68495,68499,68501,68512,68514-68515 via svnmerge from
Georg Brandl <georg@python.org>
parents:
45859
diff
changeset
|
137 return 'Misc/NEWS' in file_paths |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
138 |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
139 |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
140 def main(): |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
141 file_paths = changed_files() |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
142 python_files = [fn for fn in file_paths if fn.endswith('.py')] |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
143 c_files = [fn for fn in file_paths if fn.endswith(('.c', '.h'))] |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
144 doc_files = [fn for fn in file_paths if fn.startswith('Doc')] |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
145 special_files = {'Misc/ACKS', 'Misc/NEWS'} & set(file_paths) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
146 # PEP 8 whitespace rules enforcement. |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
147 normalize_whitespace(python_files) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
148 # C rules enforcement. |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
149 normalize_c_whitespace(c_files) |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
150 # Doc whitespace enforcement. |
|
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
151 normalize_docs_whitespace(doc_files) |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
152 # Docs updated. |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
153 docs_modified(doc_files) |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
154 # Misc/ACKS changed. |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
155 credit_given(special_files) |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
156 # Misc/NEWS changed. |
|
66460
2955f337969e
Merged revisions 85766-85767 via svnmerge from
Georg Brandl <georg@python.org>
parents:
50786
diff
changeset
|
157 reported_news(special_files) |
|
45859
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
158 |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
159 # Test suite run and passed. |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
160 print |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
161 print "Did you run the test suite?" |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
162 |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
163 |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
164 if __name__ == '__main__': |
|
c424099b4ce4
Add Tools/scripts/patchcheck.py. Invoked from ``make check``, the script does
Brett Cannon <bcannon@gmail.com>
parents:
diff
changeset
|
165 main() |
