Merged revisions 86223-86224,86226,86234 via svnmerge from

svn+ssh://pythondev@svn.python.org/python/branches/py3k

........
  r86223 | eric.araujo | 2010-11-06 00:51:56 +0100 (sam., 06 nov. 2010) | 2 lines

  Always close files in distutils code and tests (#10252).
........
  r86224 | eric.araujo | 2010-11-06 00:58:34 +0100 (sam., 06 nov. 2010) | 2 lines

  Add missing entry for r86223.
........
  r86226 | eric.araujo | 2010-11-06 00:59:32 +0100 (sam., 06 nov. 2010) | 2 lines

  Of course, I forgot one file in r86223.
........
  r86234 | eric.araujo | 2010-11-06 03:10:32 +0100 (sam., 06 nov. 2010) | 2 lines

  Also close file descriptors from os.popen and subprocess.Popen
........

Lib/distutils/ccompiler.py

@@ -794,14 +794,16 @@ class CCompiler:
             library_dirs = []
         fd, fname = tempfile.mkstemp(".c", funcname, text=True)
         f = os.fdopen(fd, "w")
-        for incl in includes:
+        try:
-            f.write("""#include "%s"\n""" % incl)
+            for incl in includes:
-        f.write("""\
+                f.write("""#include "%s"\n""" % incl)
+            f.write("""\
 main (int argc, char **argv) {
     %s();
 }
 """ % funcname)
-        f.close()
+        finally:
+            f.close()
         try:
             objects = self.compile([fname], include_dirs=include_dirs)
         except CompileError:

Lib/distutils/command/bdist_rpm.py

@@ -355,22 +355,26 @@ class bdist_rpm (Command):
             src_rpm, non_src_rpm, spec_path)
 
         out = os.popen(q_cmd)
-        binary_rpms = []
+        try:
-        source_rpm = None
+            binary_rpms = []
-        while 1:
+            source_rpm = None
-            line = out.readline()
+            while 1:
-            if not line:
+                line = out.readline()
-                break
+                if not line:
-            l = string.split(string.strip(line))
+                    break
-            assert(len(l) == 2)
+                l = string.split(string.strip(line))
-            binary_rpms.append(l[1])
+                assert(len(l) == 2)
-            # The source rpm is named after the first entry in the spec file
+                binary_rpms.append(l[1])
-            if source_rpm is None:
+                # The source rpm is named after the first entry in the spec file
-                source_rpm = l[0]
+                if source_rpm is None:
+                    source_rpm = l[0]
 
-        status = out.close()
+            status = out.close()
-        if status:
+            if status:
-            raise DistutilsExecError("Failed to execute: %s" % repr(q_cmd))
+                raise DistutilsExecError("Failed to execute: %s" % repr(q_cmd))
+
+        finally:
+            out.close()
 
         self.spawn(rpm_cmd)
 

Lib/distutils/command/bdist_wininst.py

@@ -356,5 +356,9 @@ class bdist_wininst (Command):
             sfix = ''
 
         filename = os.path.join(directory, "wininst-%.1f%s.exe" % (bv, sfix))
-        return open(filename, "rb").read()
+        f = open(filename, "rb")
+        try:
+            return f.read()
+        finally:
+            f.close()
 # class bdist_wininst

Lib/distutils/command/upload.py

@@ -79,7 +79,11 @@ class upload(PyPIRCCommand):
 
         # Fill in the data - send all the meta-data in case we need to
         # register a new release
-        content = open(filename,'rb').read()
+        f = open(filename,'rb')
+        try:
+            content = f.read()
+        finally:
+            f.close()
         meta = self.distribution.metadata
         data = {
             # action

Lib/distutils/core.py

@@ -216,7 +216,11 @@ def run_setup(script_name, script_args=None, stop_after="run"):
             sys.argv[0] = script_name
             if script_args is not None:
                 sys.argv[1:] = script_args
-            exec open(script_name, 'r').read() in g, l
+            f = open(script_name)
+            try:
+                exec f.read() in g, l
+            finally:
+                f.close()
         finally:
             sys.argv = save_argv
             _setup_stop_after = None

Lib/distutils/cygwinccompiler.py

@@ -382,8 +382,10 @@ def check_config_h():
         # It would probably better to read single lines to search.
         # But we do this only once, and it is fast enough
         f = open(fn)
-        s = f.read()
+        try:
-        f.close()
+            s = f.read()
+        finally:
+            f.close()
 
     except IOError, exc:
         # if we can't read this file, we cannot say it is wrong

Lib/distutils/dist.py

@@ -1101,9 +1101,11 @@ class DistributionMetadata:
     def write_pkg_info(self, base_dir):
         """Write the PKG-INFO file into the release tree.
         """
-        pkg_info = open( os.path.join(base_dir, 'PKG-INFO'), 'w')
+        pkg_info = open(os.path.join(base_dir, 'PKG-INFO'), 'w')
-        self.write_pkg_file(pkg_info)
+        try:
-        pkg_info.close()
+            self.write_pkg_file(pkg_info)
+        finally:
+            pkg_info.close()
 
     def write_pkg_file(self, file):
         """Write the PKG-INFO format data to a file object.

Lib/distutils/emxccompiler.py

@@ -272,8 +272,10 @@ def check_config_h():
         # It would probably better to read single lines to search.
         # But we do this only once, and it is fast enough
         f = open(fn)
-        s = f.read()
+        try:
-        f.close()
+            s = f.read()
+        finally:
+            f.close()
 
     except IOError, exc:
         # if we can't read this file, we cannot say it is wrong
@@ -300,8 +302,10 @@ def get_versions():
     gcc_exe = find_executable('gcc')
     if gcc_exe:
         out = os.popen(gcc_exe + ' -dumpversion','r')
-        out_string = out.read()
+        try:
-        out.close()
+            out_string = out.read()
+        finally:
+            out.close()
         result = re.search('(\d+\.\d+\.\d+)',out_string)
         if result:
             gcc_version = StrictVersion(result.group(1))

Lib/distutils/extension.py

@@ -150,87 +150,96 @@ def read_setup_file (filename):
     file = TextFile(filename,
                     strip_comments=1, skip_blanks=1, join_lines=1,
                     lstrip_ws=1, rstrip_ws=1)
-    extensions = []
+    try:
+        extensions = []
 
-    while 1:
+        while 1:
-        line = file.readline()
+            line = file.readline()
-        if line is None:                # eof
+            if line is None:                # eof
-            break
+                break
-        if _variable_rx.match(line):    # VAR=VALUE, handled in first pass
+            if _variable_rx.match(line):    # VAR=VALUE, handled in first pass
-            continue
-
-        if line[0] == line[-1] == "*":
-            file.warn("'%s' lines not handled yet" % line)
-            continue
-
-        #print "original line: " + line
-        line = expand_makefile_vars(line, vars)
-        words = split_quoted(line)
-        #print "expanded line: " + line
-
-        # NB. this parses a slightly different syntax than the old
-        # makesetup script: here, there must be exactly one extension per
-        # line, and it must be the first word of the line.  I have no idea
-        # why the old syntax supported multiple extensions per line, as
-        # they all wind up being the same.
-
-        module = words[0]
-        ext = Extension(module, [])
-        append_next_word = None
-
-        for word in words[1:]:
-            if append_next_word is not None:
-                append_next_word.append(word)
-                append_next_word = None
                 continue
 
-            suffix = os.path.splitext(word)[1]
+                if line[0] == line[-1] == "*":
-            switch = word[0:2] ; value = word[2:]
+                    file.warn("'%s' lines not handled yet" % line)
+                    continue
 
-            if suffix in (".c", ".cc", ".cpp", ".cxx", ".c++", ".m", ".mm"):
+            #print "original line: " + line
-                # hmm, should we do something about C vs. C++ sources?
+            line = expand_makefile_vars(line, vars)
-                # or leave it up to the CCompiler implementation to
+            words = split_quoted(line)
-                # worry about?
+            #print "expanded line: " + line
-                ext.sources.append(word)
+
-            elif switch == "-I":
+            # NB. this parses a slightly different syntax than the old
-                ext.include_dirs.append(value)
+            # makesetup script: here, there must be exactly one extension per
-            elif switch == "-D":
+            # line, and it must be the first word of the line.  I have no idea
-                equals = string.find(value, "=")
+            # why the old syntax supported multiple extensions per line, as
-                if equals == -1:        # bare "-DFOO" -- no value
+            # they all wind up being the same.
-                    ext.define_macros.append((value, None))
+
-                else:                   # "-DFOO=blah"
+            module = words[0]
-                    ext.define_macros.append((value[0:equals],
+            ext = Extension(module, [])
-                                              value[equals+2:]))
+            append_next_word = None
-            elif switch == "-U":
+
-                ext.undef_macros.append(value)
+            for word in words[1:]:
-            elif switch == "-C":        # only here 'cause makesetup has it!
+                if append_next_word is not None:
-                ext.extra_compile_args.append(word)
+                    append_next_word.append(word)
-            elif switch == "-l":
+                    append_next_word = None
-                ext.libraries.append(value)
+                    continue
-            elif switch == "-L":
+
-                ext.library_dirs.append(value)
+                suffix = os.path.splitext(word)[1]
-            elif switch == "-R":
+                switch = word[0:2] ; value = word[2:]
-                ext.runtime_library_dirs.append(value)
+
-            elif word == "-rpath":
+                if suffix in (".c", ".cc", ".cpp", ".cxx", ".c++", ".m", ".mm"):
-                append_next_word = ext.runtime_library_dirs
+                    # hmm, should we do something about C vs. C++ sources?
-            elif word == "-Xlinker":
+                    # or leave it up to the CCompiler implementation to
-                append_next_word = ext.extra_link_args
+                    # worry about?
-            elif word == "-Xcompiler":
+                    ext.sources.append(word)
-                append_next_word = ext.extra_compile_args
+                elif switch == "-I":
-            elif switch == "-u":
+                    ext.include_dirs.append(value)
-                ext.extra_link_args.append(word)
+                elif switch == "-D":
-                if not value:
+                    equals = string.find(value, "=")
+                    if equals == -1:        # bare "-DFOO" -- no value
+                        ext.define_macros.append((value, None))
+                    else:                   # "-DFOO=blah"
+                        ext.define_macros.append((value[0:equals],
+                                                  value[equals+2:]))
+                elif switch == "-U":
+                    ext.undef_macros.append(value)
+                elif switch == "-C":        # only here 'cause makesetup has it!
+                    ext.extra_compile_args.append(word)
+                elif switch == "-l":
+                    ext.libraries.append(value)
+                elif switch == "-L":
+                    ext.library_dirs.append(value)
+                elif switch == "-R":
+                    ext.runtime_library_dirs.append(value)
+                elif word == "-rpath":
+                    append_next_word = ext.runtime_library_dirs
+                elif word == "-Xlinker":
                     append_next_word = ext.extra_link_args
-            elif suffix in (".a", ".so", ".sl", ".o", ".dylib"):
+                elif word == "-Xcompiler":
-                # NB. a really faithful emulation of makesetup would
+                    append_next_word = ext.extra_compile_args
-                # append a .o file to extra_objects only if it
+                elif switch == "-u":
-                # had a slash in it; otherwise, it would s/.o/.c/
+                    ext.extra_link_args.append(word)
-                # and append it to sources.  Hmmmm.
+                    if not value:
-                ext.extra_objects.append(word)
+                        append_next_word = ext.extra_link_args
-            else:
+                elif word == "-Xcompiler":
-                file.warn("unrecognized argument '%s'" % word)
+                    append_next_word = ext.extra_compile_args
+                elif switch == "-u":
+                    ext.extra_link_args.append(word)
+                    if not value:
+                        append_next_word = ext.extra_link_args
+                elif suffix in (".a", ".so", ".sl", ".o", ".dylib"):
+                    # NB. a really faithful emulation of makesetup would
+                    # append a .o file to extra_objects only if it
+                    # had a slash in it; otherwise, it would s/.o/.c/
+                    # and append it to sources.  Hmmmm.
+                    ext.extra_objects.append(word)
+                else:
+                    file.warn("unrecognized argument '%s'" % word)
 
-        extensions.append(ext)
+            extensions.append(ext)
+    finally:
+        file.close()
 
         #print "module:", module
         #print "source files:", source_files

Lib/distutils/file_util.py

@@ -224,6 +224,8 @@ def write_file (filename, contents):
     sequence of strings without line terminators) to it.
     """
     f = open(filename, "w")
-    for line in contents:
+    try:
-        f.write(line + "\n")
+        for line in contents:
-    f.close()
+            f.write(line + "\n")
+    finally:
+        f.close()

Lib/distutils/msvc9compiler.py

@@ -273,10 +273,12 @@ def query_vcvarsall(version, arch="x86"):
     popen = subprocess.Popen('"%s" %s & set' % (vcvarsall, arch),
                              stdout=subprocess.PIPE,
                              stderr=subprocess.PIPE)
-
+    try:
-    stdout, stderr = popen.communicate()
+        stdout, stderr = popen.communicate()
-    if popen.wait() != 0:
+        if popen.wait() != 0:
-        raise DistutilsPlatformError(stderr.decode("mbcs"))
+            raise DistutilsPlatformError(stderr.decode("mbcs"))
+    finally:
+        popen.close()
 
     stdout = stdout.decode("mbcs")
     for line in stdout.split("\n"):

Lib/distutils/tests/test_build_py.py

@@ -19,11 +19,15 @@ class BuildPyTestCase(support.TempdirManager,
     def _setup_package_data(self):
         sources = self.mkdtemp()
         f = open(os.path.join(sources, "__init__.py"), "w")
-        f.write("# Pretend this is a package.")
+        try:
-        f.close()
+            f.write("# Pretend this is a package.")
+        finally:
+            f.close()
         f = open(os.path.join(sources, "README.txt"), "w")
-        f.write("Info about this package")
+        try:
-        f.close()
+            f.write("Info about this package")
+        finally:
+            f.close()
 
         destination = self.mkdtemp()
 

Lib/distutils/tests/test_build_scripts.py

@@ -71,8 +71,10 @@ class BuildScriptsTestCase(support.TempdirManager,
 
     def write_script(self, dir, name, text):
         f = open(os.path.join(dir, name), "w")
-        f.write(text)
+        try:
-        f.close()
+            f.write(text)
+        finally:
+            f.close()
 
     def test_version_int(self):
         source = self.mkdtemp()

Lib/distutils/tests/test_config.py

@@ -108,8 +108,12 @@ class PyPIRCCommandTestCase(support.TempdirManager,
         self.assertTrue(not os.path.exists(rc))
         cmd._store_pypirc('tarek', 'xxx')
         self.assertTrue(os.path.exists(rc))
-        content = open(rc).read()
+        f = open(rc)
-        self.assertEquals(content, WANTED)
+        try:
+            content = f.read()
+            self.assertEquals(content, WANTED)
+        finally:
+            f.close()
 
 def test_suite():
     return unittest.makeSuite(PyPIRCCommandTestCase)

Lib/distutils/tests/test_core.py

@@ -52,7 +52,11 @@ class CoreTestCase(support.EnvironGuard, unittest.TestCase):
             shutil.rmtree(path)
 
     def write_setup(self, text, path=test.test_support.TESTFN):
-        open(path, "w").write(text)
+        f = open(path, "w")
+        try:
+            f.write(text)
+        finally:
+            f.close()
         return path
 
     def test_run_setup_provides_file(self):

Lib/distutils/tests/test_dir_util.py

@@ -88,8 +88,10 @@ class DirUtilTestCase(support.TempdirManager, unittest.TestCase):
         mkpath(self.target, verbose=0)
         a_file = os.path.join(self.target, 'ok.txt')
         f = open(a_file, 'w')
-        f.write('some content')
+        try:
-        f.close()
+            f.write('some content')
+        finally:
+            f.close()
 
         wanted = ['copying %s -> %s' % (a_file, self.target2)]
         copy_tree(self.target, self.target2, verbose=1)

Lib/distutils/tests/test_dist.py

@@ -102,29 +102,29 @@ class DistributionTestCase(support.TempdirManager,
 
     def test_command_packages_configfile(self):
         sys.argv.append("build")
+        self.addCleanup(os.unlink, TESTFN)
         f = open(TESTFN, "w")
         try:
             print >>f, "[global]"
             print >>f, "command_packages = foo.bar, splat"
-            f.close()
-            d = self.create_distribution([TESTFN])
-            self.assertEqual(d.get_command_packages(),
-                             ["distutils.command", "foo.bar", "splat"])
-
-            # ensure command line overrides config:
-            sys.argv[1:] = ["--command-packages", "spork", "build"]
-            d = self.create_distribution([TESTFN])
-            self.assertEqual(d.get_command_packages(),
-                             ["distutils.command", "spork"])
-
-            # Setting --command-packages to '' should cause the default to
-            # be used even if a config file specified something else:
-            sys.argv[1:] = ["--command-packages", "", "build"]
-            d = self.create_distribution([TESTFN])
-            self.assertEqual(d.get_command_packages(), ["distutils.command"])
-
         finally:
-            os.unlink(TESTFN)
+            f.close()
+
+        d = self.create_distribution([TESTFN])
+        self.assertEqual(d.get_command_packages(),
+                         ["distutils.command", "foo.bar", "splat"])
+
+        # ensure command line overrides config:
+        sys.argv[1:] = ["--command-packages", "spork", "build"]
+        d = self.create_distribution([TESTFN])
+        self.assertEqual(d.get_command_packages(),
+                         ["distutils.command", "spork"])
+
+        # Setting --command-packages to '' should cause the default to
+        # be used even if a config file specified something else:
+        sys.argv[1:] = ["--command-packages", "", "build"]
+        d = self.create_distribution([TESTFN])
+        self.assertEqual(d.get_command_packages(), ["distutils.command"])
 
     def test_write_pkg_file(self):
         # Check DistributionMetadata handling of Unicode fields
@@ -341,8 +341,10 @@ class MetadataTestCase(support.TempdirManager, support.EnvironGuard,
         temp_dir = self.mkdtemp()
         user_filename = os.path.join(temp_dir, user_filename)
         f = open(user_filename, 'w')
-        f.write('.')
+        try:
-        f.close()
+            f.write('.')
+        finally:
+            f.close()
 
         try:
             dist = Distribution()

Lib/distutils/tests/test_file_util.py

@@ -31,8 +31,10 @@ class FileUtilTestCase(support.TempdirManager, unittest.TestCase):
 
     def test_move_file_verbosity(self):
         f = open(self.source, 'w')
-        f.write('some content')
+        try:
-        f.close()
+            f.write('some content')
+        finally:
+            f.close()
 
         move_file(self.source, self.target, verbose=0)
         wanted = []

Lib/distutils/tests/test_install_scripts.py

@@ -42,8 +42,10 @@ class InstallScriptsTestCase(support.TempdirManager,
         def write_script(name, text):
             expected.append(name)
             f = open(os.path.join(source, name), "w")
-            f.write(text)
+            try:
-            f.close()
+                f.write(text)
+            finally:
+                f.close()
 
         write_script("script1.py", ("#! /usr/bin/env python2.3\n"
                                     "# bogus script w/ Python sh-bang\n"

Lib/distutils/tests/test_msvc9compiler.py

@@ -113,17 +113,21 @@ class msvc9compilerTestCase(support.TempdirManager,
         tempdir = self.mkdtemp()
         manifest = os.path.join(tempdir, 'manifest')
         f = open(manifest, 'w')
-        f.write(_MANIFEST)
+        try:
-        f.close()
+            f.write(_MANIFEST)
+        finally:
+            f.close()
 
         compiler = MSVCCompiler()
         compiler._remove_visual_c_ref(manifest)
 
         # see what we got
         f = open(manifest)
-        # removing trailing spaces
+        try:
-        content = '\n'.join([line.rstrip() for line in f.readlines()])
+            # removing trailing spaces
-        f.close()
+            content = '\n'.join([line.rstrip() for line in f.readlines()])
+        finally:
+            f.close()
 
         # makes sure the manifest was properly cleaned
         self.assertEquals(content, _CLEANED_MANIFEST)

Lib/distutils/tests/test_register.py

@@ -119,8 +119,12 @@ class RegisterTestCase(PyPIRCCommandTestCase):
         self.assertTrue(os.path.exists(self.rc))
 
         # with the content similar to WANTED_PYPIRC
-        content = open(self.rc).read()
+        f = open(self.rc)
-        self.assertEquals(content, WANTED_PYPIRC)
+        try:
+            content = f.read()
+            self.assertEquals(content, WANTED_PYPIRC)
+        finally:
+            f.close()
 
         # now let's make sure the .pypirc file generated
         # really works : we shouldn't be asked anything

Lib/distutils/tests/test_sdist.py

@@ -234,8 +234,12 @@ class SDistTestCase(PyPIRCCommandTestCase):
         self.assertEquals(len(content), 11)
 
         # checking the MANIFEST
-        manifest = open(join(self.tmp_dir, 'MANIFEST')).read()
+        f = open(join(self.tmp_dir, 'MANIFEST'))
-        self.assertEquals(manifest, MANIFEST % {'sep': os.sep})
+        try:
+            manifest = f.read()
+            self.assertEquals(manifest, MANIFEST % {'sep': os.sep})
+        finally:
+            f.close()
 
     @unittest.skipUnless(zlib, "requires zlib")
     def test_metadata_check_option(self):

Lib/distutils/tests/test_sysconfig.py

@@ -50,9 +50,11 @@ class SysconfigTestCase(support.EnvironGuard,
     def test_parse_makefile_base(self):
         self.makefile = test.test_support.TESTFN
         fd = open(self.makefile, 'w')
-        fd.write(r"CONFIG_ARGS=  '--arg1=optarg1' 'ENV=LIB'" '\n')
+        try:
-        fd.write('VAR=$OTHER\nOTHER=foo')
+            fd.write(r"CONFIG_ARGS=  '--arg1=optarg1' 'ENV=LIB'" '\n')
-        fd.close()
+            fd.write('VAR=$OTHER\nOTHER=foo')
+        finally:
+            fd.close()
         d = sysconfig.parse_makefile(self.makefile)
         self.assertEquals(d, {'CONFIG_ARGS': "'--arg1=optarg1' 'ENV=LIB'",
                               'OTHER': 'foo'})
@@ -60,9 +62,11 @@ class SysconfigTestCase(support.EnvironGuard,
     def test_parse_makefile_literal_dollar(self):
         self.makefile = test.test_support.TESTFN
         fd = open(self.makefile, 'w')
-        fd.write(r"CONFIG_ARGS=  '--arg1=optarg1' 'ENV=\$$LIB'" '\n')
+        try:
-        fd.write('VAR=$OTHER\nOTHER=foo')
+            fd.write(r"CONFIG_ARGS=  '--arg1=optarg1' 'ENV=\$$LIB'" '\n')
-        fd.close()
+            fd.write('VAR=$OTHER\nOTHER=foo')
+        finally:
+            fd.close()
         d = sysconfig.parse_makefile(self.makefile)
         self.assertEquals(d, {'CONFIG_ARGS': r"'--arg1=optarg1' 'ENV=\$LIB'",
                               'OTHER': 'foo'})

Lib/distutils/tests/test_text_file.py

@@ -58,28 +58,46 @@ class TextFileTestCase(support.TempdirManager, unittest.TestCase):
         finally:
             out_file.close()
 
-        in_file = TextFile (filename, strip_comments=0, skip_blanks=0,
+        in_file = TextFile(filename, strip_comments=0, skip_blanks=0,
-                            lstrip_ws=0, rstrip_ws=0)
+                           lstrip_ws=0, rstrip_ws=0)
-        test_input (1, "no processing", in_file, result1)
+        try:
+            test_input(1, "no processing", in_file, result1)
+        finally:
+            in_file.close()
 
-        in_file = TextFile (filename, strip_comments=1, skip_blanks=0,
+        in_file = TextFile(filename, strip_comments=1, skip_blanks=0,
-                            lstrip_ws=0, rstrip_ws=0)
+                           lstrip_ws=0, rstrip_ws=0)
-        test_input (2, "strip comments", in_file, result2)
+        try:
+            test_input(2, "strip comments", in_file, result2)
+        finally:
+            in_file.close()
 
-        in_file = TextFile (filename, strip_comments=0, skip_blanks=1,
+        in_file = TextFile(filename, strip_comments=0, skip_blanks=1,
-                            lstrip_ws=0, rstrip_ws=0)
+                           lstrip_ws=0, rstrip_ws=0)
-        test_input (3, "strip blanks", in_file, result3)
+        try:
+            test_input(3, "strip blanks", in_file, result3)
+        finally:
+            in_file.close()
 
-        in_file = TextFile (filename)
+        in_file = TextFile(filename)
-        test_input (4, "default processing", in_file, result4)
+        try:
+            test_input(4, "default processing", in_file, result4)
+        finally:
+            in_file.close()
 
-        in_file = TextFile (filename, strip_comments=1, skip_blanks=1,
+        in_file = TextFile(filename, strip_comments=1, skip_blanks=1,
-                            join_lines=1, rstrip_ws=1)
+                           join_lines=1, rstrip_ws=1)
-        test_input (5, "join lines without collapsing", in_file, result5)
+        try:
+            test_input(5, "join lines without collapsing", in_file, result5)
+        finally:
+            in_file.close()
 
-        in_file = TextFile (filename, strip_comments=1, skip_blanks=1,
+        in_file = TextFile(filename, strip_comments=1, skip_blanks=1,
-                            join_lines=1, rstrip_ws=1, collapse_join=1)
+                           join_lines=1, rstrip_ws=1, collapse_join=1)
-        test_input (6, "join lines with collapsing", in_file, result6)
+        try:
+            test_input(6, "join lines with collapsing", in_file, result6)
+        finally:
+            in_file.close()
 
 def test_suite():
     return unittest.makeSuite(TextFileTestCase)

Lib/distutils/util.py

@@ -116,13 +116,15 @@ def get_platform ():
                 # behaviour.
                 pass
             else:
-                m = re.search(
+                try:
-                        r'<key>ProductUserVisibleVersion</key>\s*' +
+                    m = re.search(
-                        r'<string>(.*?)</string>', f.read())
+                            r'<key>ProductUserVisibleVersion</key>\s*' +
-                f.close()
+                            r'<string>(.*?)</string>', f.read())
-                if m is not None:
+                    if m is not None:
-                    macrelease = '.'.join(m.group(1).split('.')[:2])
+                        macrelease = '.'.join(m.group(1).split('.')[:2])
-                # else: fall back to the default behaviour
+                    # else: fall back to the default behaviour
+                finally:
+                    f.close()
 
         if not macver:
             macver = macrelease

Lib/sysconfig.py

@@ -635,13 +635,16 @@ def get_platform():
                 # behaviour.
                 pass
             else:
-                m = re.search(
+                try:
-                        r'<key>ProductUserVisibleVersion</key>\s*' +
+                    m = re.search(
-                        r'<string>(.*?)</string>', f.read())
+                            r'<key>ProductUserVisibleVersion</key>\s*' +
-                f.close()
+                            r'<string>(.*?)</string>', f.read())
-                if m is not None:
+                    f.close()
-                    macrelease = '.'.join(m.group(1).split('.')[:2])
+                    if m is not None:
-                # else: fall back to the default behaviour
+                        macrelease = '.'.join(m.group(1).split('.')[:2])
+                    # else: fall back to the default behaviour
+                finally:
+                    f.close()
 
         if not macver:
             macver = macrelease

Misc/NEWS

@@ -69,6 +69,9 @@ Core and Builtins
 Library
 -------
 
+- Issue #10252: Close file objects in a timely manner in distutils code and
+  tests.  Patch by Brian Brazil, completed by Éric Araujo.
+
 - Issue #10311: The signal module now restores errno before returning from
   its low-level signal handler.  Patch by Hallvard B Furuseth.