Guido van Rossum | f06ee5f | 1996-11-27 19:52:01 +0000 | [diff] [blame] | 1 | #! /usr/bin/env python |
Guido van Rossum | 2c4d7e7 | 1991-12-26 12:58:17 +0000 | [diff] [blame] | 2 | |
Guido van Rossum | be5e342 | 1996-09-11 23:34:32 +0000 | [diff] [blame] | 3 | # This script is obsolete -- it is kept for historical purposes only. |
| 4 | # |
Guido van Rossum | 0a60ee1 | 1992-01-01 19:22:09 +0000 | [diff] [blame] | 5 | # Fix Python source files to use the new class definition syntax, i.e., |
Guido van Rossum | be5e342 | 1996-09-11 23:34:32 +0000 | [diff] [blame] | 6 | # the syntax used in Python versions before 0.9.8: |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 7 | # class C() = base(), base(), ...: ... |
Guido van Rossum | be5e342 | 1996-09-11 23:34:32 +0000 | [diff] [blame] | 8 | # is changed to the current syntax: |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 9 | # class C(base, base, ...): ... |
Guido van Rossum | be5e342 | 1996-09-11 23:34:32 +0000 | [diff] [blame] | 10 | # |
Guido van Rossum | 0a60ee1 | 1992-01-01 19:22:09 +0000 | [diff] [blame] | 11 | # The script uses heuristics to find class definitions that usually |
| 12 | # work but occasionally can fail; carefully check the output! |
Guido van Rossum | 2c4d7e7 | 1991-12-26 12:58:17 +0000 | [diff] [blame] | 13 | # |
| 14 | # Command line arguments are files or directories to be processed. |
| 15 | # Directories are searched recursively for files whose name looks |
| 16 | # like a python module. |
| 17 | # Symbolic links are always ignored (except as explicit directory |
| 18 | # arguments). Of course, the original file is kept as a back-up |
| 19 | # (with a "~" attached to its name). |
| 20 | # |
Guido van Rossum | 11e7f62 | 1992-01-01 18:38:09 +0000 | [diff] [blame] | 21 | # Changes made are reported to stdout in a diff-like format. |
| 22 | # |
| 23 | # Undoubtedly you can do this using find and sed or perl, but this is |
Guido van Rossum | 2c4d7e7 | 1991-12-26 12:58:17 +0000 | [diff] [blame] | 24 | # a nice example of Python code that recurses down a directory tree |
| 25 | # and uses regular expressions. Also note several subtleties like |
| 26 | # preserving the file's mode and avoiding to even write a temp file |
| 27 | # when no changes are needed for a file. |
| 28 | # |
Guido van Rossum | 11e7f62 | 1992-01-01 18:38:09 +0000 | [diff] [blame] | 29 | # NB: by changing only the function fixline() you can turn this |
| 30 | # into a program for a different change to Python programs... |
Guido van Rossum | 2c4d7e7 | 1991-12-26 12:58:17 +0000 | [diff] [blame] | 31 | |
| 32 | import sys |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 33 | import re |
Guido van Rossum | b2ac809 | 1992-03-30 11:12:23 +0000 | [diff] [blame] | 34 | import os |
Guido van Rossum | 2c4d7e7 | 1991-12-26 12:58:17 +0000 | [diff] [blame] | 35 | from stat import * |
| 36 | |
| 37 | err = sys.stderr.write |
| 38 | dbg = err |
| 39 | rep = sys.stdout.write |
| 40 | |
| 41 | def main(): |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 42 | bad = 0 |
| 43 | if not sys.argv[1:]: # No arguments |
| 44 | err('usage: ' + sys.argv[0] + ' file-or-directory ...\n') |
| 45 | sys.exit(2) |
| 46 | for arg in sys.argv[1:]: |
| 47 | if os.path.isdir(arg): |
| 48 | if recursedown(arg): bad = 1 |
| 49 | elif os.path.islink(arg): |
| 50 | err(arg + ': will not process symbolic links\n') |
| 51 | bad = 1 |
| 52 | else: |
| 53 | if fix(arg): bad = 1 |
| 54 | sys.exit(bad) |
Guido van Rossum | 2c4d7e7 | 1991-12-26 12:58:17 +0000 | [diff] [blame] | 55 | |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 56 | ispythonprog = re.compile('^[a-zA-Z0-9_]+\.py$') |
Guido van Rossum | 11e7f62 | 1992-01-01 18:38:09 +0000 | [diff] [blame] | 57 | def ispython(name): |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 58 | return ispythonprog.match(name) >= 0 |
Guido van Rossum | 2c4d7e7 | 1991-12-26 12:58:17 +0000 | [diff] [blame] | 59 | |
| 60 | def recursedown(dirname): |
Walter Dörwald | 70a6b49 | 2004-02-12 17:35:32 +0000 | [diff] [blame] | 61 | dbg('recursedown(%r)\n' % (dirname,)) |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 62 | bad = 0 |
| 63 | try: |
| 64 | names = os.listdir(dirname) |
Guido van Rossum | b940e11 | 2007-01-10 16:19:56 +0000 | [diff] [blame] | 65 | except os.error as msg: |
Walter Dörwald | 70a6b49 | 2004-02-12 17:35:32 +0000 | [diff] [blame] | 66 | err('%s: cannot list directory: %r\n' % (dirname, msg)) |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 67 | return 1 |
| 68 | names.sort() |
| 69 | subdirs = [] |
| 70 | for name in names: |
| 71 | if name in (os.curdir, os.pardir): continue |
| 72 | fullname = os.path.join(dirname, name) |
| 73 | if os.path.islink(fullname): pass |
| 74 | elif os.path.isdir(fullname): |
| 75 | subdirs.append(fullname) |
| 76 | elif ispython(name): |
| 77 | if fix(fullname): bad = 1 |
| 78 | for fullname in subdirs: |
| 79 | if recursedown(fullname): bad = 1 |
| 80 | return bad |
Guido van Rossum | 2c4d7e7 | 1991-12-26 12:58:17 +0000 | [diff] [blame] | 81 | |
Guido van Rossum | 2c4d7e7 | 1991-12-26 12:58:17 +0000 | [diff] [blame] | 82 | def fix(filename): |
Walter Dörwald | 70a6b49 | 2004-02-12 17:35:32 +0000 | [diff] [blame] | 83 | ## dbg('fix(%r)\n' % (filename,)) |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 84 | try: |
| 85 | f = open(filename, 'r') |
Guido van Rossum | b940e11 | 2007-01-10 16:19:56 +0000 | [diff] [blame] | 86 | except IOError as msg: |
Walter Dörwald | 70a6b49 | 2004-02-12 17:35:32 +0000 | [diff] [blame] | 87 | err('%s: cannot open: %r\n' % (filename, msg)) |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 88 | return 1 |
| 89 | head, tail = os.path.split(filename) |
| 90 | tempname = os.path.join(head, '@' + tail) |
| 91 | g = None |
| 92 | # If we find a match, we rewind the file and start over but |
| 93 | # now copy everything to a temp file. |
| 94 | lineno = 0 |
| 95 | while 1: |
| 96 | line = f.readline() |
| 97 | if not line: break |
| 98 | lineno = lineno + 1 |
| 99 | while line[-2:] == '\\\n': |
| 100 | nextline = f.readline() |
| 101 | if not nextline: break |
| 102 | line = line + nextline |
| 103 | lineno = lineno + 1 |
| 104 | newline = fixline(line) |
| 105 | if newline != line: |
| 106 | if g is None: |
| 107 | try: |
| 108 | g = open(tempname, 'w') |
Guido van Rossum | b940e11 | 2007-01-10 16:19:56 +0000 | [diff] [blame] | 109 | except IOError as msg: |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 110 | f.close() |
Walter Dörwald | 70a6b49 | 2004-02-12 17:35:32 +0000 | [diff] [blame] | 111 | err('%s: cannot create: %r\n' % (tempname, msg)) |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 112 | return 1 |
| 113 | f.seek(0) |
| 114 | lineno = 0 |
| 115 | rep(filename + ':\n') |
| 116 | continue # restart from the beginning |
Walter Dörwald | 70a6b49 | 2004-02-12 17:35:32 +0000 | [diff] [blame] | 117 | rep(repr(lineno) + '\n') |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 118 | rep('< ' + line) |
| 119 | rep('> ' + newline) |
| 120 | if g is not None: |
| 121 | g.write(newline) |
Guido van Rossum | 11e7f62 | 1992-01-01 18:38:09 +0000 | [diff] [blame] | 122 | |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 123 | # End of file |
| 124 | f.close() |
| 125 | if not g: return 0 # No changes |
Guido van Rossum | 11e7f62 | 1992-01-01 18:38:09 +0000 | [diff] [blame] | 126 | |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 127 | # Finishing touch -- move files |
Guido van Rossum | 2c4d7e7 | 1991-12-26 12:58:17 +0000 | [diff] [blame] | 128 | |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 129 | # First copy the file's mode to the temp file |
| 130 | try: |
| 131 | statbuf = os.stat(filename) |
Guido van Rossum | cd16bf6 | 2007-06-13 18:07:49 +0000 | [diff] [blame] | 132 | os.chmod(tempname, statbuf[ST_MODE] & 0o7777) |
Guido van Rossum | b940e11 | 2007-01-10 16:19:56 +0000 | [diff] [blame] | 133 | except os.error as msg: |
Walter Dörwald | 70a6b49 | 2004-02-12 17:35:32 +0000 | [diff] [blame] | 134 | err('%s: warning: chmod failed (%r)\n' % (tempname, msg)) |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 135 | # Then make a backup of the original file as filename~ |
| 136 | try: |
| 137 | os.rename(filename, filename + '~') |
Guido van Rossum | b940e11 | 2007-01-10 16:19:56 +0000 | [diff] [blame] | 138 | except os.error as msg: |
Walter Dörwald | 70a6b49 | 2004-02-12 17:35:32 +0000 | [diff] [blame] | 139 | err('%s: warning: backup failed (%r)\n' % (filename, msg)) |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 140 | # Now move the temp file to the original file |
| 141 | try: |
| 142 | os.rename(tempname, filename) |
Guido van Rossum | b940e11 | 2007-01-10 16:19:56 +0000 | [diff] [blame] | 143 | except os.error as msg: |
Walter Dörwald | 70a6b49 | 2004-02-12 17:35:32 +0000 | [diff] [blame] | 144 | err('%s: rename failed (%r)\n' % (filename, msg)) |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 145 | return 1 |
| 146 | # Return succes |
| 147 | return 0 |
Guido van Rossum | 2c4d7e7 | 1991-12-26 12:58:17 +0000 | [diff] [blame] | 148 | |
Guido van Rossum | 11e7f62 | 1992-01-01 18:38:09 +0000 | [diff] [blame] | 149 | # This expression doesn't catch *all* class definition headers, |
| 150 | # but it's pretty darn close. |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 151 | classexpr = '^([ \t]*class +[a-zA-Z0-9_]+) *( *) *((=.*)?):' |
| 152 | classprog = re.compile(classexpr) |
Guido van Rossum | 11e7f62 | 1992-01-01 18:38:09 +0000 | [diff] [blame] | 153 | |
| 154 | # Expressions for finding base class expressions. |
Thomas Wouters | 49fd7fa | 2006-04-21 10:40:58 +0000 | [diff] [blame] | 155 | baseexpr = '^ *(.*) *( *) *$' |
| 156 | baseprog = re.compile(baseexpr) |
Guido van Rossum | 11e7f62 | 1992-01-01 18:38:09 +0000 | [diff] [blame] | 157 | |
Guido van Rossum | 11e7f62 | 1992-01-01 18:38:09 +0000 | [diff] [blame] | 158 | def fixline(line): |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 159 | if classprog.match(line) < 0: # No 'class' keyword -- no change |
| 160 | return line |
Guido van Rossum | 11e7f62 | 1992-01-01 18:38:09 +0000 | [diff] [blame] | 161 | |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 162 | (a0, b0), (a1, b1), (a2, b2) = classprog.regs[:3] |
| 163 | # a0, b0 = Whole match (up to ':') |
| 164 | # a1, b1 = First subexpression (up to classname) |
| 165 | # a2, b2 = Second subexpression (=.*) |
| 166 | head = line[:b1] |
| 167 | tail = line[b0:] # Unmatched rest of line |
| 168 | |
| 169 | if a2 == b2: # No base classes -- easy case |
| 170 | return head + ':' + tail |
| 171 | |
| 172 | # Get rid of leading '=' |
| 173 | basepart = line[a2+1:b2] |
| 174 | |
| 175 | # Extract list of base expressions |
Walter Dörwald | aaab30e | 2002-09-11 20:36:02 +0000 | [diff] [blame] | 176 | bases = basepart.split(',') |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 177 | |
| 178 | # Strip trailing '()' from each base expression |
| 179 | for i in range(len(bases)): |
| 180 | if baseprog.match(bases[i]) >= 0: |
| 181 | x1, y1 = baseprog.regs[1] |
| 182 | bases[i] = bases[i][x1:y1] |
| 183 | |
| 184 | # Join the bases back again and build the new line |
Walter Dörwald | aaab30e | 2002-09-11 20:36:02 +0000 | [diff] [blame] | 185 | basepart = ', '.join(bases) |
Tim Peters | 70c4378 | 2001-01-17 08:48:39 +0000 | [diff] [blame] | 186 | |
| 187 | return head + '(' + basepart + '):' + tail |
Guido van Rossum | 11e7f62 | 1992-01-01 18:38:09 +0000 | [diff] [blame] | 188 | |
Andrew M. Kuchling | e236b38 | 2004-08-09 17:27:55 +0000 | [diff] [blame] | 189 | if __name__ == '__main__': |
| 190 | main() |