| 1 | #! /usr/bin/env python | 
|---|
| 2 |  | 
|---|
| 3 | """cleanfuture [-d][-r][-v] path ... | 
|---|
| 4 |  | 
|---|
| 5 | -d  Dry run.  Analyze, but don't make any changes to, files. | 
|---|
| 6 | -r  Recurse.  Search for all .py files in subdirectories too. | 
|---|
| 7 | -v  Verbose.  Print informative msgs. | 
|---|
| 8 |  | 
|---|
| 9 | Search Python (.py) files for future statements, and remove the features | 
|---|
| 10 | from such statements that are already mandatory in the version of Python | 
|---|
| 11 | you're using. | 
|---|
| 12 |  | 
|---|
| 13 | Pass one or more file and/or directory paths.  When a directory path, all | 
|---|
| 14 | .py files within the directory will be examined, and, if the -r option is | 
|---|
| 15 | given, likewise recursively for subdirectories. | 
|---|
| 16 |  | 
|---|
| 17 | Overwrites files in place, renaming the originals with a .bak extension. If | 
|---|
| 18 | cleanfuture finds nothing to change, the file is left alone.  If cleanfuture | 
|---|
| 19 | does change a file, the changed file is a fixed-point (i.e., running | 
|---|
| 20 | cleanfuture on the resulting .py file won't change it again, at least not | 
|---|
| 21 | until you try it again with a later Python release). | 
|---|
| 22 |  | 
|---|
| 23 | Limitations:  You can do these things, but this tool won't help you then: | 
|---|
| 24 |  | 
|---|
| 25 | + A future statement cannot be mixed with any other statement on the same | 
|---|
| 26 | physical line (separated by semicolon). | 
|---|
| 27 |  | 
|---|
| 28 | + A future statement cannot contain an "as" clause. | 
|---|
| 29 |  | 
|---|
| 30 | Example:  Assuming you're using Python 2.2, if a file containing | 
|---|
| 31 |  | 
|---|
| 32 | from __future__ import nested_scopes, generators | 
|---|
| 33 |  | 
|---|
| 34 | is analyzed by cleanfuture, the line is rewritten to | 
|---|
| 35 |  | 
|---|
| 36 | from __future__ import generators | 
|---|
| 37 |  | 
|---|
| 38 | because nested_scopes is no longer optional in 2.2 but generators is. | 
|---|
| 39 | """ | 
|---|
| 40 |  | 
|---|
| 41 | import __future__ | 
|---|
| 42 | import tokenize | 
|---|
| 43 | import os | 
|---|
| 44 | import sys | 
|---|
| 45 |  | 
|---|
| 46 | dryrun  = 0 | 
|---|
| 47 | recurse = 0 | 
|---|
| 48 | verbose = 0 | 
|---|
| 49 |  | 
|---|
| 50 | def errprint(*args): | 
|---|
| 51 | strings = map(str, args) | 
|---|
| 52 | msg = ' '.join(strings) | 
|---|
| 53 | if msg[-1:] != '\n': | 
|---|
| 54 | msg += '\n' | 
|---|
| 55 | sys.stderr.write(msg) | 
|---|
| 56 |  | 
|---|
| 57 | def main(): | 
|---|
| 58 | import getopt | 
|---|
| 59 | global verbose, recurse, dryrun | 
|---|
| 60 | try: | 
|---|
| 61 | opts, args = getopt.getopt(sys.argv[1:], "drv") | 
|---|
| 62 | except getopt.error, msg: | 
|---|
| 63 | errprint(msg) | 
|---|
| 64 | return | 
|---|
| 65 | for o, a in opts: | 
|---|
| 66 | if o == '-d': | 
|---|
| 67 | dryrun += 1 | 
|---|
| 68 | elif o == '-r': | 
|---|
| 69 | recurse += 1 | 
|---|
| 70 | elif o == '-v': | 
|---|
| 71 | verbose += 1 | 
|---|
| 72 | if not args: | 
|---|
| 73 | errprint("Usage:", __doc__) | 
|---|
| 74 | return | 
|---|
| 75 | for arg in args: | 
|---|
| 76 | check(arg) | 
|---|
| 77 |  | 
|---|
| 78 | def check(file): | 
|---|
| 79 | if os.path.isdir(file) and not os.path.islink(file): | 
|---|
| 80 | if verbose: | 
|---|
| 81 | print "listing directory", file | 
|---|
| 82 | names = os.listdir(file) | 
|---|
| 83 | for name in names: | 
|---|
| 84 | fullname = os.path.join(file, name) | 
|---|
| 85 | if ((recurse and os.path.isdir(fullname) and | 
|---|
| 86 | not os.path.islink(fullname)) | 
|---|
| 87 | or name.lower().endswith(".py")): | 
|---|
| 88 | check(fullname) | 
|---|
| 89 | return | 
|---|
| 90 |  | 
|---|
| 91 | if verbose: | 
|---|
| 92 | print "checking", file, "...", | 
|---|
| 93 | try: | 
|---|
| 94 | f = open(file) | 
|---|
| 95 | except IOError, msg: | 
|---|
| 96 | errprint("%r: I/O Error: %s" % (file, str(msg))) | 
|---|
| 97 | return | 
|---|
| 98 |  | 
|---|
| 99 | ff = FutureFinder(f, file) | 
|---|
| 100 | changed = ff.run() | 
|---|
| 101 | if changed: | 
|---|
| 102 | ff.gettherest() | 
|---|
| 103 | f.close() | 
|---|
| 104 | if changed: | 
|---|
| 105 | if verbose: | 
|---|
| 106 | print "changed." | 
|---|
| 107 | if dryrun: | 
|---|
| 108 | print "But this is a dry run, so leaving it alone." | 
|---|
| 109 | for s, e, line in changed: | 
|---|
| 110 | print "%r lines %d-%d" % (file, s+1, e+1) | 
|---|
| 111 | for i in range(s, e+1): | 
|---|
| 112 | print ff.lines[i], | 
|---|
| 113 | if line is None: | 
|---|
| 114 | print "-- deleted" | 
|---|
| 115 | else: | 
|---|
| 116 | print "-- change to:" | 
|---|
| 117 | print line, | 
|---|
| 118 | if not dryrun: | 
|---|
| 119 | bak = file + ".bak" | 
|---|
| 120 | if os.path.exists(bak): | 
|---|
| 121 | os.remove(bak) | 
|---|
| 122 | os.rename(file, bak) | 
|---|
| 123 | if verbose: | 
|---|
| 124 | print "renamed", file, "to", bak | 
|---|
| 125 | g = open(file, "w") | 
|---|
| 126 | ff.write(g) | 
|---|
| 127 | g.close() | 
|---|
| 128 | if verbose: | 
|---|
| 129 | print "wrote new", file | 
|---|
| 130 | else: | 
|---|
| 131 | if verbose: | 
|---|
| 132 | print "unchanged." | 
|---|
| 133 |  | 
|---|
| 134 | class FutureFinder: | 
|---|
| 135 |  | 
|---|
| 136 | def __init__(self, f, fname): | 
|---|
| 137 | self.f = f | 
|---|
| 138 | self.fname = fname | 
|---|
| 139 | self.ateof = 0 | 
|---|
| 140 | self.lines = [] # raw file lines | 
|---|
| 141 |  | 
|---|
| 142 | # List of (start_index, end_index, new_line) triples. | 
|---|
| 143 | self.changed = [] | 
|---|
| 144 |  | 
|---|
| 145 | # Line-getter for tokenize. | 
|---|
| 146 | def getline(self): | 
|---|
| 147 | if self.ateof: | 
|---|
| 148 | return "" | 
|---|
| 149 | line = self.f.readline() | 
|---|
| 150 | if line == "": | 
|---|
| 151 | self.ateof = 1 | 
|---|
| 152 | else: | 
|---|
| 153 | self.lines.append(line) | 
|---|
| 154 | return line | 
|---|
| 155 |  | 
|---|
| 156 | def run(self): | 
|---|
| 157 | STRING = tokenize.STRING | 
|---|
| 158 | NL = tokenize.NL | 
|---|
| 159 | NEWLINE = tokenize.NEWLINE | 
|---|
| 160 | COMMENT = tokenize.COMMENT | 
|---|
| 161 | NAME = tokenize.NAME | 
|---|
| 162 | OP = tokenize.OP | 
|---|
| 163 |  | 
|---|
| 164 | changed = self.changed | 
|---|
| 165 | get = tokenize.generate_tokens(self.getline).next | 
|---|
| 166 | type, token, (srow, scol), (erow, ecol), line = get() | 
|---|
| 167 |  | 
|---|
| 168 | # Chew up initial comments and blank lines (if any). | 
|---|
| 169 | while type in (COMMENT, NL, NEWLINE): | 
|---|
| 170 | type, token, (srow, scol), (erow, ecol), line = get() | 
|---|
| 171 |  | 
|---|
| 172 | # Chew up docstring (if any -- and it may be implicitly catenated!). | 
|---|
| 173 | while type is STRING: | 
|---|
| 174 | type, token, (srow, scol), (erow, ecol), line = get() | 
|---|
| 175 |  | 
|---|
| 176 | # Analyze the future stmts. | 
|---|
| 177 | while 1: | 
|---|
| 178 | # Chew up comments and blank lines (if any). | 
|---|
| 179 | while type in (COMMENT, NL, NEWLINE): | 
|---|
| 180 | type, token, (srow, scol), (erow, ecol), line = get() | 
|---|
| 181 |  | 
|---|
| 182 | if not (type is NAME and token == "from"): | 
|---|
| 183 | break | 
|---|
| 184 | startline = srow - 1    # tokenize is one-based | 
|---|
| 185 | type, token, (srow, scol), (erow, ecol), line = get() | 
|---|
| 186 |  | 
|---|
| 187 | if not (type is NAME and token == "__future__"): | 
|---|
| 188 | break | 
|---|
| 189 | type, token, (srow, scol), (erow, ecol), line = get() | 
|---|
| 190 |  | 
|---|
| 191 | if not (type is NAME and token == "import"): | 
|---|
| 192 | break | 
|---|
| 193 | type, token, (srow, scol), (erow, ecol), line = get() | 
|---|
| 194 |  | 
|---|
| 195 | # Get the list of features. | 
|---|
| 196 | features = [] | 
|---|
| 197 | while type is NAME: | 
|---|
| 198 | features.append(token) | 
|---|
| 199 | type, token, (srow, scol), (erow, ecol), line = get() | 
|---|
| 200 |  | 
|---|
| 201 | if not (type is OP and token == ','): | 
|---|
| 202 | break | 
|---|
| 203 | type, token, (srow, scol), (erow, ecol), line = get() | 
|---|
| 204 |  | 
|---|
| 205 | # A trailing comment? | 
|---|
| 206 | comment = None | 
|---|
| 207 | if type is COMMENT: | 
|---|
| 208 | comment = token | 
|---|
| 209 | type, token, (srow, scol), (erow, ecol), line = get() | 
|---|
| 210 |  | 
|---|
| 211 | if type is not NEWLINE: | 
|---|
| 212 | errprint("Skipping file %r; can't parse line %d:\n%s" % | 
|---|
| 213 | (self.fname, srow, line)) | 
|---|
| 214 | return [] | 
|---|
| 215 |  | 
|---|
| 216 | endline = srow - 1 | 
|---|
| 217 |  | 
|---|
| 218 | # Check for obsolete features. | 
|---|
| 219 | okfeatures = [] | 
|---|
| 220 | for f in features: | 
|---|
| 221 | object = getattr(__future__, f, None) | 
|---|
| 222 | if object is None: | 
|---|
| 223 | # A feature we don't know about yet -- leave it in. | 
|---|
| 224 | # They'll get a compile-time error when they compile | 
|---|
| 225 | # this program, but that's not our job to sort out. | 
|---|
| 226 | okfeatures.append(f) | 
|---|
| 227 | else: | 
|---|
| 228 | released = object.getMandatoryRelease() | 
|---|
| 229 | if released is None or released <= sys.version_info: | 
|---|
| 230 | # Withdrawn or obsolete. | 
|---|
| 231 | pass | 
|---|
| 232 | else: | 
|---|
| 233 | okfeatures.append(f) | 
|---|
| 234 |  | 
|---|
| 235 | # Rewrite the line if at least one future-feature is obsolete. | 
|---|
| 236 | if len(okfeatures) < len(features): | 
|---|
| 237 | if len(okfeatures) == 0: | 
|---|
| 238 | line = None | 
|---|
| 239 | else: | 
|---|
| 240 | line = "from __future__ import " | 
|---|
| 241 | line += ', '.join(okfeatures) | 
|---|
| 242 | if comment is not None: | 
|---|
| 243 | line += ' ' + comment | 
|---|
| 244 | line += '\n' | 
|---|
| 245 | changed.append((startline, endline, line)) | 
|---|
| 246 |  | 
|---|
| 247 | # Loop back for more future statements. | 
|---|
| 248 |  | 
|---|
| 249 | return changed | 
|---|
| 250 |  | 
|---|
| 251 | def gettherest(self): | 
|---|
| 252 | if self.ateof: | 
|---|
| 253 | self.therest = '' | 
|---|
| 254 | else: | 
|---|
| 255 | self.therest = self.f.read() | 
|---|
| 256 |  | 
|---|
| 257 | def write(self, f): | 
|---|
| 258 | changed = self.changed | 
|---|
| 259 | assert changed | 
|---|
| 260 | # Prevent calling this again. | 
|---|
| 261 | self.changed = [] | 
|---|
| 262 | # Apply changes in reverse order. | 
|---|
| 263 | changed.reverse() | 
|---|
| 264 | for s, e, line in changed: | 
|---|
| 265 | if line is None: | 
|---|
| 266 | # pure deletion | 
|---|
| 267 | del self.lines[s:e+1] | 
|---|
| 268 | else: | 
|---|
| 269 | self.lines[s:e+1] = [line] | 
|---|
| 270 | f.writelines(self.lines) | 
|---|
| 271 | # Copy over the remainder of the file. | 
|---|
| 272 | if self.therest: | 
|---|
| 273 | f.write(self.therest) | 
|---|
| 274 |  | 
|---|
| 275 | if __name__ == '__main__': | 
|---|
| 276 | main() | 
|---|