classfix.py 5.88 KB
Newer Older
1
#! /usr/bin/env python
Guido van Rossum's avatar
Guido van Rossum committed
2

Guido van Rossum's avatar
Guido van Rossum committed
3 4
# This script is obsolete -- it is kept for historical purposes only.
#
Guido van Rossum's avatar
Guido van Rossum committed
5
# Fix Python source files to use the new class definition syntax, i.e.,
Guido van Rossum's avatar
Guido van Rossum committed
6
# the syntax used in Python versions before 0.9.8:
Tim Peters's avatar
Tim Peters committed
7
#       class C() = base(), base(), ...: ...
Guido van Rossum's avatar
Guido van Rossum committed
8
# is changed to the current syntax:
Tim Peters's avatar
Tim Peters committed
9
#       class C(base, base, ...): ...
Guido van Rossum's avatar
Guido van Rossum committed
10
#
Guido van Rossum's avatar
Guido van Rossum committed
11 12
# The script uses heuristics to find class definitions that usually
# work but occasionally can fail; carefully check the output!
Guido van Rossum's avatar
Guido van Rossum committed
13 14 15 16 17 18 19 20
#
# Command line arguments are files or directories to be processed.
# Directories are searched recursively for files whose name looks
# like a python module.
# Symbolic links are always ignored (except as explicit directory
# arguments).  Of course, the original file is kept as a back-up
# (with a "~" attached to its name).
#
21 22 23
# Changes made are reported to stdout in a diff-like format.
#
# Undoubtedly you can do this using find and sed or perl, but this is
Guido van Rossum's avatar
Guido van Rossum committed
24 25 26 27 28
# a nice example of Python code that recurses down a directory tree
# and uses regular expressions.  Also note several subtleties like
# preserving the file's mode and avoiding to even write a temp file
# when no changes are needed for a file.
#
29 30
# NB: by changing only the function fixline() you can turn this
# into a program for a different change to Python programs...
Guido van Rossum's avatar
Guido van Rossum committed
31 32

import sys
33
import regex
34
import os
Guido van Rossum's avatar
Guido van Rossum committed
35 36 37 38 39 40 41
from stat import *

err = sys.stderr.write
dbg = err
rep = sys.stdout.write

def main():
Tim Peters's avatar
Tim Peters committed
42 43 44 45 46 47 48 49 50 51 52 53 54
    bad = 0
    if not sys.argv[1:]: # No arguments
        err('usage: ' + sys.argv[0] + ' file-or-directory ...\n')
        sys.exit(2)
    for arg in sys.argv[1:]:
        if os.path.isdir(arg):
            if recursedown(arg): bad = 1
        elif os.path.islink(arg):
            err(arg + ': will not process symbolic links\n')
            bad = 1
        else:
            if fix(arg): bad = 1
    sys.exit(bad)
Guido van Rossum's avatar
Guido van Rossum committed
55

56 57
ispythonprog = regex.compile('^[a-zA-Z0-9_]+\.py$')
def ispython(name):
Tim Peters's avatar
Tim Peters committed
58
    return ispythonprog.match(name) >= 0
Guido van Rossum's avatar
Guido van Rossum committed
59 60

def recursedown(dirname):
Tim Peters's avatar
Tim Peters committed
61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80
    dbg('recursedown(' + `dirname` + ')\n')
    bad = 0
    try:
        names = os.listdir(dirname)
    except os.error, msg:
        err(dirname + ': cannot list directory: ' + `msg` + '\n')
        return 1
    names.sort()
    subdirs = []
    for name in names:
        if name in (os.curdir, os.pardir): continue
        fullname = os.path.join(dirname, name)
        if os.path.islink(fullname): pass
        elif os.path.isdir(fullname):
            subdirs.append(fullname)
        elif ispython(name):
            if fix(fullname): bad = 1
    for fullname in subdirs:
        if recursedown(fullname): bad = 1
    return bad
Guido van Rossum's avatar
Guido van Rossum committed
81 82

def fix(filename):
83
##  dbg('fix(' + `filename` + ')\n')
Tim Peters's avatar
Tim Peters committed
84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148
    try:
        f = open(filename, 'r')
    except IOError, msg:
        err(filename + ': cannot open: ' + `msg` + '\n')
        return 1
    head, tail = os.path.split(filename)
    tempname = os.path.join(head, '@' + tail)
    g = None
    # If we find a match, we rewind the file and start over but
    # now copy everything to a temp file.
    lineno = 0
    while 1:
        line = f.readline()
        if not line: break
        lineno = lineno + 1
        while line[-2:] == '\\\n':
            nextline = f.readline()
            if not nextline: break
            line = line + nextline
            lineno = lineno + 1
        newline = fixline(line)
        if newline != line:
            if g is None:
                try:
                    g = open(tempname, 'w')
                except IOError, msg:
                    f.close()
                    err(tempname+': cannot create: '+\
                        `msg`+'\n')
                    return 1
                f.seek(0)
                lineno = 0
                rep(filename + ':\n')
                continue # restart from the beginning
            rep(`lineno` + '\n')
            rep('< ' + line)
            rep('> ' + newline)
        if g is not None:
            g.write(newline)

    # End of file
    f.close()
    if not g: return 0 # No changes

    # Finishing touch -- move files

    # First copy the file's mode to the temp file
    try:
        statbuf = os.stat(filename)
        os.chmod(tempname, statbuf[ST_MODE] & 07777)
    except os.error, msg:
        err(tempname + ': warning: chmod failed (' + `msg` + ')\n')
    # Then make a backup of the original file as filename~
    try:
        os.rename(filename, filename + '~')
    except os.error, msg:
        err(filename + ': warning: backup failed (' + `msg` + ')\n')
    # Now move the temp file to the original file
    try:
        os.rename(tempname, filename)
    except os.error, msg:
        err(filename + ': rename failed (' + `msg` + ')\n')
        return 1
    # Return succes
    return 0
Guido van Rossum's avatar
Guido van Rossum committed
149

150 151 152 153 154 155 156 157 158 159 160 161
# This expression doesn't catch *all* class definition headers,
# but it's pretty darn close.
classexpr = '^\([ \t]*class +[a-zA-Z0-9_]+\) *( *) *\(\(=.*\)?\):'
classprog = regex.compile(classexpr)

# Expressions for finding base class expressions.
baseexpr = '^ *\(.*\) *( *) *$'
baseprog = regex.compile(baseexpr)

import string

def fixline(line):
Tim Peters's avatar
Tim Peters committed
162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190
    if classprog.match(line) < 0: # No 'class' keyword -- no change
        return line

    (a0, b0), (a1, b1), (a2, b2) = classprog.regs[:3]
    # a0, b0 = Whole match (up to ':')
    # a1, b1 = First subexpression (up to classname)
    # a2, b2 = Second subexpression (=.*)
    head = line[:b1]
    tail = line[b0:] # Unmatched rest of line

    if a2 == b2: # No base classes -- easy case
        return head + ':' + tail

    # Get rid of leading '='
    basepart = line[a2+1:b2]

    # Extract list of base expressions
    bases = string.splitfields(basepart, ',')

    # Strip trailing '()' from each base expression
    for i in range(len(bases)):
        if baseprog.match(bases[i]) >= 0:
            x1, y1 = baseprog.regs[1]
            bases[i] = bases[i][x1:y1]

    # Join the bases back again and build the new line
    basepart = string.joinfields(bases, ', ')

    return head + '(' + basepart + '):' + tail
191

Guido van Rossum's avatar
Guido van Rossum committed
192
main()