Python 3 preparation: use open() instead of file().

Tool-assisted patch.  Note one case where a simple substitution did not
suffice because the change was in a class that defined its own open()
method earlier, and thus needed to use builtins.open (respectively
__builtin__.open in Python 2).

Author: jsm28

roundup/cgi/TAL/talgettext.py

@@ -285,7 +285,7 @@ def write(self, s):
     if (outfile is None):
         outfile = sys.stdout
     else:
-        outfile = file(outfile, update_mode and "a" or "w")
+        outfile = open(outfile, update_mode and "a" or "w")
 
     catalog = {}
     for domain in engine.catalog.keys():

roundup/configuration.py

@@ -1437,7 +1437,7 @@ def save(self, ini_file=None):
         _tmp_file = os.path.splitext(ini_file)[0]
         _bak_file = _tmp_file + ".bak"
         _tmp_file = _tmp_file + ".tmp"
-        _fp = file(_tmp_file, "wt")
+        _fp = open(_tmp_file, "wt")
         _fp.write("# %s configuration file\n" % self._get_name())
         _fp.write("# Autogenerated at %s\n" % time.asctime())
         need_set = self._get_unset_options()

roundup/instance.py

@@ -28,6 +28,11 @@
 """
 __docformat__ = 'restructuredtext'
 
+try:
+    import builtins
+except ImportError:
+    import __builtin__ as builtins
+
 import os
 import sys
 import warnings
@@ -80,7 +85,7 @@ def __init__(self, tracker_home, optimize=0):
 update your config.ini
 """
             try:
-                with file(filename) as backend_file:
+                with open(filename) as backend_file:
                     rdbms_backend = backend_file.readline().strip()
 
                 with warnings.catch_warnings():
@@ -224,7 +229,7 @@ def nuke(self):
 
     def _compile(self, fname):
         fname = os.path.join(self.tracker_home, fname)
-        return compile(file(fname).read(), fname, 'exec')
+        return compile(builtins.open(fname).read(), fname, 'exec')
 
     def _exec(self, obj, env):
         if self.libdir: