repo: bump launcher version for accumulated fixes

Change-Id: I5d9b866cc53d3824a01f5f0af127cf0c3ff97366 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/254757 Reviewed-by: David Pursehouse <dpursehouse@collab.net> Tested-by: Mike Frysinger <vapier@google.com>
repo: avoid bare excepts to allow SystemExit to bubble
2025-06-26 20:17:52 +00:00 · 2020-02-12 23:27:11 +00:00 · 2020-02-12 22:21:52 +00:00 · 2020-02-12 20:54:57 +00:00 · 2020-02-12 20:54:50 +00:00 · 2020-02-12 19:57:42 +00:00
51 changed files with 804 additions and 419 deletions
--- a/.flake8
+++ b/.flake8
@ -1,3 +1,13 @@
 [flake8]
-max-line-length=80
-ignore=E111,E114,E402
+max-line-length=100
+ignore=
+    # E111: Indentation is not a multiple of four
+    E111,
+    # E114: Indentation is not a multiple of four (comment)
+    E114,
+    # E402: Module level import not at top of file
+    E402,
+    # W503: Line break before binary operator
+    W503,
+    # W504: Line break after binary operator
+    W504
--- a/color.py
+++ b/color.py
@ -84,6 +84,7 @@ def _Color(fg=None, bg=None, attr=None):
    code = ''
  return code

+
 DEFAULT = None


--- a/command.py
+++ b/command.py
@ -123,9 +123,9 @@ class Command(object):
    project = None
    if os.path.exists(path):
      oldpath = None
-      while path and \
-            path != oldpath and \
-            path != manifest.topdir:
+      while (path and
+             path != oldpath and
+             path != manifest.topdir):
        try:
          project = self._by_path[path]
          break
@ -236,6 +236,7 @@ class InteractiveCommand(Command):
  """Command which requires user interaction on the tty and
     must not run within a pager, even if the user asks to.
  """
+
  def WantPager(self, _opt):
    return False

@ -244,6 +245,7 @@ class PagedCommand(Command):
  """Command which defaults to output in a pager, as its
     display tends to be larger than one screen full.
  """
+
  def WantPager(self, _opt):
    return True

