Éric Araujo | a0e92a8 | 2011-07-26 18:01:08 +0200 | [diff] [blame] | 1 | #!/usr/bin/env python3 |
Georg Brandl | a9afb68 | 2010-10-21 12:49:28 +0000 | [diff] [blame] | 2 | import re |
| 3 | import sys |
| 4 | import shutil |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 5 | import os.path |
| 6 | import subprocess |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 7 | |
| 8 | import reindent |
Georg Brandl | a9afb68 | 2010-10-21 12:49:28 +0000 | [diff] [blame] | 9 | import untabify |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 10 | |
| 11 | |
Brett Cannon | 058173e | 2010-07-04 22:05:34 +0000 | [diff] [blame] | 12 | def n_files_str(count): |
| 13 | """Return 'N file(s)' with the proper plurality on 'file'.""" |
| 14 | return "{} file{}".format(count, "s" if count != 1 else "") |
| 15 | |
Florent Xicluna | e4a3380 | 2010-08-09 12:24:20 +0000 | [diff] [blame] | 16 | |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 17 | def status(message, modal=False, info=None): |
| 18 | """Decorator to output status info to stdout.""" |
| 19 | def decorated_fxn(fxn): |
| 20 | def call_fxn(*args, **kwargs): |
| 21 | sys.stdout.write(message + ' ... ') |
| 22 | sys.stdout.flush() |
| 23 | result = fxn(*args, **kwargs) |
| 24 | if not modal and not info: |
| 25 | print("done") |
| 26 | elif info: |
| 27 | print(info(result)) |
| 28 | else: |
Florent Xicluna | e4a3380 | 2010-08-09 12:24:20 +0000 | [diff] [blame] | 29 | print("yes" if result else "NO") |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 30 | return result |
| 31 | return call_fxn |
| 32 | return decorated_fxn |
| 33 | |
Florent Xicluna | e4a3380 | 2010-08-09 12:24:20 +0000 | [diff] [blame] | 34 | |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 35 | @status("Getting the list of files that have been added/changed", |
Georg Brandl | a9afb68 | 2010-10-21 12:49:28 +0000 | [diff] [blame] | 36 | info=lambda x: n_files_str(len(x))) |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 37 | def changed_files(): |
Georg Brandl | a9afb68 | 2010-10-21 12:49:28 +0000 | [diff] [blame] | 38 | """Get the list of changed or added files from the VCS.""" |
| 39 | if os.path.isdir('.hg'): |
Georg Brandl | a9afb68 | 2010-10-21 12:49:28 +0000 | [diff] [blame] | 40 | cmd = 'hg status --added --modified --no-status' |
Georg Brandl | a9afb68 | 2010-10-21 12:49:28 +0000 | [diff] [blame] | 41 | else: |
| 42 | sys.exit('need a checkout to get modified files') |
| 43 | |
| 44 | st = subprocess.Popen(cmd.split(), stdout=subprocess.PIPE) |
Éric Araujo | 1e600dc | 2010-11-22 03:13:47 +0000 | [diff] [blame] | 45 | try: |
| 46 | st.wait() |
Benjamin Peterson | 4177eff | 2011-06-27 18:25:06 -0500 | [diff] [blame] | 47 | return [x.decode().rstrip() for x in st.stdout] |
Éric Araujo | 1e600dc | 2010-11-22 03:13:47 +0000 | [diff] [blame] | 48 | finally: |
| 49 | st.stdout.close() |
Florent Xicluna | e4a3380 | 2010-08-09 12:24:20 +0000 | [diff] [blame] | 50 | |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 51 | |
Brett Cannon | 058173e | 2010-07-04 22:05:34 +0000 | [diff] [blame] | 52 | def report_modified_files(file_paths): |
| 53 | count = len(file_paths) |
| 54 | if count == 0: |
| 55 | return n_files_str(count) |
| 56 | else: |
| 57 | lines = ["{}:".format(n_files_str(count))] |
| 58 | for path in file_paths: |
| 59 | lines.append(" {}".format(path)) |
| 60 | return "\n".join(lines) |
| 61 | |
Florent Xicluna | e4a3380 | 2010-08-09 12:24:20 +0000 | [diff] [blame] | 62 | |
Brett Cannon | 058173e | 2010-07-04 22:05:34 +0000 | [diff] [blame] | 63 | @status("Fixing whitespace", info=report_modified_files) |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 64 | def normalize_whitespace(file_paths): |
| 65 | """Make sure that the whitespace for .py files have been normalized.""" |
| 66 | reindent.makebackup = False # No need to create backups. |
Benjamin Peterson | 4177eff | 2011-06-27 18:25:06 -0500 | [diff] [blame] | 67 | fixed = [path for path in file_paths if path.endswith('.py') and |
| 68 | reindent.check(path)] |
Brett Cannon | 058173e | 2010-07-04 22:05:34 +0000 | [diff] [blame] | 69 | return fixed |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 70 | |
Florent Xicluna | e4a3380 | 2010-08-09 12:24:20 +0000 | [diff] [blame] | 71 | |
Georg Brandl | a9afb68 | 2010-10-21 12:49:28 +0000 | [diff] [blame] | 72 | @status("Fixing C file whitespace", info=report_modified_files) |
| 73 | def normalize_c_whitespace(file_paths): |
| 74 | """Report if any C files """ |
| 75 | fixed = [] |
| 76 | for path in file_paths: |
| 77 | with open(path, 'r') as f: |
| 78 | if '\t' not in f.read(): |
| 79 | continue |
| 80 | untabify.process(path, 8, verbose=False) |
| 81 | fixed.append(path) |
| 82 | return fixed |
| 83 | |
| 84 | |
| 85 | ws_re = re.compile(br'\s+(\r?\n)$') |
| 86 | |
| 87 | @status("Fixing docs whitespace", info=report_modified_files) |
| 88 | def normalize_docs_whitespace(file_paths): |
| 89 | fixed = [] |
| 90 | for path in file_paths: |
| 91 | try: |
| 92 | with open(path, 'rb') as f: |
| 93 | lines = f.readlines() |
| 94 | new_lines = [ws_re.sub(br'\1', line) for line in lines] |
| 95 | if new_lines != lines: |
| 96 | shutil.copyfile(path, path + '.bak') |
| 97 | with open(path, 'wb') as f: |
| 98 | f.writelines(new_lines) |
| 99 | fixed.append(path) |
| 100 | except Exception as err: |
| 101 | print('Cannot fix %s: %s' % (path, err)) |
| 102 | return fixed |
| 103 | |
| 104 | |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 105 | @status("Docs modified", modal=True) |
| 106 | def docs_modified(file_paths): |
Brett Cannon | 058173e | 2010-07-04 22:05:34 +0000 | [diff] [blame] | 107 | """Report if any file in the Doc directory has been changed.""" |
| 108 | return bool(file_paths) |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 109 | |
Florent Xicluna | e4a3380 | 2010-08-09 12:24:20 +0000 | [diff] [blame] | 110 | |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 111 | @status("Misc/ACKS updated", modal=True) |
| 112 | def credit_given(file_paths): |
| 113 | """Check if Misc/ACKS has been changed.""" |
Benjamin Peterson | 058e31e | 2009-01-16 03:54:08 +0000 | [diff] [blame] | 114 | return 'Misc/ACKS' in file_paths |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 115 | |
Florent Xicluna | e4a3380 | 2010-08-09 12:24:20 +0000 | [diff] [blame] | 116 | |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 117 | @status("Misc/NEWS updated", modal=True) |
| 118 | def reported_news(file_paths): |
| 119 | """Check if Misc/NEWS has been changed.""" |
Benjamin Peterson | 058e31e | 2009-01-16 03:54:08 +0000 | [diff] [blame] | 120 | return 'Misc/NEWS' in file_paths |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 121 | |
| 122 | |
| 123 | def main(): |
| 124 | file_paths = changed_files() |
Brett Cannon | 058173e | 2010-07-04 22:05:34 +0000 | [diff] [blame] | 125 | python_files = [fn for fn in file_paths if fn.endswith('.py')] |
| 126 | c_files = [fn for fn in file_paths if fn.endswith(('.c', '.h'))] |
Georg Brandl | a9afb68 | 2010-10-21 12:49:28 +0000 | [diff] [blame] | 127 | doc_files = [fn for fn in file_paths if fn.startswith('Doc')] |
Brett Cannon | 058173e | 2010-07-04 22:05:34 +0000 | [diff] [blame] | 128 | special_files = {'Misc/ACKS', 'Misc/NEWS'} & set(file_paths) |
| 129 | # PEP 8 whitespace rules enforcement. |
| 130 | normalize_whitespace(python_files) |
Georg Brandl | a9afb68 | 2010-10-21 12:49:28 +0000 | [diff] [blame] | 131 | # C rules enforcement. |
| 132 | normalize_c_whitespace(c_files) |
| 133 | # Doc whitespace enforcement. |
| 134 | normalize_docs_whitespace(doc_files) |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 135 | # Docs updated. |
Georg Brandl | a9afb68 | 2010-10-21 12:49:28 +0000 | [diff] [blame] | 136 | docs_modified(doc_files) |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 137 | # Misc/ACKS changed. |
Brett Cannon | 058173e | 2010-07-04 22:05:34 +0000 | [diff] [blame] | 138 | credit_given(special_files) |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 139 | # Misc/NEWS changed. |
Brett Cannon | 058173e | 2010-07-04 22:05:34 +0000 | [diff] [blame] | 140 | reported_news(special_files) |
Christian Heimes | ada8c3b | 2008-03-18 18:26:33 +0000 | [diff] [blame] | 141 | |
| 142 | # Test suite run and passed. |
| 143 | print() |
| 144 | print("Did you run the test suite?") |
| 145 | |
| 146 | |
| 147 | if __name__ == '__main__': |
| 148 | main() |