handle configparser rename in python3

checkout.py

 # coding=utf8
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 """Manages a project checkout.
 
 Includes support for svn, git-svn and git.
 """
 
-import ConfigParser
 import fnmatch
 import logging
 import os
 import re
 import shutil
 import subprocess
 import sys
 import tempfile
 
+# The configparser module was renamed in Python 3.

[nodir, 2016/06/16 18:55:58]

this comment should be inside except clause

[vapier, 2016/06/16 19:13:47]

"configparser" is the py3 name which is in the try block.

that said, i was copying existing style used in the code base.  check out the
upload.py diff and the "# The md5 module was deprecated in Python 2.5." comment.

+try:
+  import configparser
+except ImportError:
+  import ConfigParser as configparser
+
 import patch
 import scm
 import subprocess2
 
 
 if sys.platform in ('cygwin', 'win32'):
   # Disable timeouts on Windows since we can't have shells with timeouts.
   GLOBAL_TIMEOUT = None
   FETCH_TIMEOUT = None
 else:
@@ skipping 221 matching lines @@
     super(SvnConfig, self).__init__()
     self.svn_config_dir = svn_config_dir
     self.default = not bool(self.svn_config_dir)
     if not self.svn_config_dir:
       if sys.platform == 'win32':
         self.svn_config_dir = os.path.join(os.environ['APPDATA'], 'Subversion')
       else:
         self.svn_config_dir = os.path.expanduser(
             os.path.join('~', '.subversion'))
     svn_config_file = os.path.join(self.svn_config_dir, 'config')
-    parser = ConfigParser.SafeConfigParser()
+    parser = configparser.SafeConfigParser()
     if os.path.isfile(svn_config_file):
       parser.read(svn_config_file)
     else:
       parser.add_section('auto-props')
     self.auto_props = dict(parser.items('auto-props'))
 
 
 class SvnMixIn(object):
   """MixIn class to add svn commands common to both svn and git-svn clients."""
   # These members need to be set by the subclass.
@@ skipping 558 matching lines @@
   def revisions(self, rev1, rev2):
     return self.checkout.revisions(rev1, rev2)
 
   @property
   def project_name(self):
     return self.checkout.project_name
 
   @property
   def project_path(self):
     return self.checkout.project_path