--- a/docs/release-process.md
+++ b/docs/release-process.md
@ -161,7 +161,91 @@ You can create a short changelog using the command:
 $ git log --format="%h (%aN) %s" --no-merges origin/stable..$r
 ```

+## Project References

+Here's a table showing the relationship of major tools, their EOL dates, and
+their status in Ubuntu & Debian.
+Those distros tend to be good indicators of how long we need to support things.
+
+Things in bold indicate stuff to take note of, but does not guarantee that we
+still support them.
+Things in italics are things we used to care about but probably don't anymore.
+
+|   Date   |     EOL      | [Git][rel-g] | [Python][rel-p] | [Ubuntu][rel-u] / [Debian][rel-d] | Git | Python |
+|:--------:|:------------:|--------------|-----------------|-----------------------------------|-----|--------|
+| Oct 2008 | *Oct 2013*   |              | 2.6.0           | *10.04 Lucid* - 10.10 Maverick / *Squeeze* |
+| Dec 2008 | *Feb 2009*   |              | 3.0.0           |
+| Feb 2009 | *Mar 2012*   |              |                 | Debian 5 Lenny       | 1.5.6.5 | 2.5.2 |
+| Jun 2009 | *Jun 2016*   |              | 3.1.0           | *10.04 Lucid* - 10.10 Maverick / *Squeeze* |
+| Feb 2010 | *Oct 2012*   | 1.7.0        |                 | *10.04 Lucid* - *12.04 Precise* - 12.10 Quantal |
+| Apr 2010 | *Apr 2015*   |              |                 | *10.04 Lucid*        | 1.7.0.4  | 2.6.5 3.1.2  |
+| Jul 2010 | *Dec 2019*   |              | **2.7.0**       | 11.04 Natty - **<current>** |
+| Oct 2010 |              |              |                 | 10.10 Maverick       | 1.7.1    | 2.6.6 3.1.3  |
+| Feb 2011 | *Feb 2016*   |              |                 | Debian 6 Squeeze     | 1.7.2.5  | 2.6.6 3.1.3  |
+| Apr 2011 |              |              |                 | 11.04 Natty          | 1.7.4    | 2.7.1 3.2.0  |
+| Oct 2011 | *Feb 2016*   |              | 3.2.0           | 11.04 Natty - 12.10 Quantal |
+| Oct 2011 |              |              |                 | 11.10 Ocelot         | 1.7.5.4  | 2.7.2 3.2.2  |
+| Apr 2012 | *Apr 2019*   |              |                 | *12.04 Precise*      | 1.7.9.5  | 2.7.3 3.2.3  |
+| Sep 2012 | *Sep 2017*   |              | 3.3.0           | 13.04 Raring - 13.10 Saucy |
+| Oct 2012 | *Dec 2014*   | 1.8.0        |                 | 13.04 Raring - 13.10 Saucy |
+| Oct 2012 |              |              |                 | 12.10 Quantal        | 1.7.10.4 | 2.7.3 3.2.3  |
+| Apr 2013 |              |              |                 | 13.04 Raring         | 1.8.1.2  | 2.7.4 3.3.1  |
+| May 2013 | *May 2018*   |              |                 | Debian 7 Wheezy      | 1.7.10.4 | 2.7.3 3.2.3  |
+| Oct 2013 |              |              |                 | 13.10 Saucy          | 1.8.3.2  | 2.7.5 3.3.2  |
+| Feb 2014 | *Dec 2014*   | **1.9.0**    |                 | **14.04 Trusty** |
+| Mar 2014 | *Mar 2019*   |              | **3.4.0**       | **14.04 Trusty** - 15.10 Wily / **Jessie** |
+| Apr 2014 | **Apr 2022** |              |                 | **14.04 Trusty**     | 1.9.1    | 2.7.5 3.4.0  |
+| May 2014 | *Dec 2014*   | 2.0.0        |
+| Aug 2014 | *Dec 2014*   | **2.1.0**    |                 | 14.10 Utopic - 15.04 Vivid / **Jessie** |
+| Oct 2014 |              |              |                 | 14.10 Utopic         | 2.1.0    | 2.7.8 3.4.2  |
+| Nov 2014 | *Sep 2015*   | 2.2.0        |
+| Feb 2015 | *Sep 2015*   | 2.3.0        |
+| Apr 2015 | *May 2017*   | 2.4.0        |
+| Apr 2015 | **Jun 2020** |              |                 | **Debian 8 Jessie**  | 2.1.4    | 2.7.9 3.4.2  |
+| Apr 2015 |              |              |                 | 15.04 Vivid          | 2.1.4    | 2.7.9 3.4.3  |
+| Jul 2015 | *May 2017*   | 2.5.0        |                 | 15.10 Wily |
+| Sep 2015 | *May 2017*   | 2.6.0        |
+| Sep 2015 | **Sep 2020** |              | **3.5.0**       | **16.04 Xenial** - 17.04 Zesty / **Stretch** |
+| Oct 2015 |              |              |                 | 15.10 Wily           | 2.5.0    | 2.7.9 3.4.3  |
+| Jan 2016 | *Jul 2017*   | **2.7.0**    |                 | **16.04 Xenial** |
+| Mar 2016 | *Jul 2017*   | 2.8.0        |
+| Apr 2016 | **Apr 2024** |              |                 | **16.04 Xenial**     | 2.7.4    | 2.7.11 3.5.1 |
+| Jun 2016 | *Jul 2017*   | 2.9.0        |                 | 16.10 Yakkety |
+| Sep 2016 | *Sep 2017*   | 2.10.0       |
+| Oct 2016 |              |              |                 | 16.10 Yakkety        | 2.9.3    | 2.7.11 3.5.1 |
+| Nov 2016 | *Sep 2017*   | **2.11.0**   |                 | 17.04 Zesty / **Stretch** |
+| Dec 2016 | **Dec 2021** |              | **3.6.0**       | 17.10 Artful - **18.04 Bionic** - 18.10 Cosmic |
+| Feb 2017 | *Sep 2017*   | 2.12.0       |
+| Apr 2017 |              |              |                 | 17.04 Zesty          | 2.11.0   | 2.7.13 3.5.3 |
+| May 2017 | *May 2018*   | 2.13.0       |
+| Jun 2017 | **Jun 2022** |              |                 | **Debian 9 Stretch** | 2.11.0   | 2.7.13 3.5.3 |
+| Aug 2017 | *Dec 2019*   | 2.14.0       |                 | 17.10 Artful |
+| Oct 2017 | *Dec 2019*   | 2.15.0       |
+| Oct 2017 |              |              |                 | 17.10 Artful         | 2.14.1   | 2.7.14 3.6.3 |
+| Jan 2018 | *Dec 2019*   | 2.16.0       |
+| Apr 2018 | *Dec 2019*   | 2.17.0       |                 | **18.04 Bionic**     |
+| Apr 2018 | **Apr 2028** |              |                 | **18.04 Bionic**     | 2.17.0   | 2.7.15 3.6.5 |
+| Jun 2018 | *Dec 2019*   | 2.18.0       |
+| Jun 2018 | **Jun 2023** |              | 3.7.0           | 19.04 Disco - **20.04 Focal** / **Buster** |
+| Sep 2018 | *Dec 2019*   | 2.19.0       |                 | 18.10 Cosmic |
+| Oct 2018 |              |              |                 | 18.10 Cosmic         | 2.19.1   | 2.7.15 3.6.6 |
+| Dec 2018 | *Dec 2019*   | **2.20.0**   |                 | 19.04 Disco / **Buster** |
+| Feb 2019 | *Dec 2019*   | 2.21.0       |
+| Apr 2019 |              |              |                 | 19.04 Disco          | 2.20.1   | 2.7.16 3.7.3 |
+| Jun 2019 |              | 2.22.0       |
+| Jul 2019 | **Jul 2024** |              |                 | **Debian 10 Buster** | 2.20.1   | 2.7.16 3.7.3 |
+| Aug 2019 |              | 2.23.0       |
+| Oct 2019 | **Oct 2024** |              | 3.8.0           |
+| Oct 2019 |              |              |                 | 19.10 Eoan           | 2.20.1   | 2.7.17 3.7.5 |
+| Nov 2019 |              | 2.24.0       |
+| Jan 2020 |              | 2.25.0       |                 | **20.04 Focal** |
+| Apr 2020 | **Apr 2030** |              |                 | **20.04 Focal**      | 2.25.0   | 2.7.17 3.7.5 |
+
+
+[rel-d]: https://en.wikipedia.org/wiki/Debian_version_history
+[rel-g]: https://en.wikipedia.org/wiki/Git#Releases
+[rel-p]: https://en.wikipedia.org/wiki/History_of_Python#Table_of_versions
+[rel-u]: https://en.wikipedia.org/wiki/Ubuntu_version_history#Table_of_versions
 [example announcement]: https://groups.google.com/d/topic/repo-discuss/UGBNismWo1M/discussion
 [repo-discuss@googlegroups.com]: https://groups.google.com/forum/#!forum/repo-discuss
 [go/repo-release]: https://goto.google.com/repo-release
--- a/editor.py
+++ b/editor.py
@ -24,6 +24,7 @@ import tempfile
 from error import EditorError
 import platform_utils

+
 class Editor(object):
  """Manages the user's preferred text editor."""

@ -57,7 +58,7 @@ class Editor(object):

    if os.getenv('TERM') == 'dumb':
      print(
-"""No editor specified in GIT_EDITOR, core.editor, VISUAL or EDITOR.
+          """No editor specified in GIT_EDITOR, core.editor, VISUAL or EDITOR.
 Tried to fall back to vi but terminal is dumb.  Please configure at
 least one of these before using this command.""", file=sys.stderr)
      sys.exit(1)
@ -104,10 +105,10 @@ least one of these before using this command.""", file=sys.stderr)
        rc = subprocess.Popen(args, shell=shell).wait()
      except OSError as e:
        raise EditorError('editor failed, %s: %s %s'
-          % (str(e), editor, path))
+                          % (str(e), editor, path))
      if rc != 0:
        raise EditorError('editor failed with exit status %d: %s %s'
-          % (rc, editor, path))
+                          % (rc, editor, path))

      with open(path, mode='rb') as fd2:
        return fd2.read().decode('utf-8')
--- a/error.py
+++ b/error.py
@ -14,21 +14,26 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.

+
 class ManifestParseError(Exception):
  """Failed to parse the manifest file.
  """

+
 class ManifestInvalidRevisionError(Exception):
  """The revision value in a project is incorrect.
  """

+
 class ManifestInvalidPathError(Exception):
  """A path used in <copyfile> or <linkfile> is incorrect.
  """

+
 class NoManifestException(Exception):
  """The required manifest does not exist.
  """
+
  def __init__(self, path, reason):
    super(NoManifestException, self).__init__()
    self.path = path
@ -37,9 +42,11 @@ class NoManifestException(Exception):
  def __str__(self):
    return self.reason

+
 class EditorError(Exception):
  """Unspecified error from the user's text editor.
  """
+
  def __init__(self, reason):
    super(EditorError, self).__init__()
    self.reason = reason
@ -47,9 +54,11 @@ class EditorError(Exception):
  def __str__(self):
    return self.reason

+
 class GitError(Exception):
  """Unspecified internal error from git.
  """
+
  def __init__(self, command):
    super(GitError, self).__init__()
    self.command = command
@ -57,9 +66,11 @@ class GitError(Exception):
  def __str__(self):
    return self.command

+
 class UploadError(Exception):
  """A bundle upload to Gerrit did not succeed.
  """
+
  def __init__(self, reason):
    super(UploadError, self).__init__()
    self.reason = reason
@ -67,9 +78,11 @@ class UploadError(Exception):
  def __str__(self):
    return self.reason

+
 class DownloadError(Exception):
  """Cannot download a repository.
  """
+
  def __init__(self, reason):
    super(DownloadError, self).__init__()
    self.reason = reason
@ -77,9 +90,11 @@ class DownloadError(Exception):
  def __str__(self):
    return self.reason

+
 class NoSuchProjectError(Exception):
  """A specified project does not exist in the work tree.
  """
+
  def __init__(self, name=None):
    super(NoSuchProjectError, self).__init__()
    self.name = name
@ -93,6 +108,7 @@ class NoSuchProjectError(Exception):
 class InvalidProjectGroupsError(Exception):
  """A specified project is not suitable for the specified groups
  """
+
  def __init__(self, name=None):
    super(InvalidProjectGroupsError, self).__init__()
    self.name = name
@ -102,15 +118,18 @@ class InvalidProjectGroupsError(Exception):
      return 'in current directory'
    return self.name

+
 class RepoChangedException(Exception):
  """Thrown if 'repo sync' results in repo updating its internal
     repo or manifest repositories.  In this special case we must
     use exec to re-execute repo with the new code and manifest.
  """
+
  def __init__(self, extra_args=None):
    super(RepoChangedException, self).__init__()
    self.extra_args = extra_args or []

+
 class HookError(Exception):
  """Thrown if a 'repo-hook' could not be run.

--- a/event_log.py
+++ b/event_log.py
@ -23,6 +23,7 @@ TASK_COMMAND = 'command'
 TASK_SYNC_NETWORK = 'sync-network'
 TASK_SYNC_LOCAL = 'sync-local'

+
 class EventLog(object):
  """Event log that records events that occurred during a repo invocation.

@ -138,7 +139,7 @@ class EventLog(object):
    Returns:
      A dictionary of the event added to the log.
    """
-    event['status'] =  self.GetStatusString(success)
+    event['status'] = self.GetStatusString(success)
    event['finish_time'] = finish
    return event

@ -165,6 +166,7 @@ class EventLog(object):
 # An integer id that is unique across this invocation of the program.
 _EVENT_ID = multiprocessing.Value('i', 1)

+
 def _NextEventId():
  """Helper function for grabbing the next unique id.

--- a/git_command.py
+++ b/git_command.py
@ -28,8 +28,17 @@ from repo_trace import REPO_TRACE, IsTrace, Trace
 from wrapper import Wrapper

 GIT = 'git'
-# Should keep in sync with the "repo" launcher file.
-MIN_GIT_VERSION = (2, 10, 2)
+# NB: These do not need to be kept in sync with the repo launcher script.
+# These may be much newer as it allows the repo launcher to roll between
+# different repo releases while source versions might require a newer git.
+#
+# The soft version is when we start warning users that the version is old and
+# we'll be dropping support for it.  We'll refuse to work with versions older
+# than the hard version.
+#
+# git-1.7 is in (EOL) Ubuntu Precise.  git-1.9 is in Ubuntu Trusty.
+MIN_GIT_VERSION_SOFT = (1, 9, 1)
+MIN_GIT_VERSION_HARD = (1, 7, 2)
 GIT_DIR = 'GIT_DIR'

 LAST_GITDIR = None
@ -39,6 +48,7 @@ _ssh_proxy_path = None
 _ssh_sock_path = None
 _ssh_clients = []

+
 def ssh_sock(create=True):
  global _ssh_sock_path
  if _ssh_sock_path is None:
@ -48,27 +58,31 @@ def ssh_sock(create=True):
    if not os.path.exists(tmp_dir):
      tmp_dir = tempfile.gettempdir()
    _ssh_sock_path = os.path.join(
-      tempfile.mkdtemp('', 'ssh-', tmp_dir),
-      'master-%r@%h:%p')
+        tempfile.mkdtemp('', 'ssh-', tmp_dir),
+        'master-%r@%h:%p')
  return _ssh_sock_path

+
 def _ssh_proxy():
  global _ssh_proxy_path
  if _ssh_proxy_path is None:
    _ssh_proxy_path = os.path.join(
-      os.path.dirname(__file__),
-      'git_ssh')
+        os.path.dirname(__file__),
+        'git_ssh')
  return _ssh_proxy_path

+
 def _add_ssh_client(p):
  _ssh_clients.append(p)

+
 def _remove_ssh_client(p):
  try:
    _ssh_clients.remove(p)
  except ValueError:
    pass

+
 def terminate_ssh_clients():
  global _ssh_clients
  for p in _ssh_clients:
@ -79,8 +93,10 @@ def terminate_ssh_clients():
      pass
  _ssh_clients = []

+
 _git_version = None

+
 class _GitCall(object):
  def version_tuple(self):
    global _git_version
@ -92,12 +108,15 @@ class _GitCall(object):
    return _git_version

  def __getattr__(self, name):
-    name = name.replace('_','-')
+    name = name.replace('_', '-')
+
    def fun(*cmdv):
      command = [name]
      command.extend(cmdv)
      return GitCommand(None, command).Wait() == 0
    return fun
+
+
 git = _GitCall()


@ -178,8 +197,10 @@ class UserAgent(object):

    return self._git_ua

+
 user_agent = UserAgent()

+
 def git_require(min_version, fail=False, msg=''):
  git_version = git.version_tuple()
  if min_version <= git_version:
@ -192,21 +213,23 @@ def git_require(min_version, fail=False, msg=''):
    sys.exit(1)
  return False

+
 def _setenv(env, name, value):
  env[name] = value.encode()

+
 class GitCommand(object):
  def __init__(self,
               project,
               cmdv,
-               bare = False,
-               provide_stdin = False,
-               capture_stdout = False,
-               capture_stderr = False,
-               disable_editor = False,
-               ssh_proxy = False,
-               cwd = None,
-               gitdir = None):
+               bare=False,
+               provide_stdin=False,
+               capture_stdout=False,
+               capture_stderr=False,
+               disable_editor=False,
+               ssh_proxy=False,
+               cwd=None,
+               gitdir=None):
    env = self._GetBasicEnv()

    # If we are not capturing std* then need to print it.
@ -286,11 +309,11 @@ class GitCommand(object):

    try:
      p = subprocess.Popen(command,
-                           cwd = cwd,
-                           env = env,
-                           stdin = stdin,
-                           stdout = stdout,
-                           stderr = stderr)
+                           cwd=cwd,
+                           env=env,
+                           stdin=stdin,
+                           stdout=stdout,
+                           stderr=stderr)
    except Exception as e:
      raise GitError('%s: %s' % (command[1], e))

--- a/git_config.py
+++ b/git_config.py
@ -59,39 +59,45 @@ ID_RE = re.compile(r'^[0-9a-f]{40}$')

 REVIEW_CACHE = dict()

+
 def IsChange(rev):
  return rev.startswith(R_CHANGES)

+
 def IsId(rev):
  return ID_RE.match(rev)

+
 def IsTag(rev):
  return rev.startswith(R_TAGS)

+
 def IsImmutable(rev):
    return IsChange(rev) or IsId(rev) or IsTag(rev)

+
 def _key(name):
  parts = name.split('.')
  if len(parts) < 2:
    return name.lower()
-  parts[ 0] = parts[ 0].lower()
+  parts[0] = parts[0].lower()
  parts[-1] = parts[-1].lower()
  return '.'.join(parts)

+
 class GitConfig(object):
  _ForUser = None

  @classmethod
  def ForUser(cls):
    if cls._ForUser is None:
-      cls._ForUser = cls(configfile = os.path.expanduser('~/.gitconfig'))
+      cls._ForUser = cls(configfile=os.path.expanduser('~/.gitconfig'))
    return cls._ForUser

  @classmethod
  def ForRepository(cls, gitdir, defaults=None):
-    return cls(configfile = os.path.join(gitdir, 'config'),
-               defaults = defaults)
+    return cls(configfile=os.path.join(gitdir, 'config'),
+               defaults=defaults)

  def __init__(self, configfile, defaults=None, jsonFile=None):
    self.file = configfile
@ -104,16 +110,16 @@ class GitConfig(object):
    self._json = jsonFile
    if self._json is None:
      self._json = os.path.join(
-        os.path.dirname(self.file),
-        '.repo_' + os.path.basename(self.file) + '.json')
+          os.path.dirname(self.file),
+          '.repo_' + os.path.basename(self.file) + '.json')

-  def Has(self, name, include_defaults = True):
+  def Has(self, name, include_defaults=True):
    """Return true if this configuration file has the key.
    """
    if _key(name) in self._cache:
      return True
    if include_defaults and self.defaults:
-      return self.defaults.Has(name, include_defaults = True)
+      return self.defaults.Has(name, include_defaults=True)
    return False

  def GetBoolean(self, name):
@ -142,7 +148,7 @@ class GitConfig(object):
      v = self._cache[_key(name)]
    except KeyError:
      if self.defaults:
-        return self.defaults.GetString(name, all_keys = all_keys)
+        return self.defaults.GetString(name, all_keys=all_keys)
      v = []

    if not all_keys:
@ -153,7 +159,7 @@ class GitConfig(object):
    r = []
    r.extend(v)
    if self.defaults:
-      r.extend(self.defaults.GetString(name, all_keys = True))
+      r.extend(self.defaults.GetString(name, all_keys=True))
    return r

  def SetString(self, name, value):
@ -217,7 +223,7 @@ class GitConfig(object):
    """
    return self._sections.get(section, set())

-  def HasSection(self, section, subsection = ''):
+  def HasSection(self, section, subsection=''):
    """Does at least one key in section.subsection exist?
    """
    try:
@ -268,8 +274,7 @@ class GitConfig(object):

  def _ReadJson(self):
    try:
-      if os.path.getmtime(self._json) \
-      <= os.path.getmtime(self.file):
+      if os.path.getmtime(self._json) <= os.path.getmtime(self.file):
        platform_utils.remove(self._json)
        return None
    except OSError:
@ -323,8 +328,8 @@ class GitConfig(object):

    p = GitCommand(None,
                   command,
-                   capture_stdout = True,
-                   capture_stderr = True)
+                   capture_stdout=True,
+                   capture_stderr=True)
    if p.Wait() == 0:
      return p.stdout
    else:
@ -392,6 +397,7 @@ _master_keys = set()
 _ssh_master = True
 _master_keys_lock = None

+
 def init_ssh():
  """Should be called once at the start of repo to init ssh master handling.

@ -401,6 +407,7 @@ def init_ssh():
  assert _master_keys_lock is None, "Should only call init_ssh once"
  _master_keys_lock = _threading.Lock()

+
 def _open_ssh(host, port=None):
  global _ssh_master

@ -421,17 +428,17 @@ def _open_ssh(host, port=None):
    if key in _master_keys:
      return True

-    if not _ssh_master \
-    or 'GIT_SSH' in os.environ \
-    or sys.platform in ('win32', 'cygwin'):
+    if (not _ssh_master
+            or 'GIT_SSH' in os.environ
+            or sys.platform in ('win32', 'cygwin')):
      # failed earlier, or cygwin ssh can't do this
      #
      return False

    # We will make two calls to ssh; this is the common part of both calls.
    command_base = ['ssh',
-                     '-o','ControlPath %s' % ssh_sock(),
-                     host]
+                    '-o', 'ControlPath %s' % ssh_sock(),
+                    host]
    if port is not None:
      command_base[1:1] = ['-p', str(port)]

@ -439,13 +446,13 @@ def _open_ssh(host, port=None):
    # ...but before actually starting a master, we'll double-check.  This can
    # be important because we can't tell that that 'git@myhost.com' is the same
    # as 'myhost.com' where "User git" is setup in the user's ~/.ssh/config file.
-    check_command = command_base + ['-O','check']
+    check_command = command_base + ['-O', 'check']
    try:
      Trace(': %s', ' '.join(check_command))
      check_process = subprocess.Popen(check_command,
                                       stdout=subprocess.PIPE,
                                       stderr=subprocess.PIPE)
-      check_process.communicate() # read output, but ignore it...
+      check_process.communicate()  # read output, but ignore it...
      isnt_running = check_process.wait()

      if not isnt_running:
@ -458,16 +465,14 @@ def _open_ssh(host, port=None):
      # to the log there.
      pass

-    command = command_base[:1] + \
-              ['-M', '-N'] + \
-              command_base[1:]
+    command = command_base[:1] + ['-M', '-N'] + command_base[1:]
    try:
      Trace(': %s', ' '.join(command))
      p = subprocess.Popen(command)
    except Exception as e:
      _ssh_master = False
      print('\nwarn: cannot enable ssh control master for %s:%s\n%s'
-             % (host,port, str(e)), file=sys.stderr)
+            % (host, port, str(e)), file=sys.stderr)
      return False

    time.sleep(1)
@ -481,6 +486,7 @@ def _open_ssh(host, port=None):
  finally:
    _master_keys_lock.release()

+
 def close_ssh():
  global _master_keys_lock

@ -505,15 +511,18 @@ def close_ssh():
  # We're done with the lock, so we can delete it.
  _master_keys_lock = None

+
 URI_SCP = re.compile(r'^([^@:]*@?[^:/]{1,}):')
 URI_ALL = re.compile(r'^([a-z][a-z+-]*)://([^@/]*@?[^/]*)/')

+
 def GetSchemeFromUrl(url):
  m = URI_ALL.match(url)
  if m:
    return m.group(1)
  return None

+
@contextlib.contextmanager
 def GetUrlCookieFile(url, quiet):
  if url.startswith('persistent-'):
@ -554,6 +563,7 @@ def GetUrlCookieFile(url, quiet):
    cookiefile = os.path.expanduser(cookiefile)
  yield cookiefile, None

+
 def _preconnect(url):
  m = URI_ALL.match(url)
  if m:
@ -574,9 +584,11 @@ def _preconnect(url):

  return False

+
 class Remote(object):
  """Configuration options related to a remote.
  """
+
  def __init__(self, config, name):
    self._config = config
    self.name = name
@ -585,7 +597,7 @@ class Remote(object):
    self.review = self._Get('review')
    self.projectname = self._Get('projectname')
    self.fetch = list(map(RefSpec.FromString,
-                      self._Get('fetch', all_keys=True)))
+                          self._Get('fetch', all_keys=True)))
    self._review_url = None

  def _InsteadOf(self):
@ -599,8 +611,8 @@ class Remote(object):
      insteadOfList = globCfg.GetString(key, all_keys=True)

      for insteadOf in insteadOfList:
-        if self.url.startswith(insteadOf) \
-        and len(insteadOf) > len(longest):
+        if (self.url.startswith(insteadOf)
+                and len(insteadOf) > len(longest)):
          longest = insteadOf
          longestUrl = url

@ -731,12 +743,13 @@ class Remote(object):

  def _Get(self, key, all_keys=False):
    key = 'remote.%s.%s' % (self.name, key)
-    return self._config.GetString(key, all_keys = all_keys)
+    return self._config.GetString(key, all_keys=all_keys)


 class Branch(object):
  """Configuration options related to a single branch.
  """
+
  def __init__(self, config, name):
    self._config = config
    self.name = name
@ -780,4 +793,4 @@ class Branch(object):

  def _Get(self, key, all_keys=False):
    key = 'branch.%s.%s' % (self.name, key)
-    return self._config.GetString(key, all_keys = all_keys)
+    return self._config.GetString(key, all_keys=all_keys)
--- a/git_refs.py
+++ b/git_refs.py
@ -18,12 +18,12 @@ import os
 from repo_trace import Trace
 import platform_utils

-HEAD      = 'HEAD'
+HEAD = 'HEAD'
 R_CHANGES = 'refs/changes/'
-R_HEADS   = 'refs/heads/'
-R_TAGS    = 'refs/tags/'
-R_PUB     = 'refs/published/'
-R_M       = 'refs/remotes/m/'
+R_HEADS = 'refs/heads/'
+R_TAGS = 'refs/tags/'
+R_PUB = 'refs/published/'
+R_M = 'refs/remotes/m/'


 class GitRefs(object):
--- a/gitc_utils.py
+++ b/gitc_utils.py
@ -29,12 +29,15 @@ from error import ManifestParseError

 NUM_BATCH_RETRIEVE_REVISIONID = 32

+
 def get_gitc_manifest_dir():
  return wrapper.Wrapper().get_gitc_manifest_dir()

+
 def parse_clientdir(gitc_fs_path):
  return wrapper.Wrapper().gitc_parse_clientdir(gitc_fs_path)

+
 def _set_project_revisions(projects):
  """Sets the revisionExpr for a list of projects.

@ -52,7 +55,7 @@ def _set_project_revisions(projects):
                                       project.remote.url,
                                       project.revisionExpr],
                                      capture_stdout=True, cwd='/tmp'))
-      for project in projects if not git_config.IsId(project.revisionExpr)]
+                     for project in projects if not git_config.IsId(project.revisionExpr)]
  for proj, gitcmd in project_gitcmds:
    if gitcmd.Wait():
      print('FATAL: Failed to retrieve revisionExpr for %s' % proj)
@ -63,6 +66,7 @@ def _set_project_revisions(projects):
                               (proj.remote.url, proj.revisionExpr))
    proj.revisionExpr = revisionExpr

+
 def _manifest_groups(manifest):
  """Returns the manifest group string that should be synced

@ -77,6 +81,7 @@ def _manifest_groups(manifest):
    groups = 'default,platform-' + platform.system().lower()
  return groups

+
 def generate_gitc_manifest(gitc_manifest, manifest, paths=None):
  """Generate a manifest for shafsd to use for this GITC client.

@ -104,11 +109,11 @@ def generate_gitc_manifest(gitc_manifest, manifest, paths=None):
      if not proj.upstream and not git_config.IsId(proj.revisionExpr):
        proj.upstream = proj.revisionExpr

-      if not path in gitc_manifest.paths:
+      if path not in gitc_manifest.paths:
        # Any new projects need their first revision, even if we weren't asked
        # for them.
        projects.append(proj)
-      elif not path in paths:
+      elif path not in paths:
        # And copy revisions from the previous manifest if we're not updating
        # them now.
        gitc_proj = gitc_manifest.paths[path]
@ -121,7 +126,7 @@ def generate_gitc_manifest(gitc_manifest, manifest, paths=None):
  index = 0
  while index < len(projects):
    _set_project_revisions(
-        projects[index:(index+NUM_BATCH_RETRIEVE_REVISIONID)])
+        projects[index:(index + NUM_BATCH_RETRIEVE_REVISIONID)])
    index += NUM_BATCH_RETRIEVE_REVISIONID

  if gitc_manifest is not None:
@ -140,6 +145,7 @@ def generate_gitc_manifest(gitc_manifest, manifest, paths=None):
  # Save the manifest.
  save_manifest(manifest)

+
 def save_manifest(manifest, client_dir=None):
  """Save the manifest file in the client_dir.

--- a/main.py
+++ b/main.py
@ -47,7 +47,7 @@ except ImportError:
 from color import SetDefaultColoring
 import event_log
 from repo_trace import SetTrace
-from git_command import git, GitCommand, user_agent
+from git_command import user_agent
 from git_config import init_ssh, close_ssh
 from command import InteractiveCommand
 from command import MirrorSafeCommand
@ -101,6 +101,7 @@ global_options.add_option('--event-log',
                          dest='event_log', action='store',
                          help='filename of event log to append timeline to')

+
 class _Repo(object):
  def __init__(self, repodir):
    self.repodir = repodir
@ -188,7 +189,7 @@ class _Repo(object):
      copts = cmd.ReadEnvironmentOptions(copts)
    except NoManifestException as e:
      print('error: in `%s`: %s' % (' '.join([name] + argv), str(e)),
-        file=sys.stderr)
+            file=sys.stderr)
      print('error: manifest missing or unreadable -- please run init',
            file=sys.stderr)
      return 1
@ -211,9 +212,9 @@ class _Repo(object):
      cmd.ValidateOptions(copts, cargs)
      result = cmd.Execute(copts, cargs)
    except (DownloadError, ManifestInvalidRevisionError,
-        NoManifestException) as e:
+            NoManifestException) as e:
      print('error: in `%s`: %s' % (' '.join([name] + argv), str(e)),
-        file=sys.stderr)
+            file=sys.stderr)
      if isinstance(e, NoManifestException):
        print('error: manifest missing or unreadable -- please run init',
              file=sys.stderr)
@ -300,11 +301,13 @@ repo: error:
    cp %s %s
 """ % (exp_str, WrapperPath(), repo_path), file=sys.stderr)

+
 def _CheckRepoDir(repo_dir):
  if not repo_dir:
    print('no --repo-dir argument', file=sys.stderr)
    sys.exit(1)

+
 def _PruneOptions(argv, opt):
  i = 0
  while i < len(argv):
@ -320,6 +323,7 @@ def _PruneOptions(argv, opt):
      continue
    i += 1

+
 class _UserAgentHandler(urllib.request.BaseHandler):
  def http_request(self, req):
    req.add_header('User-Agent', user_agent.repo)
@ -329,6 +333,7 @@ class _UserAgentHandler(urllib.request.BaseHandler):
    req.add_header('User-Agent', user_agent.repo)
    return req

+
 def _AddPasswordFromUserInput(handler, msg, req):
  # If repo could not find auth info from netrc, try to get it from user input
  url = req.get_full_url()
@ -342,22 +347,24 @@ def _AddPasswordFromUserInput(handler, msg, req):
      return
    handler.passwd.add_password(None, url, user, password)

+
 class _BasicAuthHandler(urllib.request.HTTPBasicAuthHandler):
  def http_error_401(self, req, fp, code, msg, headers):
    _AddPasswordFromUserInput(self, msg, req)
    return urllib.request.HTTPBasicAuthHandler.http_error_401(
-      self, req, fp, code, msg, headers)
+        self, req, fp, code, msg, headers)

  def http_error_auth_reqed(self, authreq, host, req, headers):
    try:
      old_add_header = req.add_header
+
      def _add_header(name, val):
        val = val.replace('\n', '')
        old_add_header(name, val)
      req.add_header = _add_header
      return urllib.request.AbstractBasicAuthHandler.http_error_auth_reqed(
-        self, authreq, host, req, headers)
-    except:
+          self, authreq, host, req, headers)
+    except Exception:
      reset = getattr(self, 'reset_retry_count', None)
      if reset is not None:
        reset()
@ -365,22 +372,24 @@ class _BasicAuthHandler(urllib.request.HTTPBasicAuthHandler):
        self.retried = 0
      raise

+
 class _DigestAuthHandler(urllib.request.HTTPDigestAuthHandler):
  def http_error_401(self, req, fp, code, msg, headers):
    _AddPasswordFromUserInput(self, msg, req)
    return urllib.request.HTTPDigestAuthHandler.http_error_401(
-      self, req, fp, code, msg, headers)
+        self, req, fp, code, msg, headers)

  def http_error_auth_reqed(self, auth_header, host, req, headers):
    try:
      old_add_header = req.add_header
+
      def _add_header(name, val):
        val = val.replace('\n', '')
        old_add_header(name, val)
      req.add_header = _add_header
      return urllib.request.AbstractDigestAuthHandler.http_error_auth_reqed(
-        self, auth_header, host, req, headers)
-    except:
+          self, auth_header, host, req, headers)
+    except Exception:
      reset = getattr(self, 'reset_retry_count', None)
      if reset is not None:
        reset()
@ -388,6 +397,7 @@ class _DigestAuthHandler(urllib.request.HTTPDigestAuthHandler):
        self.retried = 0
      raise

+
 class _KerberosAuthHandler(urllib.request.BaseHandler):
  def __init__(self):
    self.retried = 0
@ -406,7 +416,7 @@ class _KerberosAuthHandler(urllib.request.BaseHandler):

      if self.retried > 3:
        raise urllib.request.HTTPError(req.get_full_url(), 401,
-          "Negotiate auth failed", headers, None)
+                                       "Negotiate auth failed", headers, None)
      else:
        self.retried += 1

@ -422,7 +432,7 @@ class _KerberosAuthHandler(urllib.request.BaseHandler):
        return response
    except kerberos.GSSError:
      return None
-    except:
+    except Exception:
      self.reset_retry_count()
      raise
    finally:
@ -468,6 +478,7 @@ class _KerberosAuthHandler(urllib.request.BaseHandler):
      kerberos.authGSSClientClean(self.context)
      self.context = None

+
 def init_http():
  handlers = [_UserAgentHandler()]

@ -476,7 +487,7 @@ def init_http():
    n = netrc.netrc()
    for host in n.hosts:
      p = n.hosts[host]
-      mgr.add_password(p[1], 'http://%s/'  % host, p[0], p[2])
+      mgr.add_password(p[1], 'http://%s/' % host, p[0], p[2])
      mgr.add_password(p[1], 'https://%s/' % host, p[0], p[2])
  except netrc.NetrcParseError:
    pass
@ -495,6 +506,7 @@ def init_http():
    handlers.append(urllib.request.HTTPSHandler(debuglevel=1))
  urllib.request.install_opener(urllib.request.build_opener(*handlers))

+
 def _Main(argv):
  result = 0

@ -551,5 +563,6 @@ def _Main(argv):
  TerminatePager()
  sys.exit(result)

+
 if __name__ == '__main__':
  _Main(sys.argv[1:])
--- a/manifest_xml.py
+++ b/manifest_xml.py
@ -56,6 +56,7 @@ urllib.parse.uses_netloc.extend([
    'sso',
    'rpc'])

+
 class _Default(object):
  """Project defaults within the manifest."""

@ -74,6 +75,7 @@ class _Default(object):
  def __ne__(self, other):
    return self.__dict__ != other.__dict__

+
 class _XmlRemote(object):
  def __init__(self,
               name,
@ -127,6 +129,7 @@ class _XmlRemote(object):
                      orig_name=self.name,
                      fetchUrl=self.fetchUrl)

+
 class XmlManifest(object):
  """manages the repo configuration file"""

@ -140,12 +143,12 @@ class XmlManifest(object):
    self._load_local_manifests = True

    self.repoProject = MetaProject(self, 'repo',
-      gitdir   = os.path.join(repodir, 'repo/.git'),
-      worktree = os.path.join(repodir, 'repo'))
+                                   gitdir=os.path.join(repodir, 'repo/.git'),
+                                   worktree=os.path.join(repodir, 'repo'))

    self.manifestProject = MetaProject(self, 'manifests',
-      gitdir   = os.path.join(repodir, 'manifests.git'),
-      worktree = os.path.join(repodir, 'manifests'))
+                                       gitdir=os.path.join(repodir, 'manifests.git'),
+                                       worktree=os.path.join(repodir, 'manifests'))

    self._Unload()

@ -224,7 +227,7 @@ class XmlManifest(object):
    if self.notice:
      notice_element = root.appendChild(doc.createElement('notice'))
      notice_lines = self.notice.splitlines()
-      indented_notice = ('\n'.join(" "*4 + line for line in notice_lines))[4:]
+      indented_notice = ('\n'.join(" " * 4 + line for line in notice_lines))[4:]
      notice_element.appendChild(doc.createTextNode(indented_notice))

    d = self.default
@ -462,12 +465,12 @@ class XmlManifest(object):
            self.localManifestWarning = True
            print('warning: %s is deprecated; put local manifests '
                  'in `%s` instead' % (LOCAL_MANIFEST_NAME,
-                  os.path.join(self.repodir, LOCAL_MANIFESTS_DIR_NAME)),
+                                       os.path.join(self.repodir, LOCAL_MANIFESTS_DIR_NAME)),
                  file=sys.stderr)
          nodes.append(self._ParseManifestXml(local, self.repodir))

        local_dir = os.path.abspath(os.path.join(self.repodir,
-                                    LOCAL_MANIFESTS_DIR_NAME))
+                                                 LOCAL_MANIFESTS_DIR_NAME))
        try:
          for local_file in sorted(platform_utils.listdir(local_dir)):
            if local_file.endswith('.xml'):
@ -512,7 +515,7 @@ class XmlManifest(object):
        fp = os.path.join(include_root, name)
        if not os.path.isfile(fp):
          raise ManifestParseError("include %s doesn't exist or isn't a file"
-              % (name,))
+                                   % (name,))
        try:
          nodes.extend(self._ParseManifestXml(fp, include_root))
        # should isolate this to the exact exception, but that's
@ -655,7 +658,6 @@ class XmlManifest(object):
        if self._repo_hooks_project and (self._repo_hooks_project.name == name):
          self._repo_hooks_project = None

-
  def _AddMetaProjectMirror(self, m):
    name = None
    m_url = m.GetRemote(m.remote.name).url
@ -682,15 +684,15 @@ class XmlManifest(object):
    if name not in self._projects:
      m.PreSync()
      gitdir = os.path.join(self.topdir, '%s.git' % name)
-      project = Project(manifest = self,
-                        name = name,
-                        remote = remote.ToRemoteSpec(name),
-                        gitdir = gitdir,
-                        objdir = gitdir,
-                        worktree = None,
-                        relpath = name or None,
-                        revisionExpr = m.revisionExpr,
-                        revisionId = None)
+      project = Project(manifest=self,
+                        name=name,
+                        remote=remote.ToRemoteSpec(name),
+                        gitdir=gitdir,
+                        objdir=gitdir,
+                        worktree=None,
+                        relpath=name or None,
+                        revisionExpr=m.revisionExpr,
+                        revisionId=None)
      self._projects[project.name] = [project]
      self._paths[project.relpath] = project

@ -798,7 +800,7 @@ class XmlManifest(object):
  def _UnjoinName(self, parent_name, name):
    return os.path.relpath(name, parent_name)

-  def _ParseProject(self, node, parent = None, **extra_proj_attrs):
+  def _ParseProject(self, node, parent=None, **extra_proj_attrs):
    """
    reads a <project> element from the manifest file
    """
@ -811,21 +813,21 @@ class XmlManifest(object):
      remote = self._default.remote
    if remote is None:
      raise ManifestParseError("no remote for project %s within %s" %
-            (name, self.manifestFile))
+                               (name, self.manifestFile))

    revisionExpr = node.getAttribute('revision') or remote.revision
    if not revisionExpr:
      revisionExpr = self._default.revisionExpr
    if not revisionExpr:
      raise ManifestParseError("no revision for project %s within %s" %
-            (name, self.manifestFile))
+                               (name, self.manifestFile))

    path = node.getAttribute('path')
    if not path:
      path = name
    if path.startswith('/'):
      raise ManifestParseError("project %s path cannot be absolute in %s" %
-            (name, self.manifestFile))
+                               (name, self.manifestFile))

    rebase = node.getAttribute('rebase')
    if not rebase:
@ -855,7 +857,7 @@ class XmlManifest(object):
    if clone_depth:
      try:
        clone_depth = int(clone_depth)
-        if  clone_depth <= 0:
+        if clone_depth <= 0:
          raise ValueError()
      except ValueError:
        raise ManifestParseError('invalid clone-depth %s in %s' %
@ -883,24 +885,24 @@ class XmlManifest(object):
      if node.getAttribute('force-path').lower() in ("yes", "true", "1"):
        gitdir = os.path.join(self.topdir, '%s.git' % path)

-    project = Project(manifest = self,
-                      name = name,
-                      remote = remote.ToRemoteSpec(name),
-                      gitdir = gitdir,
-                      objdir = objdir,
-                      worktree = worktree,
-                      relpath = relpath,
-                      revisionExpr = revisionExpr,
-                      revisionId = None,
-                      rebase = rebase,
-                      groups = groups,
-                      sync_c = sync_c,
-                      sync_s = sync_s,
-                      sync_tags = sync_tags,
-                      clone_depth = clone_depth,
-                      upstream = upstream,
-                      parent = parent,
-                      dest_branch = dest_branch,
+    project = Project(manifest=self,
+                      name=name,
+                      remote=remote.ToRemoteSpec(name),
+                      gitdir=gitdir,
+                      objdir=objdir,
+                      worktree=worktree,
+                      relpath=relpath,
+                      revisionExpr=revisionExpr,
+                      revisionId=None,
+                      rebase=rebase,
+                      groups=groups,
+                      sync_c=sync_c,
+                      sync_s=sync_s,
+                      sync_tags=sync_tags,
+                      clone_depth=clone_depth,
+                      upstream=upstream,
+                      parent=parent,
+                      dest_branch=dest_branch,
                      **extra_proj_attrs)

    for n in node.childNodes:
@ -911,7 +913,7 @@ class XmlManifest(object):
      if n.nodeName == 'annotation':
        self._ParseAnnotation(project, n)
      if n.nodeName == 'project':
-        project.subprojects.append(self._ParseProject(n, parent = project))
+        project.subprojects.append(self._ParseProject(n, parent=project))

    return project

@ -1054,7 +1056,7 @@ class XmlManifest(object):
      keep = "true"
    if keep != "true" and keep != "false":
      raise ManifestParseError('optional "keep" attribute must be '
-            '"true" or "false"')
+                               '"true" or "false"')
    project.AddAnnotation(name, value, keep)

  def _get_remote(self, node):
@ -1065,7 +1067,7 @@ class XmlManifest(object):
    v = self._remotes.get(name)
    if not v:
      raise ManifestParseError("remote %s not defined in %s" %
-            (name, self.manifestFile))
+                               (name, self.manifestFile))
    return v

  def _reqatt(self, node, attname):
@ -1075,7 +1077,7 @@ class XmlManifest(object):
    v = node.getAttribute(attname)
    if not v:
      raise ManifestParseError("no %s in <%s> within %s" %
-            (attname, node.nodeName, self.manifestFile))
+                               (attname, node.nodeName, self.manifestFile))
    return v

  def projectsDiff(self, manifest):
@ -1093,7 +1095,7 @@ class XmlManifest(object):
    diff = {'added': [], 'removed': [], 'changed': [], 'unreachable': []}

    for proj in fromKeys:
-      if not proj in toKeys:
+      if proj not in toKeys:
        diff['removed'].append(fromProjects[proj])
      else:
        fromProj = fromProjects[proj]
@ -1125,7 +1127,7 @@ class GitcManifest(XmlManifest):
                                        gitc_client_name)
    self.manifestFile = os.path.join(self.gitc_client_dir, '.manifest')

-  def _ParseProject(self, node, parent = None):
+  def _ParseProject(self, node, parent=None):
    """Override _ParseProject and add support for GITC specific attributes."""
    return super(GitcManifest, self)._ParseProject(
        node, parent=parent, old_revision=node.getAttribute('old-revision'))
@ -1134,4 +1136,3 @@ class GitcManifest(XmlManifest):
    """Output GITC Specific Project attributes"""
    if p.old_revision:
      e.setAttribute('old-revision', str(p.old_revision))
-
--- a/pager.py
+++ b/pager.py
@ -27,6 +27,7 @@ pager_process = None
 old_stdout = None
 old_stderr = None

+
 def RunPager(globalConfig):
  if not os.isatty(0) or not os.isatty(1):
    return
@ -35,23 +36,25 @@ def RunPager(globalConfig):
    return

  if platform_utils.isWindows():
-    _PipePager(pager);
+    _PipePager(pager)
  else:
    _ForkPager(pager)

+
 def TerminatePager():
  global pager_process, old_stdout, old_stderr
  if pager_process:
    sys.stdout.flush()
    sys.stderr.flush()
    pager_process.stdin.close()
-    pager_process.wait();
+    pager_process.wait()
    pager_process = None
    # Restore initial stdout/err in case there is more output in this process
    # after shutting down the pager process
    sys.stdout = old_stdout
    sys.stderr = old_stderr

+
 def _PipePager(pager):
  global pager_process, old_stdout, old_stderr
  assert pager_process is None, "Only one active pager process at a time"
@ -62,6 +65,7 @@ def _PipePager(pager):
  sys.stdout = pager_process.stdin
  sys.stderr = pager_process.stdin

+
 def _ForkPager(pager):
  global active
  # This process turns into the pager; a child it forks will
@ -88,6 +92,7 @@ def _ForkPager(pager):
    print("fatal: cannot start pager '%s'" % pager, file=sys.stderr)
    sys.exit(255)

+
 def _SelectPager(globalConfig):
  try:
    return os.environ['GIT_PAGER']
@ -105,6 +110,7 @@ def _SelectPager(globalConfig):

  return 'less'

+
 def _BecomePager(pager):
  # Delaying execution of the pager until we have output
  # ready works around a long-standing bug in popularly
--- a/platform_utils.py
+++ b/platform_utils.py
@ -92,6 +92,7 @@ class _FileDescriptorStreamsNonBlocking(FileDescriptorStreams):
  """
  class Stream(object):
    """ Encapsulates a file descriptor """
+
    def __init__(self, fd, dest, std_name):
      self.fd = fd
      self.dest = dest
@ -125,6 +126,7 @@ class _FileDescriptorStreamsThreads(FileDescriptorStreams):
  non blocking I/O. This implementation requires creating threads issuing
  blocking read operations on file descriptors.
  """
+
  def __init__(self):
    super(_FileDescriptorStreamsThreads, self).__init__()
    # The queue is shared accross all threads so we can simulate the
@ -144,12 +146,14 @@ class _FileDescriptorStreamsThreads(FileDescriptorStreams):

  class QueueItem(object):
    """ Item put in the shared queue """
+
    def __init__(self, stream, data):
      self.stream = stream
      self.data = data

  class Stream(object):
    """ Encapsulates a file descriptor """
+
    def __init__(self, fd, dest, std_name, queue):
      self.fd = fd
      self.dest = dest
--- a/progress.py
+++ b/progress.py
@ -26,6 +26,7 @@ _NOT_TTY = not os.isatty(2)
 # column 0.
 CSI_ERASE_LINE = '\x1b[2K'

+
 class Progress(object):
  def __init__(self, title, total=0, units='', print_newline=False,
               always_print_percentage=False):
@ -53,9 +54,9 @@ class Progress(object):

    if self._total <= 0:
      sys.stderr.write('%s\r%s: %d,' % (
-        CSI_ERASE_LINE,
-        self._title,
-        self._done))
+          CSI_ERASE_LINE,
+          self._title,
+          self._done))
      sys.stderr.flush()
    else:
      p = (100 * self._done) / self._total
@ -63,13 +64,13 @@ class Progress(object):
      if self._lastp != p or self._always_print_percentage:
        self._lastp = p
        sys.stderr.write('%s\r%s: %3d%% (%d%s/%d%s)%s%s%s' % (
-          CSI_ERASE_LINE,
-          self._title,
-          p,
-          self._done, self._units,
-          self._total, self._units,
-          ' ' if msg else '', msg,
-          "\n" if self._print_newline else ""))
+            CSI_ERASE_LINE,
+            self._title,
+            p,
+            self._done, self._units,
+            self._total, self._units,
+            ' ' if msg else '', msg,
+            "\n" if self._print_newline else ""))
        sys.stderr.flush()

  def end(self):
@ -78,16 +79,16 @@ class Progress(object):

    if self._total <= 0:
      sys.stderr.write('%s\r%s: %d, done.\n' % (
-        CSI_ERASE_LINE,
-        self._title,
-        self._done))
+          CSI_ERASE_LINE,
+          self._title,
+          self._done))
      sys.stderr.flush()
    else:
      p = (100 * self._done) / self._total
      sys.stderr.write('%s\r%s: %3d%% (%d%s/%d%s), done.\n' % (
-        CSI_ERASE_LINE,
-        self._title,
-        p,
-        self._done, self._units,
-        self._total, self._units))
+          CSI_ERASE_LINE,
+          self._title,
+          p,
+          self._done, self._units,
+          self._total, self._units))
      sys.stderr.flush()
--- a/project.py
+++ b/project.py
@ -85,6 +85,7 @@ def not_rev(r):
 def sq(r):
  return "'" + r.replace("'", "'\''") + "'"

+
 _project_hook_list = None


@ -1256,9 +1257,7 @@ class Project(object):
      print(line[:-1])
    return p.Wait() == 0

-
 # Publish / Upload ##
-
  def WasPublished(self, branch, all_refs=None):
    """Was the branch published (uploaded) for code review?
       If so, returns the SHA-1 hash of the last published
@ -1410,9 +1409,7 @@ class Project(object):
                            R_HEADS + branch.name,
                            message=msg)

-
 # Sync ##
-
  def _ExtractArchive(self, tarpath, path=None):
    """Extract the given tar on its current location

@ -1819,9 +1816,7 @@ class Project(object):
                            patch_id,
                            self.bare_git.rev_parse('FETCH_HEAD'))

-
 # Branch Management ##
-
  def GetHeadPath(self):
    """Return the full path to the HEAD ref."""
    dotgit = os.path.join(self.worktree, '.git')
@ -2019,9 +2014,7 @@ class Project(object):
        kept.append(ReviewableBranch(self, branch, base))
    return kept

-
 # Submodule Management ##
-
  def GetRegisteredSubprojects(self):
    result = []

@ -2073,7 +2066,7 @@ class Project(object):
      gitmodules_lines = []
      fd, temp_gitmodules_path = tempfile.mkstemp()
      try:
-        os.write(fd, p.stdout)
+        os.write(fd, p.stdout.encode('utf-8'))
        os.close(fd)
        cmd = ['config', '--file', temp_gitmodules_path, '--list']
        p = GitCommand(None, cmd, capture_stdout=True, capture_stderr=True,
@ -2172,7 +2165,6 @@ class Project(object):
      result.extend(subproject.GetDerivedSubprojects())
    return result

-
 # Direct Git Commands ##
  def _CheckForImmutableRevision(self):
    try:
@ -2341,7 +2333,7 @@ class Project(object):
    else:
      branch = self.revisionExpr
    if (not self.manifest.IsMirror and is_sha1 and depth
-        and git_require((1, 8, 3))):
+            and git_require((1, 8, 3))):
      # Shallow checkout of a specific commit, fetch from that commit and not
      # the heads only as the commit might be deeper in the history.
      spec.append(branch)
@ -2482,7 +2474,7 @@ class Project(object):
        platform_utils.remove(tmpPath)
    with GetUrlCookieFile(srcUrl, quiet) as (cookiefile, proxy):
      if cookiefile:
-        cmd += ['--cookie', cookiefile, '--cookie-jar', cookiefile]
+        cmd += ['--cookie', cookiefile]
      if proxy:
        cmd += ['--proxy', proxy]
      elif 'http_proxy' in os.environ and 'darwin' == sys.platform:
@ -2627,7 +2619,7 @@ class Project(object):
                                                  (self.worktree)):
                platform_utils.rmtree(platform_utils.realpath(self.worktree))
              return self._InitGitDir(mirror_git=mirror_git, force_sync=False)
-            except:
+            except Exception:
              raise e
          raise e

@ -2760,9 +2752,31 @@ class Project(object):
      symlink_dirs += self.working_tree_dirs
    to_symlink = symlink_files + symlink_dirs
    for name in set(to_symlink):
-      dst = platform_utils.realpath(os.path.join(destdir, name))
+      # Try to self-heal a bit in simple cases.
+      dst_path = os.path.join(destdir, name)
+      src_path = os.path.join(srcdir, name)
+
+      if name in self.working_tree_dirs:
+        # If the dir is missing under .repo/projects/, create it.
+        if not os.path.exists(src_path):
+          os.makedirs(src_path)
+
+      elif name in self.working_tree_files:
+        # If it's a file under the checkout .git/ and the .repo/projects/ has
+        # nothing, move the file under the .repo/projects/ tree.
+        if not os.path.exists(src_path) and os.path.isfile(dst_path):
+          platform_utils.rename(dst_path, src_path)
+
+      # If the path exists under the .repo/projects/ and there's no symlink
+      # under the checkout .git/, recreate the symlink.
+      if name in self.working_tree_dirs or name in self.working_tree_files:
+        if os.path.exists(src_path) and not os.path.exists(dst_path):
+          platform_utils.symlink(
+              os.path.relpath(src_path, os.path.dirname(dst_path)), dst_path)
+
+      dst = platform_utils.realpath(dst_path)
      if os.path.lexists(dst):
-        src = platform_utils.realpath(os.path.join(srcdir, name))
+        src = platform_utils.realpath(src_path)
        # Fail if the links are pointing to the wrong place
        if src != dst:
          _error('%s is different in %s vs %s', name, destdir, srcdir)
@ -2850,7 +2864,7 @@ class Project(object):
        try:
          platform_utils.rmtree(dotgit)
          return self._InitWorkTree(force_sync=False, submodules=submodules)
-        except:
+        except Exception:
          raise e
      raise e

@ -3111,9 +3125,6 @@ class Project(object):
          raise TypeError('%s() got an unexpected keyword argument %r'
                          % (name, k))
        if config is not None:
-          if not git_require((1, 7, 2)):
-            raise ValueError('cannot set config on command line for %s()'
-                             % name)
          for k, v in config.items():
            cmdv.append('-c')
            cmdv.append('%s=%s' % (k, v))
--- a/pyversion.py
+++ b/pyversion.py
@ -16,5 +16,6 @@

 import sys

+
 def is_python3():
  return sys.version_info[0] == 3
--- a/218
+++ b/218
@ -10,6 +10,7 @@ copy of repo in the checkout.

 from __future__ import print_function

+import datetime
 import os
 import platform
 import subprocess
@ -24,7 +25,7 @@ def exec_command(cmd):
      sys.exit(ret)
    else:
      os.execvp(cmd[0], cmd)
-  except:
+  except Exception:
    pass


@ -113,7 +114,7 @@ if not REPO_REV:
 # limitations under the License.

 # increment this whenever we make important changes to this script
-VERSION = (2, 0)
+VERSION = (2, 3)

 # increment this if the MAINTAINER_KEYS block is modified
 KEYRING_VERSION = (2, 0)
@ -166,7 +167,12 @@ TACbBS+Up3RpfYVfd63c1cDdlru13pQAn3NQy/SN858MkxN+zym86UBgOad2
 """

 GIT = 'git'                      # our git command
-MIN_GIT_VERSION = (2, 10, 2)     # minimum supported git version
+# NB: The version of git that the repo launcher requires may be much older than
+# the version of git that the main repo source tree requires.  Keeping this at
+# an older version also makes it easier for users to upgrade/rollback as needed.
+#
+# git-1.7 is in (EOL) Ubuntu Precise.
+MIN_GIT_VERSION = (1, 7, 2)      # minimum supported git version
 repodir = '.repo'                # name of repo's private directory
 S_repo = 'repo'                  # special repo repository
 S_manifests = 'manifests'        # special manifest repository
@ -199,88 +205,73 @@ gpg_dir = os.path.join(home_dot_repo, 'gnupg')
 extra_args = []
 init_optparse = optparse.OptionParser(usage="repo init -u url [options]")

-# Logging
-group = init_optparse.add_option_group('Logging options')
-group.add_option('-q', '--quiet',
-                 dest="quiet", action="store_true", default=False,
-                 help="be quiet")
+def _InitParser():
+  """Setup the init subcommand parser."""
+  # Logging.
+  group = init_optparse.add_option_group('Logging options')
+  group.add_option('-q', '--quiet',
+                   action='store_true', default=False,
+                   help='be quiet')

-# Manifest
-group = init_optparse.add_option_group('Manifest options')
-group.add_option('-u', '--manifest-url',
-                 dest='manifest_url',
-                 help='manifest repository location', metavar='URL')
-group.add_option('-b', '--manifest-branch',
-                 dest='manifest_branch',
-                 help='manifest branch or revision', metavar='REVISION')
-group.add_option('-m', '--manifest-name',
-                 dest='manifest_name',
-                 help='initial manifest file', metavar='NAME.xml')
-group.add_option('--current-branch',
-                 dest='current_branch_only', action='store_true',
-                 help='fetch only current manifest branch from server')
-group.add_option('--mirror',
-                 dest='mirror', action='store_true',
-                 help='create a replica of the remote repositories '
-                      'rather than a client working directory')
-group.add_option('--reference',
-                 dest='reference',
-                 help='location of mirror directory', metavar='DIR')
-group.add_option('--dissociate',
-                 dest='dissociate', action='store_true',
-                 help='dissociate from reference mirrors after clone')
-group.add_option('--depth', type='int', default=None,
-                 dest='depth',
-                 help='create a shallow clone with given depth; see git clone')
-group.add_option('--partial-clone', action='store_true',
-                 dest='partial_clone',
-                 help='perform partial clone (https://git-scm.com/'
-                 'docs/gitrepository-layout#_code_partialclone_code)')
-group.add_option('--clone-filter', action='store', default='blob:none',
-                 dest='clone_filter',
-                 help='filter for use with --partial-clone [default: %default]')
-group.add_option('--archive',
-                 dest='archive', action='store_true',
-                 help='checkout an archive instead of a git repository for '
-                      'each project. See git archive.')
-group.add_option('--submodules',
-                 dest='submodules', action='store_true',
-                 help='sync any submodules associated with the manifest repo')
-group.add_option('-g', '--groups',
-                 dest='groups', default='default',
-                 help='restrict manifest projects to ones with specified '
-                      'group(s) [default|all|G1,G2,G3|G4,-G5,-G6]',
-                 metavar='GROUP')
-group.add_option('-p', '--platform',
-                 dest='platform', default="auto",
-                 help='restrict manifest projects to ones with a specified '
-                      'platform group [auto|all|none|linux|darwin|...]',
-                 metavar='PLATFORM')
-group.add_option('--no-clone-bundle',
-                 dest='no_clone_bundle', action='store_true',
-                 help='disable use of /clone.bundle on HTTP/HTTPS')
-group.add_option('--no-tags',
-                 dest='no_tags', action='store_true',
-                 help="don't fetch tags in the manifest")
+  # Manifest.
+  group = init_optparse.add_option_group('Manifest options')
+  group.add_option('-u', '--manifest-url',
+                   help='manifest repository location', metavar='URL')
+  group.add_option('-b', '--manifest-branch',
+                   help='manifest branch or revision', metavar='REVISION')
+  group.add_option('-m', '--manifest-name',
+                   help='initial manifest file', metavar='NAME.xml')
+  group.add_option('--current-branch',
+                   dest='current_branch_only', action='store_true',
+                   help='fetch only current manifest branch from server')
+  group.add_option('--mirror', action='store_true',
+                   help='create a replica of the remote repositories '
+                        'rather than a client working directory')
+  group.add_option('--reference',
+                   help='location of mirror directory', metavar='DIR')
+  group.add_option('--dissociate', action='store_true',
+                   help='dissociate from reference mirrors after clone')
+  group.add_option('--depth', type='int', default=None,
+                   help='create a shallow clone with given depth; '
+                        'see git clone')
+  group.add_option('--partial-clone', action='store_true',
+                   help='perform partial clone (https://git-scm.com/'
+                        'docs/gitrepository-layout#_code_partialclone_code)')
+  group.add_option('--clone-filter', action='store', default='blob:none',
+                   help='filter for use with --partial-clone '
+                        '[default: %default]')
+  group.add_option('--archive', action='store_true',
+                   help='checkout an archive instead of a git repository for '
+                        'each project. See git archive.')
+  group.add_option('--submodules', action='store_true',
+                   help='sync any submodules associated with the manifest repo')
+  group.add_option('-g', '--groups', default='default',
+                   help='restrict manifest projects to ones with specified '
+                        'group(s) [default|all|G1,G2,G3|G4,-G5,-G6]',
+                   metavar='GROUP')
+  group.add_option('-p', '--platform', default='auto',
+                   help='restrict manifest projects to ones with a specified '
+                        'platform group [auto|all|none|linux|darwin|...]',
+                   metavar='PLATFORM')
+  group.add_option('--no-clone-bundle', action='store_true',
+                   help='disable use of /clone.bundle on HTTP/HTTPS')
+  group.add_option('--no-tags', action='store_true',
+                   help="don't fetch tags in the manifest")

+  # Tool.
+  group = init_optparse.add_option_group('repo Version options')
+  group.add_option('--repo-url', metavar='URL',
+                   help='repo repository location ($REPO_URL)')
+  group.add_option('--repo-branch', metavar='REVISION',
+                   help='repo branch or revision ($REPO_REV)')
+  group.add_option('--no-repo-verify', action='store_true',
+                   help='do not verify repo source code')

-# Tool
-group = init_optparse.add_option_group('repo Version options')
-group.add_option('--repo-url',
-                 dest='repo_url',
-                 help='repo repository location ($REPO_URL)', metavar='URL')
-group.add_option('--repo-branch',
-                 dest='repo_branch',
-                 help='repo branch or revision ($REPO_REV)', metavar='REVISION')
-group.add_option('--no-repo-verify',
-                 dest='no_repo_verify', action='store_true',
-                 help='do not verify repo source code')
-
-# Other
-group = init_optparse.add_option_group('Other options')
-group.add_option('--config-name',
-                 dest='config_name', action="store_true", default=False,
-                 help='Always prompt for name/e-mail')
+  # Other.
+  group = init_optparse.add_option_group('Other options')
+  group.add_option('--config-name',
+                   action='store_true', default=False,
+                   help='Always prompt for name/e-mail')


 def _GitcInitOptions(init_optparse_arg):
@ -488,6 +479,39 @@ def _CheckGitVersion():
    raise CloneFailure()


+def SetGitTrace2ParentSid(env=None):
+  """Set up GIT_TRACE2_PARENT_SID for git tracing."""
+  # We roughly follow the format git itself uses in trace2/tr2_sid.c.
+  # (1) Be unique (2) be valid filename (3) be fixed length.
+  #
+  # Since we always export this variable, we try to avoid more expensive calls.
+  # e.g. We don't attempt hostname lookups or hashing the results.
+  if env is None:
+    env = os.environ
+
+  KEY = 'GIT_TRACE2_PARENT_SID'
+
+  now = datetime.datetime.utcnow()
+  value = 'repo-%s-P%08x' % (now.strftime('%Y%m%dT%H%M%SZ'), os.getpid())
+
+  # If it's already set, then append ourselves.
+  if KEY in env:
+    value = env[KEY] + '/' + value
+
+  _setenv(KEY, value, env=env)
+
+
+def _setenv(key, value, env=None):
+  """Set |key| in the OS environment |env| to |value|."""
+  if env is None:
+    env = os.environ
+  # Environment handling across systems is messy.
+  try:
+    env[key] = value
+  except UnicodeEncodeError:
+    env[key] = value.encode()
+
+
 def NeedSetupGnuPG():
  if not os.path.isdir(home_dot_repo):
    return True
@ -524,10 +548,7 @@ def SetupGnuPG(quiet):
      sys.exit(1)

  env = os.environ.copy()
-  try:
-    env['GNUPGHOME'] = gpg_dir
-  except UnicodeEncodeError:
-    env['GNUPGHOME'] = gpg_dir.encode()
+  _setenv('GNUPGHOME', gpg_dir, env)

  cmd = ['gpg', '--import']
  try:
@ -733,10 +754,7 @@ def _Verify(cwd, branch, quiet):
      print(file=sys.stderr)

  env = os.environ.copy()
-  try:
-    env['GNUPGHOME'] = gpg_dir
-  except UnicodeEncodeError:
-    env['GNUPGHOME'] = gpg_dir.encode()
+  _setenv('GNUPGHOME', gpg_dir, env)

  cmd = [GIT, 'tag', '-v', cur]
  proc = subprocess.Popen(cmd,
@ -801,6 +819,7 @@ def _FindRepo():

 class _Options(object):
  help = False
+  version = False


 def _ParseArguments(args):
@ -812,7 +831,8 @@ def _ParseArguments(args):
    a = args[i]
    if a == '-h' or a == '--help':
      opt.help = True
-
+    elif a == '--version':
+      opt.version = True
    elif not a.startswith('-'):
      cmd = a
      arg = args[i + 1:]
@ -859,6 +879,16 @@ def _Help(args):
  sys.exit(1)


+def _Version():
+  """Show version information."""
+  print('<repo not installed>')
+  print('repo launcher version %s' % ('.'.join(str(x) for x in VERSION),))
+  print('       (from %s)' % (__file__,))
+  print('git %s' % (ParseGitVersion().full,))
+  print('Python %s' % sys.version)
+  sys.exit(0)
+
+
 def _NotInstalled():
  print('error: repo is not installed.  Use "repo init" to install it here.',
        file=sys.stderr)
@ -911,6 +941,9 @@ def _SetDefaultsTo(gitdir):
 def main(orig_args):
  cmd, opt, args = _ParseArguments(orig_args)

+  # We run this early as we run some git commands ourselves.
+  SetGitTrace2ParentSid()
+
  repo_main, rel_repo_dir = None, None
  # Don't use the local repo copy, make sure to switch to the gitc client first.
  if cmd != 'gitc-init':
@ -926,11 +959,14 @@ def main(orig_args):
          'command from the corresponding client under /gitc/',
          file=sys.stderr)
    sys.exit(1)
+  _InitParser()
  if not repo_main:
    if opt.help:
      _Usage()
    if cmd == 'help':
      _Help(args)
+    if opt.version or cmd == 'version':
+      _Version()
    if not cmd:
      _NotInstalled()
    if cmd == 'init' or cmd == 'gitc-init':
--- a/repo_trace.py
+++ b/repo_trace.py
@ -28,13 +28,16 @@ REPO_TRACE = 'REPO_TRACE'

 _TRACE = os.environ.get(REPO_TRACE) == '1'

+
 def IsTrace():
  return _TRACE

+
 def SetTrace():
  global _TRACE
  _TRACE = True

+
 def Trace(fmt, *args):
  if IsTrace():
    print(fmt % args, file=sys.stderr)
--- a/subcmds/init.py
+++ b/subcmds/init.py
@ -40,7 +40,7 @@ for py in os.listdir(my_dir):
      cmd = getattr(mod, clsn)()
    except AttributeError:
      raise SyntaxError('%s/%s does not define class %s' % (
-                         __name__, py, clsn))
+          __name__, py, clsn))

    name = name.replace('_', '-')
    cmd.NAME = name
--- a/subcmds/abandon.py
+++ b/subcmds/abandon.py
@ -21,6 +21,7 @@ from collections import defaultdict
 from git_command import git
 from progress import Progress

+
 class Abandon(Command):
  common = True
  helpSummary = "Permanently abandon a development branch"
@ -32,6 +33,7 @@ deleting it (and all its history) from your local repository.

 It is equivalent to "git branch -D <branchname>".
 """
+
  def _Options(self, p):
    p.add_option('--all',
                 dest='all', action='store_true',
@ -79,10 +81,10 @@ It is equivalent to "git branch -D <branchname>".

    if err:
      for br in err.keys():
-        err_msg = "error: cannot abandon %s" %br
+        err_msg = "error: cannot abandon %s" % br
        print(err_msg, file=sys.stderr)
        for proj in err[br]:
-          print(' '*len(err_msg) + " | %s" % proj.relpath, file=sys.stderr)
+          print(' ' * len(err_msg) + " | %s" % proj.relpath, file=sys.stderr)
      sys.exit(1)
    elif not success:
      print('error: no project has local branch(es) : %s' % nb,
@ -95,5 +97,5 @@ It is equivalent to "git branch -D <branchname>".
          result = "all project"
        else:
          result = "%s" % (
-            ('\n'+' '*width + '| ').join(p.relpath for p in success[br]))
-        print("%s%s| %s\n" % (br,' '*(width-len(br)), result),file=sys.stderr)
+              ('\n' + ' ' * width + '| ').join(p.relpath for p in success[br]))
+        print("%s%s| %s\n" % (br, ' ' * (width - len(br)), result), file=sys.stderr)
--- a/subcmds/branches.py
+++ b/subcmds/branches.py
@ -19,13 +19,15 @@ import sys
 from color import Coloring
 from command import Command

+
 class BranchColoring(Coloring):
  def __init__(self, config):
    Coloring.__init__(self, config, 'branch')
    self.current = self.printer('current', fg='green')
-    self.local   = self.printer('local')
+    self.local = self.printer('local')
    self.notinproject = self.printer('notinproject', fg='red')

+
 class BranchInfo(object):
  def __init__(self, name):
    self.name = name
@ -158,7 +160,7 @@ is shown, then the branch appears in all projects.
          for b in i.projects:
            have.add(b.project)
          for p in projects:
-            if not p in have:
+            if p not in have:
              paths.append(p.relpath)

        s = ' %s %s' % (in_type, ', '.join(paths))
@ -170,11 +172,11 @@ is shown, then the branch appears in all projects.
          fmt = out.current if i.IsCurrent else out.write
          for p in paths:
            out.nl()
-            fmt(width*' ' + '          %s' % p)
+            fmt(width * ' ' + '          %s' % p)
          fmt = out.write
          for p in non_cur_paths:
            out.nl()
-            fmt(width*' ' + '          %s' % p)
+            fmt(width * ' ' + '          %s' % p)
      else:
        out.write(' in all projects')
      out.nl()
--- a/subcmds/checkout.py
+++ b/subcmds/checkout.py
@ -19,6 +19,7 @@ import sys
 from command import Command
 from progress import Progress

+
 class Checkout(Command):
  common = True
  helpSummary = "Checkout a branch for development"
--- a/subcmds/cherry_pick.py
+++ b/subcmds/cherry_pick.py
@ -22,6 +22,7 @@ from git_command import GitCommand

 CHANGE_ID_RE = re.compile(r'^\s*Change-Id: I([0-9a-f]{40})\s*$')

+
 class CherryPick(Command):
  common = True
  helpSummary = "Cherry-pick a change."
@ -46,8 +47,8 @@ change id will be added.

    p = GitCommand(None,
                   ['rev-parse', '--verify', reference],
-                   capture_stdout = True,
-                   capture_stderr = True)
+                   capture_stdout=True,
+                   capture_stderr=True)
    if p.Wait() != 0:
      print(p.stderr, file=sys.stderr)
      sys.exit(1)
@ -61,8 +62,8 @@ change id will be added.

    p = GitCommand(None,
                   ['cherry-pick', sha1],
-                   capture_stdout = True,
-                   capture_stderr = True)
+                   capture_stdout=True,
+                   capture_stderr=True)
    status = p.Wait()

    print(p.stdout, file=sys.stdout)
@ -74,9 +75,9 @@ change id will be added.
      new_msg = self._Reformat(old_msg, sha1)

      p = GitCommand(None, ['commit', '--amend', '-F', '-'],
-                     provide_stdin = True,
-                     capture_stdout = True,
-                     capture_stderr = True)
+                     provide_stdin=True,
+                     capture_stdout=True,
+                     capture_stderr=True)
      p.stdin.write(new_msg)
      p.stdin.close()
      if p.Wait() != 0:
@ -97,7 +98,7 @@ change id will be added.

  def _StripHeader(self, commit_msg):
    lines = commit_msg.splitlines()
-    return "\n".join(lines[lines.index("")+1:])
+    return "\n".join(lines[lines.index("") + 1:])

  def _Reformat(self, old_msg, sha1):
    new_msg = []
--- a/subcmds/diff.py
+++ b/subcmds/diff.py
@ -16,6 +16,7 @@

 from command import PagedCommand

+
 class Diff(PagedCommand):
  common = True
  helpSummary = "Show changes between commit and working tree"
--- a/subcmds/diffmanifests.py
+++ b/subcmds/diffmanifests.py
@ -18,10 +18,12 @@ from color import Coloring
 from command import PagedCommand
 from manifest_xml import XmlManifest

+
 class _Coloring(Coloring):
  def __init__(self, config):
    Coloring.__init__(self, config, "status")

+
 class Diffmanifests(PagedCommand):
  """ A command to see logs in projects represented by manifests

@ -184,10 +186,10 @@ synced and their revisions won't be found.
    self.out = _Coloring(self.manifest.globalConfig)
    self.printText = self.out.nofmt_printer('text')
    if opt.color:
-      self.printProject = self.out.nofmt_printer('project', attr = 'bold')
-      self.printAdded = self.out.nofmt_printer('green', fg = 'green', attr = 'bold')
-      self.printRemoved = self.out.nofmt_printer('red', fg = 'red', attr = 'bold')
-      self.printRevision = self.out.nofmt_printer('revision', fg = 'yellow')
+      self.printProject = self.out.nofmt_printer('project', attr='bold')
+      self.printAdded = self.out.nofmt_printer('green', fg='green', attr='bold')
+      self.printRemoved = self.out.nofmt_printer('red', fg='red', attr='bold')
+      self.printRevision = self.out.nofmt_printer('revision', fg='yellow')
    else:
      self.printProject = self.printAdded = self.printRemoved = self.printRevision = self.printText

--- a/subcmds/download.py
+++ b/subcmds/download.py
@ -23,6 +23,7 @@ from error import GitError

 CHANGE_RE = re.compile(r'^([1-9][0-9]*)(?:[/\.-]([1-9][0-9]*))?$')

+
 class Download(Command):
  common = True
  helpSummary = "Download and checkout a change"
@ -93,7 +94,7 @@ If no project is specified try to use current directory as a project.
        continue

      if len(dl.commits) > 1:
-        print('[%s] %d/%d depends on %d unmerged changes:' \
+        print('[%s] %d/%d depends on %d unmerged changes:'
              % (project.name, change_id, ps_id, len(dl.commits)),
              file=sys.stderr)
        for c in dl.commits:
@ -102,7 +103,7 @@ If no project is specified try to use current directory as a project.
        try:
          project._CherryPick(dl.commit)
        except GitError:
-          print('[%s] Could not complete the cherry-pick of %s' \
+          print('[%s] Could not complete the cherry-pick of %s'
                % (project.name, dl.commit), file=sys.stderr)
          sys.exit(1)

--- a/subcmds/forall.py
+++ b/subcmds/forall.py
@ -28,10 +28,10 @@ from command import Command, MirrorSafeCommand
 import platform_utils

 _CAN_COLOR = [
-  'branch',
-  'diff',
-  'grep',
-  'log',
+    'branch',
+    'diff',
+    'grep',
+    'log',
 ]


@ -170,14 +170,14 @@ without iterating through the remaining projects.
    else:
      lrev = None
    return {
-      'name': project.name,
-      'relpath': project.relpath,
-      'remote_name': project.remote.name,
-      'lrev': lrev,
-      'rrev': project.revisionExpr,
-      'annotations': dict((a.name, a.value) for a in project.annotations),
-      'gitdir': project.gitdir,
-      'worktree': project.worktree,
+        'name': project.name,
+        'relpath': project.relpath,
+        'remote_name': project.remote.name,
+        'lrev': lrev,
+        'rrev': project.revisionExpr,
+        'annotations': dict((a.name, a.value) for a in project.annotations),
+        'gitdir': project.gitdir,
+        'worktree': project.worktree,
    }

  def ValidateOptions(self, opt, args):
@ -195,9 +195,9 @@ without iterating through the remaining projects.
      cmd.append(cmd[0])
    cmd.extend(opt.command[1:])

-    if  opt.project_header \
-    and not shell \
-    and cmd[0] == 'git':
+    if opt.project_header \
+            and not shell \
+            and cmd[0] == 'git':
      # If this is a direct git command that can enable colorized
      # output and the user prefers coloring, add --color into the
      # command line because we are going to wrap the command into
@ -220,7 +220,7 @@ without iterating through the remaining projects.

    smart_sync_manifest_name = "smart_sync_override.xml"
    smart_sync_manifest_path = os.path.join(
-      self.manifest.manifestProject.worktree, smart_sync_manifest_name)
+        self.manifest.manifestProject.worktree, smart_sync_manifest_name)

    if os.path.isfile(smart_sync_manifest_path):
      self.manifest.Override(smart_sync_manifest_path)
@ -238,8 +238,8 @@ without iterating through the remaining projects.
    try:
      config = self.manifest.manifestProject.config
      results_it = pool.imap(
-         DoWorkWrapper,
-         self.ProjectArgs(projects, mirror, opt, cmd, shell, config))
+          DoWorkWrapper,
+          self.ProjectArgs(projects, mirror, opt, cmd, shell, config))
      pool.close()
      for r in results_it:
        rc = rc or r
@ -253,7 +253,7 @@ without iterating through the remaining projects.
    except Exception as e:
      # Catch any other exceptions raised
      print('Got an error, terminating the pool: %s: %s' %
-              (type(e).__name__, e),
+            (type(e).__name__, e),
            file=sys.stderr)
      pool.terminate()
      rc = rc or getattr(e, 'errno', 1)
@ -268,7 +268,7 @@ without iterating through the remaining projects.
        project = self._SerializeProject(p)
      except Exception as e:
        print('Project list error on project %s: %s: %s' %
-                (p.name, type(e).__name__, e),
+              (p.name, type(e).__name__, e),
              file=sys.stderr)
        return
      except KeyboardInterrupt:
@ -277,6 +277,7 @@ without iterating through the remaining projects.
        return
      yield [mirror, opt, cmd, shell, cnt, config, project]

+
 class WorkerKeyboardInterrupt(Exception):
  """ Keyboard interrupt exception for worker processes. """
  pass
@ -285,6 +286,7 @@ class WorkerKeyboardInterrupt(Exception):
 def InitWorker():
  signal.signal(signal.SIGINT, signal.SIG_IGN)

+
 def DoWorkWrapper(args):
  """ A wrapper around the DoWork() method.

@ -303,6 +305,7 @@ def DoWorkWrapper(args):

 def DoWork(project, mirror, opt, cmd, shell, cnt, config):
  env = os.environ.copy()
+
  def setenv(name, val):
    if val is None:
      val = ''
@ -331,7 +334,7 @@ def DoWork(project, mirror, opt, cmd, shell, cnt, config):
    if opt.ignore_missing:
      return 0
    if ((opt.project_header and opt.verbose)
-        or not opt.project_header):
+            or not opt.project_header):
      print('skipping %s/' % project['relpath'], file=sys.stderr)
    return 1

--- a/subcmds/gitc_delete.py
+++ b/subcmds/gitc_delete.py
@ -24,6 +24,7 @@ from pyversion import is_python3
 if not is_python3():
  input = raw_input

+
 class GitcDelete(Command, GitcClientCommand):
  common = True
  visible_everywhere = False
--- a/subcmds/grep.py
+++ b/subcmds/grep.py
@ -21,7 +21,8 @@ import sys
 from color import Coloring
 from command import PagedCommand
 from error import GitError
-from git_command import git_require, GitCommand
+from git_command import GitCommand
+

 class GrepColoring(Coloring):
  def __init__(self, config):
@ -29,6 +30,7 @@ class GrepColoring(Coloring):
    self.project = self.printer('project', attr='bold')
    self.fail = self.printer('fail', fg='red')

+
 class Grep(PagedCommand):
  common = True
  helpSummary = "Print lines matching a pattern"
@ -156,12 +158,11 @@ contain a line that matches both expressions:
                 action='callback', callback=carry,
                 help='Show only file names not containing matching lines')

-
  def Execute(self, opt, args):
    out = GrepColoring(self.manifest.manifestProject.config)

    cmd_argv = ['grep']
-    if out.is_on and git_require((1, 6, 3)):
+    if out.is_on:
      cmd_argv.append('--color')
    cmd_argv.extend(getattr(opt, 'cmd_argv', []))

--- a/subcmds/help.py
+++ b/subcmds/help.py
@ -23,6 +23,7 @@ from color import Coloring
 from command import PagedCommand, MirrorSafeCommand, GitcAvailableCommand, GitcClientCommand
 import gitc_utils

+
 class Help(PagedCommand, MirrorSafeCommand):
  common = False
  helpSummary = "Display detailed help on a command"
@ -72,13 +73,13 @@ Displays detailed usage information about a command.
      return False

    commandNames = list(sorted([name
-                    for name, command in self.commands.items()
-                    if command.common and gitc_supported(command)]))
+                                for name, command in self.commands.items()
+                                if command.common and gitc_supported(command)]))
    self._PrintCommands(commandNames)

    print(
-"See 'repo help <command>' for more information on a specific command.\n"
-"See 'repo help --all' for a complete list of recognized commands.")
+        "See 'repo help <command>' for more information on a specific command.\n"
+        "See 'repo help --all' for a complete list of recognized commands.")

  def _PrintCommandHelp(self, cmd, header_prefix=''):
    class _Out(Coloring):
--- a/subcmds/info.py
+++ b/subcmds/info.py
@ -18,10 +18,12 @@ from command import PagedCommand
 from color import Coloring
 from git_refs import R_M

+
 class _Coloring(Coloring):
  def __init__(self, config):
    Coloring.__init__(self, config, "status")

+
 class Info(PagedCommand):
  common = True
  helpSummary = "Get info on the manifest branch, current branch or unmerged branches"
@ -41,15 +43,14 @@ class Info(PagedCommand):
                 dest="local", action="store_true",
                 help="Disable all remote operations")

-
  def Execute(self, opt, args):
    self.out = _Coloring(self.manifest.globalConfig)
-    self.heading = self.out.printer('heading', attr = 'bold')
-    self.headtext = self.out.nofmt_printer('headtext', fg = 'yellow')
-    self.redtext = self.out.printer('redtext', fg = 'red')
-    self.sha = self.out.printer("sha", fg = 'yellow')
+    self.heading = self.out.printer('heading', attr='bold')
+    self.headtext = self.out.nofmt_printer('headtext', fg='yellow')
+    self.redtext = self.out.printer('redtext', fg='red')
+    self.sha = self.out.printer("sha", fg='yellow')
    self.text = self.out.nofmt_printer('text')
-    self.dimtext = self.out.printer('dimtext', attr = 'dim')
+    self.dimtext = self.out.printer('dimtext', attr='dim')

    self.opt = opt

@ -122,7 +123,7 @@ class Info(PagedCommand):
      self.printSeparator()

  def findRemoteLocalDiff(self, project):
-    #Fetch all the latest commits
+    # Fetch all the latest commits.
    if not self.opt.local:
      project.Sync_NetworkHalf(quiet=True, current_branch_only=True)

@ -195,16 +196,16 @@ class Info(PagedCommand):
      commits = branch.commits
      date = branch.date
      self.text('%s %-33s (%2d commit%s, %s)' % (
-        branch.name == project.CurrentBranch and '*' or ' ',
-        branch.name,
-        len(commits),
-        len(commits) != 1 and 's' or '',
-        date))
+          branch.name == project.CurrentBranch and '*' or ' ',
+          branch.name,
+          len(commits),
+          len(commits) != 1 and 's' or '',
+          date))
      self.out.nl()

      for commit in commits:
        split = commit.split()
-        self.text('{0:38}{1} '.format('','-'))
+        self.text('{0:38}{1} '.format('', '-'))
        self.sha(split[0] + " ")
        self.text(" ".join(split[1:]))
        self.out.nl()
--- a/subcmds/init.py
+++ b/subcmds/init.py
@ -34,9 +34,10 @@ from command import InteractiveCommand, MirrorSafeCommand
 from error import ManifestParseError
 from project import SyncBuffer
 from git_config import GitConfig
-from git_command import git_require, MIN_GIT_VERSION
+from git_command import git_require, MIN_GIT_VERSION_SOFT, MIN_GIT_VERSION_HARD
 import platform_utils

+
 class Init(InteractiveCommand, MirrorSafeCommand):
  common = True
  helpSummary = "Initialize repo in the current directory"
@ -223,7 +224,7 @@ to update the working directory files.
    platformize = lambda x: 'platform-' + x
    if opt.platform == 'auto':
      if (not opt.mirror and
-          not m.config.GetString('repo.mirror') == 'true'):
+              not m.config.GetString('repo.mirror') == 'true'):
        groups.append(platformize(platform.system().lower()))
    elif opt.platform == 'all':
      groups.extend(map(platformize, all_platforms))
@ -280,10 +281,10 @@ to update the working directory files.
      m.config.SetString('repo.submodules', 'true')

    if not m.Sync_NetworkHalf(is_new=is_new, quiet=opt.quiet,
-        clone_bundle=not opt.no_clone_bundle,
-        current_branch_only=opt.current_branch_only,
-        no_tags=opt.no_tags, submodules=opt.submodules,
-        clone_filter=opt.clone_filter):
+                              clone_bundle=not opt.no_clone_bundle,
+                              current_branch_only=opt.current_branch_only,
+                              no_tags=opt.no_tags, submodules=opt.submodules,
+                              clone_filter=opt.clone_filter):
      r = m.GetRemote(m.remote.name)
      print('fatal: cannot obtain manifest %s' % r.url, file=sys.stderr)

@ -349,7 +350,7 @@ to update the working directory files.

    while True:
      print()
-      name  = self._Prompt('Your Name', mp.UserName)
+      name = self._Prompt('Your Name', mp.UserName)
      email = self._Prompt('Your Email', mp.UserEmail)

      print()
@ -451,7 +452,12 @@ to update the working directory files.
      self.OptionParser.error('--mirror and --archive cannot be used together.')

  def Execute(self, opt, args):
-    git_require(MIN_GIT_VERSION, fail=True)
+    git_require(MIN_GIT_VERSION_HARD, fail=True)
+    if not git_require(MIN_GIT_VERSION_SOFT):
+      print('repo: warning: git-%s+ will soon be required; please upgrade your '
+            'version of git to maintain support.'
+            % ('.'.join(str(x) for x in MIN_GIT_VERSION_SOFT),),
+            file=sys.stderr)

    self._SyncManifest(opt)
    self._LinkManifest(opt.manifest_name)
--- a/subcmds/list.py
+++ b/subcmds/list.py
@ -15,10 +15,10 @@
 # limitations under the License.

 from __future__ import print_function
-import sys

 from command import Command, MirrorSafeCommand

+
 class List(Command, MirrorSafeCommand):
  common = True
  helpSummary = "List projects and their associated directories"
@ -77,7 +77,7 @@ This is similar to running: repo forall -c 'echo "$REPO_PATH : $REPO_PROJECT"'.
    lines = []
    for project in projects:
      if opt.name_only and not opt.path_only:
-        lines.append("%s" % ( project.name))
+        lines.append("%s" % (project.name))
      elif opt.path_only and not opt.name_only:
        lines.append("%s" % (_getpath(project)))
      else:
--- a/subcmds/manifest.py
+++ b/subcmds/manifest.py
@ -20,6 +20,7 @@ import sys

 from command import PagedCommand

+
 class Manifest(PagedCommand):
  common = False
  helpSummary = "Manifest inspection utility"
@ -66,8 +67,8 @@ in a Git repository for use during future 'repo init' invocations.
    else:
      fd = open(opt.output_file, 'w')
    self.manifest.Save(fd,
-                       peg_rev = opt.peg_rev,
-                       peg_rev_upstream = opt.peg_rev_upstream)
+                       peg_rev=opt.peg_rev,
+                       peg_rev_upstream=opt.peg_rev_upstream)
    fd.close()
    if opt.output_file != '-':
      print('Saved manifest to %s' % opt.output_file, file=sys.stderr)
--- a/subcmds/prune.py
+++ b/subcmds/prune.py
@ -18,6 +18,7 @@ from __future__ import print_function
 from color import Coloring
 from command import PagedCommand

+
 class Prune(PagedCommand):
  common = True
  helpSummary = "Prune (delete) already merged topics"
--- a/subcmds/rebase.py
+++ b/subcmds/rebase.py
@ -43,8 +43,8 @@ branch but need to incorporate new upstream changes "underneath" them.

  def _Options(self, p):
    p.add_option('-i', '--interactive',
-                dest="interactive", action="store_true",
-                help="interactive rebase (single project only)")
+                 dest="interactive", action="store_true",
+                 help="interactive rebase (single project only)")

    p.add_option('--fail-fast',
                 dest='fail_fast', action='store_true',
@ -82,7 +82,7 @@ branch but need to incorporate new upstream changes "underneath" them.
            file=sys.stderr)
      if len(args) == 1:
        print('note: project %s is mapped to more than one path' % (args[0],),
-            file=sys.stderr)
+              file=sys.stderr)
      return 1

    # Setup the common git rebase args that we use for all projects.
--- a/subcmds/selfupdate.py
+++ b/subcmds/selfupdate.py
@ -22,6 +22,7 @@ from command import Command, MirrorSafeCommand
 from subcmds.sync import _PostRepoUpgrade
 from subcmds.sync import _PostRepoFetch

+
 class Selfupdate(Command, MirrorSafeCommand):
  common = False
  helpSummary = "Update repo to the latest version"
@ -59,5 +60,5 @@ need to be performed by an end-user.

      rp.bare_git.gc('--auto')
      _PostRepoFetch(rp,
-                     no_repo_verify = opt.no_repo_verify,
-                     verbose = True)
+                     no_repo_verify=opt.no_repo_verify,
+                     verbose=True)
--- a/subcmds/smartsync.py
+++ b/subcmds/smartsync.py
@ -16,6 +16,7 @@

 from subcmds.sync import Sync

+
 class Smartsync(Sync):
  common = True
  helpSummary = "Update working tree to the latest known good revision"
--- a/subcmds/stage.py
+++ b/subcmds/stage.py
@ -21,6 +21,7 @@ from color import Coloring
 from command import InteractiveCommand
 from git_command import GitCommand

+
 class _ProjectList(Coloring):
  def __init__(self, gc):
    Coloring.__init__(self, gc, 'interactive')
@ -28,6 +29,7 @@ class _ProjectList(Coloring):
    self.header = self.printer('header', attr='bold')
    self.help = self.printer('help', fg='red', attr='bold')

+
 class Stage(InteractiveCommand):
  common = True
  helpSummary = "Stage file(s) for commit"
@ -105,6 +107,7 @@ The '%prog' command stages files to prepare the next commit.
        continue
    print('Bye.')

+
 def _AddI(project):
  p = GitCommand(project, ['add', '--interactive'], bare=False)
  p.Wait()
--- a/subcmds/start.py
+++ b/subcmds/start.py
@ -25,6 +25,7 @@ import gitc_utils
 from progress import Progress
 from project import SyncBuffer

+
 class Start(Command):
  common = True
  helpSummary = "Start a new branch for development"
@ -60,7 +61,7 @@ revision specified in the manifest.
    if not opt.all:
      projects = args[1:]
      if len(projects) < 1:
-        projects = ['.',]  # start it in the local project by default
+        projects = ['.']  # start it in the local project by default

    all_projects = self.GetProjects(projects,
                                    missing_ok=bool(self.gitc_manifest))
@ -113,7 +114,7 @@ revision specified in the manifest.
          branch_merge = self.manifest.default.revisionExpr

      if not project.StartBranch(
-          nb, branch_merge=branch_merge, revision=opt.revision):
+              nb, branch_merge=branch_merge, revision=opt.revision):
        err.append(project)
    pm.end()

--- a/subcmds/status.py
+++ b/subcmds/status.py
@ -31,6 +31,7 @@ import os
 from color import Coloring
 import platform_utils

+
 class Status(PagedCommand):
  common = True
  helpSummary = "Show the working tree status"
@ -126,8 +127,8 @@ the following meanings:
        continue
      if item in proj_dirs_parents:
        self._FindOrphans(glob.glob('%s/.*' % item) +
-            glob.glob('%s/*' % item),
-            proj_dirs, proj_dirs_parents, outstring)
+                          glob.glob('%s/*' % item),
+                          proj_dirs, proj_dirs_parents, outstring)
        continue
      outstring.append(''.join([status_header, item, '/']))

@ -170,8 +171,8 @@ the following meanings:
      class StatusColoring(Coloring):
        def __init__(self, config):
          Coloring.__init__(self, config, 'status')
-          self.project = self.printer('header', attr = 'bold')
-          self.untracked = self.printer('untracked', fg = 'red')
+          self.project = self.printer('header', attr='bold')
+          self.untracked = self.printer('untracked', fg='red')

      orig_path = os.getcwd()
      try:
@ -179,8 +180,8 @@ the following meanings:

        outstring = []
        self._FindOrphans(glob.glob('.*') +
-            glob.glob('*'),
-            proj_dirs, proj_dirs_parents, outstring)
+                          glob.glob('*'),
+                          proj_dirs, proj_dirs_parents, outstring)

        if outstring:
          output = StatusColoring(self.manifest.globalConfig)
--- a/subcmds/sync.py
+++ b/subcmds/sync.py
@ -53,6 +53,7 @@ except ImportError:

 try:
  import resource
+
  def _rlimit_nofile():
    return resource.getrlimit(resource.RLIMIT_NOFILE)
 except ImportError:
@ -81,13 +82,16 @@ from manifest_xml import GitcManifest

 _ONE_DAY_S = 24 * 60 * 60

+
 class _FetchError(Exception):
  """Internal error thrown in _FetchHelper() when we don't want stack trace."""
  pass

+
 class _CheckoutError(Exception):
  """Internal error thrown in _CheckoutOne() when we don't want stack trace."""

+
 class Sync(Command, MirrorSafeCommand):
  jobs = 1
  common = True
@ -217,7 +221,7 @@ later is required to fix a server side protocol bug.
    p.add_option('-l', '--local-only',
                 dest='local_only', action='store_true',
                 help="only update working tree, don't fetch")
-    p.add_option('--no-manifest-update','--nmu',
+    p.add_option('--no-manifest-update', '--nmu',
                 dest='mp_update', action='store_false', default='true',
                 help='use the existing manifest checkout as-is. '
                      '(do not update to the latest revision)')
@ -327,14 +331,14 @@ later is required to fix a server side protocol bug.
    try:
      try:
        success = project.Sync_NetworkHalf(
-          quiet=opt.quiet,
-          current_branch_only=opt.current_branch_only,
-          force_sync=opt.force_sync,
-          clone_bundle=not opt.no_clone_bundle,
-          no_tags=opt.no_tags, archive=self.manifest.IsArchive,
-          optimized_fetch=opt.optimized_fetch,
-          prune=opt.prune,
-          clone_filter=clone_filter)
+            quiet=opt.quiet,
+            current_branch_only=opt.current_branch_only,
+            force_sync=opt.force_sync,
+            clone_bundle=not opt.no_clone_bundle,
+            no_tags=opt.no_tags, archive=self.manifest.IsArchive,
+            optimized_fetch=opt.optimized_fetch,
+            prune=opt.prune,
+            clone_filter=clone_filter)
        self._fetch_times.Set(project, time.time() - start)

        # Lock around all the rest of the code, since printing, updating a set
@ -355,8 +359,8 @@ later is required to fix a server side protocol bug.
      except _FetchError:
        pass
      except Exception as e:
-        print('error: Cannot fetch %s (%s: %s)' \
-            % (project.name, type(e).__name__, str(e)), file=sys.stderr)
+        print('error: Cannot fetch %s (%s: %s)'
+              % (project.name, type(e).__name__, str(e)), file=sys.stderr)
        err_event.set()
        raise
    finally:
@ -396,8 +400,8 @@ later is required to fix a server side protocol bug.
                    err_event=err_event,
                    clone_filter=self.manifest.CloneFilter)
      if self.jobs > 1:
-        t = _threading.Thread(target = self._FetchProjectList,
-                              kwargs = kwargs)
+        t = _threading.Thread(target=self._FetchProjectList,
+                              kwargs=kwargs)
        # Ensure that Ctrl-C will not freeze the repo process.
        t.daemon = True
        threads.add(t)
@ -560,13 +564,23 @@ later is required to fix a server side protocol bug.
  def _GCProjects(self, projects, opt, err_event):
    gc_gitdirs = {}
    for project in projects:
+      # Make sure pruning never kicks in with shared projects.
      if len(project.manifest.GetProjectsWithName(project.name)) > 1:
-        print('Shared project %s found, disabling pruning.' % project.name)
-        project.bare_git.config('--replace-all', 'gc.pruneExpire', 'never')
+        print('%s: Shared project %s found, disabling pruning.' %
+              (project.relpath, project.name))
+        if git_require((2, 7, 0)):
+          project.config.SetString('core.repositoryFormatVersion', '1')
+          project.config.SetString('extensions.preciousObjects', 'true')
+        else:
+          # This isn't perfect, but it's the best we can do with old git.
+          print('%s: WARNING: shared projects are unreliable when using old '
+                'versions of git; please upgrade to git-2.7.0+.'
+                % (project.relpath,),
+                file=sys.stderr)
+          project.config.SetString('gc.pruneExpire', 'never')
      gc_gitdirs[project.gitdir] = project.bare_git

-    has_dash_c = git_require((1, 7, 2))
-    if multiprocessing and has_dash_c:
+    if multiprocessing:
      cpu_count = multiprocessing.cpu_count()
    else:
      cpu_count = 1
@ -588,7 +602,7 @@ later is required to fix a server side protocol bug.
          bare_git.gc('--auto', config=config)
        except GitError:
          err_event.set()
-        except:
+        except Exception:
          err_event.set()
          raise
      finally:
@ -693,16 +707,16 @@ later is required to fix a server side protocol bug.
          gitdir = os.path.join(self.manifest.topdir, path, '.git')
          if os.path.exists(gitdir):
            project = Project(
-                           manifest = self.manifest,
-                           name = path,
-                           remote = RemoteSpec('origin'),
-                           gitdir = gitdir,
-                           objdir = gitdir,
-                           worktree = os.path.join(self.manifest.topdir, path),
-                           relpath = path,
-                           revisionExpr = 'HEAD',
-                           revisionId = None,
-                           groups = None)
+                manifest=self.manifest,
+                name=path,
+                remote=RemoteSpec('origin'),
+                gitdir=gitdir,
+                objdir=gitdir,
+                worktree=os.path.join(self.manifest.topdir, path),
+                relpath=path,
+                revisionExpr='HEAD',
+                revisionId=None,
+                groups=None)

            if project.IsDirty() and opt.force_remove_dirty:
              print('WARNING: Removing dirty project "%s": uncommitted changes '
@ -733,7 +747,7 @@ later is required to fix a server side protocol bug.
    if not opt.quiet:
      print('Using manifest server %s' % manifest_server)

-    if not '@' in manifest_server:
+    if '@' not in manifest_server:
      username = None
      password = None
      if opt.manifest_server_username and opt.manifest_server_password:
@ -874,7 +888,7 @@ later is required to fix a server side protocol bug.

    manifest_name = opt.manifest_name
    smart_sync_manifest_path = os.path.join(
-      self.manifest.manifestProject.worktree, 'smart_sync_override.xml')
+        self.manifest.manifestProject.worktree, 'smart_sync_override.xml')

    if opt.smart_sync or opt.smart_tag:
      manifest_name = self._SmartSyncSetup(opt, smart_sync_manifest_path)
@ -1033,6 +1047,10 @@ later is required to fix a server side protocol bug.
            file=sys.stderr)
      sys.exit(1)

+    if not opt.quiet:
+      print('repo sync has finished successfully.')
+
+
 def _PostRepoUpgrade(manifest, quiet=False):
  wrapper = Wrapper()
  if wrapper.NeedSetupGnuPG():
@ -1041,6 +1059,7 @@ def _PostRepoUpgrade(manifest, quiet=False):
    if project.Exists:
      project.PostRepoUpgrade()

+
 def _PostRepoFetch(rp, no_repo_verify=False, verbose=False):
  if rp.HasChanges:
    print('info: A new version of repo is available', file=sys.stderr)
@ -1059,6 +1078,7 @@ def _PostRepoFetch(rp, no_repo_verify=False, verbose=False):
      print('repo version %s is current' % rp.work_git.describe(HEAD),
            file=sys.stderr)

+
 def _VerifyTag(project):
  gpg_dir = os.path.expanduser('~/.repoconfig/gnupg')
  if not os.path.exists(gpg_dir):
@ -1089,9 +1109,9 @@ def _VerifyTag(project):

  cmd = [GIT, 'tag', '-v', cur]
  proc = subprocess.Popen(cmd,
-                          stdout = subprocess.PIPE,
-                          stderr = subprocess.PIPE,
-                          env = env)
+                          stdout=subprocess.PIPE,
+                          stderr=subprocess.PIPE,
+                          env=env)
  out = proc.stdout.read()
  proc.stdout.close()

@ -1125,7 +1145,7 @@ class _FetchTimes(object):
    old = self._times.get(name, t)
    self._seen.add(name)
    a = self._ALPHA
-    self._times[name] = (a*t) + ((1-a) * old)
+    self._times[name] = (a * t) + ((1 - a) * old)

  def _Load(self):
    if self._times is None:
@ -1163,6 +1183,8 @@ class _FetchTimes(object):
 # and supporting persistent-http[s]. It cannot change hosts from
 # request to request like the normal transport, the real url
 # is passed during initialization.
+
+
 class PersistentTransport(xmlrpc.client.Transport):
  def __init__(self, orig_host):
    self.orig_host = orig_host
@ -1197,7 +1219,7 @@ class PersistentTransport(xmlrpc.client.Transport):
      if proxy:
        proxyhandler = urllib.request.ProxyHandler({
            "http": proxy,
-            "https": proxy })
+            "https": proxy})

      opener = urllib.request.build_opener(
          urllib.request.HTTPCookieProcessor(cookiejar),
@ -1254,4 +1276,3 @@ class PersistentTransport(xmlrpc.client.Transport):

  def close(self):
    pass
-
--- a/subcmds/upload.py
+++ b/subcmds/upload.py
@ -33,6 +33,7 @@ else:

 UNUSUAL_COMMIT_THRESHOLD = 5

+
 def _ConfirmManyUploads(multiple_branches=False):
  if multiple_branches:
    print('ATTENTION: One or more branches has an unusually high number '
@ -44,17 +45,20 @@ def _ConfirmManyUploads(multiple_branches=False):
  answer = input("If you are sure you intend to do this, type 'yes': ").strip()
  return answer == "yes"

+
 def _die(fmt, *args):
  msg = fmt % args
  print('error: %s' % msg, file=sys.stderr)
  sys.exit(1)

+
 def _SplitEmails(values):
  result = []
  for value in values:
    result.extend([s.strip() for s in value.split(',')])
  return result

+
 class Upload(InteractiveCommand):
  common = True
  helpSummary = "Upload changes for code review"
@ -137,13 +141,13 @@ Gerrit Code Review:  https://www.gerritcodereview.com/
                 dest='auto_topic', action='store_true',
                 help='Send local branch name to Gerrit Code Review')
    p.add_option('--re', '--reviewers',
-                 type='string',  action='append', dest='reviewers',
+                 type='string', action='append', dest='reviewers',
                 help='Request reviews from these people.')
    p.add_option('--cc',
-                 type='string',  action='append', dest='cc',
+                 type='string', action='append', dest='cc',
                 help='Also send email to these email addresses.')
    p.add_option('--br',
-                 type='string',  action='store', dest='branch',
+                 type='string', action='store', dest='branch',
                 help='Branch to upload.')
    p.add_option('--cbr', '--current-branch',
                 dest='current_branch', action='store_true',
@ -168,6 +172,9 @@ Gerrit Code Review:  https://www.gerritcodereview.com/
                 type='string', action='store', dest='dest_branch',
                 metavar='BRANCH',
                 help='Submit for review on this target branch.')
+    p.add_option('--no-cert-checks',
+                 dest='validate_certs', action='store_false', default=True,
+                 help='Disable verifying ssl certs (unsafe).')

    # Options relating to upload hook.  Note that verify and no-verify are NOT
    # opposites of each other, which is why they store to different locations.
@ -185,15 +192,16 @@ Gerrit Code Review:  https://www.gerritcodereview.com/
    #   Never run upload hooks, but upload anyway (AKA bypass hooks).
    # - no-verify=True, verify=True:
    #   Invalid
-    p.add_option('--no-cert-checks',
-                 dest='validate_certs', action='store_false', default=True,
-                 help='Disable verifying ssl certs (unsafe).')
-    p.add_option('--no-verify',
+    g = p.add_option_group('Upload hooks')
+    g.add_option('--no-verify',
                 dest='bypass_hooks', action='store_true',
                 help='Do not run the upload hook.')
-    p.add_option('--verify',
+    g.add_option('--verify',
                 dest='allow_all_hooks', action='store_true',
                 help='Run the upload hook without prompting.')
+    g.add_option('--ignore-hooks',
+                 dest='ignore_hooks', action='store_true',
+                 help='Do not abort uploading if upload hooks fail.')

  def _SingleBranch(self, opt, branch, people):
    project = branch.project
@ -214,10 +222,10 @@ Gerrit Code Review:  https://www.gerritcodereview.com/
      print('Upload project %s/ to remote branch %s%s:' %
            (project.relpath, destination, ' (draft)' if opt.draft else ''))
      print('  branch %s (%2d commit%s, %s):' % (
-                    name,
-                    len(commit_list),
-                    len(commit_list) != 1 and 's' or '',
-                    date))
+          name,
+          len(commit_list),
+          len(commit_list) != 1 and 's' or '',
+          date))
      for commit in commit_list:
        print('         %s' % commit)

@ -322,12 +330,12 @@ Gerrit Code Review:  https://www.gerritcodereview.com/

    key = 'review.%s.autoreviewer' % project.GetBranch(name).remote.review
    raw_list = project.config.GetString(key)
-    if not raw_list is None:
+    if raw_list is not None:
      people[0].extend([entry.strip() for entry in raw_list.split(',')])

    key = 'review.%s.autocopy' % project.GetBranch(name).remote.review
    raw_list = project.config.GetString(key)
-    if not raw_list is None and len(people[0]) > 0:
+    if raw_list is not None and len(people[0]) > 0:
      people[1].extend([entry.strip() for entry in raw_list.split(',')])

  def _FindGerritChange(self, branch):
@ -418,18 +426,18 @@ Gerrit Code Review:  https://www.gerritcodereview.com/
          else:
            fmt = '\n       (%s)'
          print(('[FAILED] %-15s %-15s' + fmt) % (
-                 branch.project.relpath + '/', \
-                 branch.name, \
-                 str(branch.error)),
-                 file=sys.stderr)
+              branch.project.relpath + '/',
+              branch.name,
+              str(branch.error)),
+              file=sys.stderr)
      print()

    for branch in todo:
      if branch.uploaded:
        print('[OK    ] %-15s %s' % (
-               branch.project.relpath + '/',
-               branch.name),
-               file=sys.stderr)
+            branch.project.relpath + '/',
+            branch.name),
+            file=sys.stderr)

    if have_errors:
      sys.exit(1)
@ -437,14 +445,14 @@ Gerrit Code Review:  https://www.gerritcodereview.com/
  def _GetMergeBranch(self, project):
    p = GitCommand(project,
                   ['rev-parse', '--abbrev-ref', 'HEAD'],
-                   capture_stdout = True,
-                   capture_stderr = True)
+                   capture_stdout=True,
+                   capture_stderr=True)
    p.Wait()
    local_branch = p.stdout.strip()
    p = GitCommand(project,
                   ['config', '--get', 'branch.%s.merge' % local_branch],
-                   capture_stdout = True,
-                   capture_stderr = True)
+                   capture_stdout=True,
+                   capture_stderr=True)
    p.Wait()
    merge_branch = p.stdout.strip()
    return merge_branch
@ -488,12 +496,24 @@ Gerrit Code Review:  https://www.gerritcodereview.com/
                      abort_if_user_denies=True)
      pending_proj_names = [project.name for (project, available) in pending]
      pending_worktrees = [project.worktree for (project, available) in pending]
+      passed = True
      try:
        hook.Run(opt.allow_all_hooks, project_list=pending_proj_names,
                 worktree_list=pending_worktrees)
+      except SystemExit:
+        passed = False
+        if not opt.ignore_hooks:
+          raise
      except HookError as e:
+        passed = False
        print("ERROR: %s" % str(e), file=sys.stderr)
-        return
+
+      if not passed:
+        if opt.ignore_hooks:
+          print('\nWARNING: pre-upload hooks failed, but uploading anyways.',
+                file=sys.stderr)
+        else:
+          return

    if opt.reviewers:
      reviewers = _SplitEmails(opt.reviewers)
--- a/subcmds/version.py
+++ b/subcmds/version.py
@ -20,6 +20,7 @@ from command import Command, MirrorSafeCommand
 from git_command import git, RepoSourceVersion, user_agent
 from git_refs import HEAD

+
 class Version(Command, MirrorSafeCommand):
  wrapper_version = None
  wrapper_path = None
--- a/tests/test_git_config.py
+++ b/tests/test_git_config.py
@ -23,14 +23,17 @@ import unittest

 import git_config

+
 def fixture(*paths):
  """Return a path relative to test/fixtures.
  """
  return os.path.join(os.path.dirname(__file__), 'fixtures', *paths)

+
 class GitConfigUnitTest(unittest.TestCase):
  """Tests the GitConfig class.
  """
+
  def setUp(self):
    """Create a GitConfig object using the test.gitconfig fixture.
    """
@ -68,5 +71,6 @@ class GitConfigUnitTest(unittest.TestCase):
    val = config.GetString('empty')
    self.assertEqual(val, None)

+
 if __name__ == '__main__':
  unittest.main()
--- a/tests/test_project.py
+++ b/tests/test_project.py
@ -163,7 +163,7 @@ class CopyLinkTestCase(unittest.TestCase):

  @staticmethod
  def touch(path):
-    with open(path, 'w') as f:
+    with open(path, 'w'):
      pass

  def assertExists(self, path, msg=None):
@ -256,7 +256,7 @@ class CopyFile(CopyLinkTestCase):
    cf = self.CopyFile('bar/foo.txt', 'foo')
    self.assertRaises(error.ManifestInvalidPathError, cf._Copy)

-  def test_src_block_dir(self):
+  def test_src_block_copy_from_dir(self):
    """Do not allow copying from a directory."""
    src = os.path.join(self.worktree, 'dir')
    os.makedirs(src)
@ -279,7 +279,7 @@ class CopyFile(CopyLinkTestCase):
    cf = self.CopyFile('foo.txt', 'sym/foo.txt')
    self.assertRaises(error.ManifestInvalidPathError, cf._Copy)

-  def test_src_block_dir(self):
+  def test_src_block_copy_to_dir(self):
    """Do not allow copying to a directory."""
    src = os.path.join(self.worktree, 'foo.txt')
    self.touch(src)
--- a/tests/test_wrapper.py
+++ b/tests/test_wrapper.py
@ -19,24 +19,53 @@
 from __future__ import print_function

 import os
+import re
 import unittest

+from pyversion import is_python3
 import wrapper

+
+if is_python3():
+  from unittest import mock
+  from io import StringIO
+else:
+  import mock
+  from StringIO import StringIO
+
+
 def fixture(*paths):
  """Return a path relative to tests/fixtures.
  """
  return os.path.join(os.path.dirname(__file__), 'fixtures', *paths)

-class RepoWrapperUnitTest(unittest.TestCase):
-  """Tests helper functions in the repo wrapper
-  """
+
+class RepoWrapperTestCase(unittest.TestCase):
+  """TestCase for the wrapper module."""
+
  def setUp(self):
-    """Load the wrapper module every time
-    """
+    """Load the wrapper module every time."""
    wrapper._wrapper_module = None
    self.wrapper = wrapper.Wrapper()

+    if not is_python3():
+      self.assertRegex = self.assertRegexpMatches
+
+
+class RepoWrapperUnitTest(RepoWrapperTestCase):
+  """Tests helper functions in the repo wrapper
+  """
+
+  def test_version(self):
+    """Make sure _Version works."""
+    with self.assertRaises(SystemExit) as e:
+      with mock.patch('sys.stdout', new_callable=StringIO) as stdout:
+        with mock.patch('sys.stderr', new_callable=StringIO) as stderr:
+          self.wrapper._Version()
+    self.assertEqual(0, e.exception.code)
+    self.assertEqual('', stderr.getvalue())
+    self.assertIn('repo launcher version', stdout.getvalue())
+
  def test_get_gitc_manifest_dir_no_gitc(self):
    """
    Test reading a missing gitc config file
@ -76,5 +105,38 @@ class RepoWrapperUnitTest(unittest.TestCase):
    self.assertEqual(self.wrapper.gitc_parse_clientdir('/gitc/manifest-rw/'), None)
    self.assertEqual(self.wrapper.gitc_parse_clientdir('/test/usr/local/google/gitc/'), None)

+
+class SetGitTrace2ParentSid(RepoWrapperTestCase):
+  """Check SetGitTrace2ParentSid behavior."""
+
+  KEY = 'GIT_TRACE2_PARENT_SID'
+  VALID_FORMAT = re.compile(r'^repo-[0-9]{8}T[0-9]{6}Z-P[0-9a-f]{8}$')
+
+  def test_first_set(self):
+    """Test env var not yet set."""
+    env = {}
+    self.wrapper.SetGitTrace2ParentSid(env)
+    self.assertIn(self.KEY, env)
+    value = env[self.KEY]
+    self.assertRegex(value, self.VALID_FORMAT)
+
+  def test_append(self):
+    """Test env var is appended."""
+    env = {self.KEY: 'pfx'}
+    self.wrapper.SetGitTrace2ParentSid(env)
+    self.assertIn(self.KEY, env)
+    value = env[self.KEY]
+    self.assertTrue(value.startswith('pfx/'))
+    self.assertRegex(value[4:], self.VALID_FORMAT)
+
+  def test_global_context(self):
+    """Check os.environ gets updated by default."""
+    os.environ.pop(self.KEY, None)
+    self.wrapper.SetGitTrace2ParentSid()
+    self.assertIn(self.KEY, os.environ)
+    value = os.environ[self.KEY]
+    self.assertRegex(value, self.VALID_FORMAT)
+
+
 if __name__ == '__main__':
  unittest.main()
--- a/tox.ini
+++ b/tox.ini
@ -20,3 +20,8 @@ envlist = py27, py36, py37, py38
 [testenv]
 deps = pytest
 commands = {toxinidir}/run_tests
+
+[testenv:py27]
+deps =
+    mock
+    pytest
--- a/wrapper.py
+++ b/wrapper.py
@ -27,7 +27,10 @@ import os
 def WrapperPath():
  return os.path.join(os.path.dirname(__file__), 'repo')

+
 _wrapper_module = None
+
+
 def Wrapper():
  global _wrapper_module
  if not _wrapper_module: