libv8 5.3.332.38.5 → 5.6.326.50.0beta1

data/vendor/depot_tools/gclient_utils.py

@@ -5,6 +5,7 @@
 """Generic utils."""
 
 import codecs
+import contextlib
 import cStringIO
 import datetime
 import logging
@@ -143,6 +144,16 @@ def FileWrite(filename, content, mode='w'):
     f.write(content)
 
 
+@contextlib.contextmanager
+def temporary_directory(**kwargs):
+  tdir = tempfile.mkdtemp(**kwargs)
+  try:
+    yield tdir
+  finally:
+    if tdir:
+      rmtree(tdir)
+
+
 def safe_rename(old, new):
   """Renames a file reliably.
 
@@ -179,9 +190,8 @@ def rmtree(path):
   Recursively removes a directory, even if it's marked read-only.
 
   shutil.rmtree() doesn't work on Windows if any of the files or directories
-  are read-only, which svn repositories and some .svn files are.  We need to
-  be able to force the files to be writable (i.e., deletable) as we traverse
-  the tree.
+  are read-only. We need to be able to force the files to be writable (i.e.,
+  deletable) as we traverse the tree.
 
   Even with all this, Windows still sometimes fails to delete a file, citing
   a permission error (maybe something to do with antivirus scans or disk
@@ -459,71 +469,9 @@ class GClientChildren(object):
           print >> sys.stderr, '  ', zombie.pid
 
 
-class _KillTimer(object):
-  """Timer that kills child process after certain interval since last poke or
-  creation.
-  """
-  # TODO(tandrii): we really want to make use of subprocess42 here, and not
-  # re-invent the wheel, but it's too much work :(
-
-  def __init__(self, timeout, child, child_info):
-    self._timeout = timeout
-    self._child = child
-    self._child_info = child_info
-
-    self._cv = threading.Condition()
-    # All items below are protected by condition above.
-    self._kill_at = None
-    self._killing_attempted = False
-    self._working = True
-    self._thread = None
-
-    # Start the timer immediately.
-    if self._timeout:
-      self._kill_at = time.time() + self._timeout
-      self._thread = threading.Thread(name='_KillTimer', target=self._work)
-      self._thread.daemon = True
-      self._thread.start()
-
-  @property
-  def killing_attempted(self):
-    return self._killing_attempted
-
-  def poke(self):
-    if not self._timeout:
-      return
-    with self._cv:
-      self._kill_at = time.time() + self._timeout
-
-  def cancel(self):
-    with self._cv:
-      self._working = False
-      self._cv.notifyAll()
-
-  def _work(self):
-    if not self._timeout:
-      return
-    while True:
-      with self._cv:
-        if not self._working:
-          return
-        left = self._kill_at - time.time()
-        if left > 0:
-          self._cv.wait(timeout=left)
-          continue
-        try:
-          logging.warn('killing child %s %s because of no output for %fs',
-                       self._child.pid, self._child_info, self._timeout)
-          self._killing_attempted = True
-          self._child.kill()
-        except OSError:
-          logging.exception('failed to kill child %s', self._child)
-        return
-
-
 def CheckCallAndFilter(args, stdout=None, filter_fn=None,
                        print_stdout=None, call_filter_on_first_line=False,
-                       retry=False, kill_timeout=None, **kwargs):
+                       retry=False, **kwargs):
   """Runs a command and calls back a filter function if needed.
 
   Accepts all subprocess2.Popen() parameters plus:
@@ -534,23 +482,16 @@ def CheckCallAndFilter(args, stdout=None, filter_fn=None,
     stdout: Can be any bufferable output.
     retry: If the process exits non-zero, sleep for a brief interval and try
            again, up to RETRY_MAX times.
-    kill_timeout: (float) if given, number of seconds after which process would
-           be killed. Must not be used with shell=True as only shell process
-           would be killed, but not processes spawned by shell.
 
   stderr is always redirected to stdout.
   """
   assert print_stdout or filter_fn
-  assert not kwargs.get('shell', False) or not kill_timeout, (
-      'kill_timeout should not be used with shell=True')
   stdout = stdout or sys.stdout
   output = cStringIO.StringIO()
   filter_fn = filter_fn or (lambda x: None)
 
   sleep_interval = RETRY_INITIAL_SLEEP
   run_cwd = kwargs.get('cwd', os.getcwd())
-  debug_kid_info = "'%s' in %s" % (' '.join('"%s"' % x for x in args), run_cwd)
-
   for _ in xrange(RETRY_MAX + 1):
     kid = subprocess2.Popen(
         args, bufsize=0, stdout=subprocess2.PIPE, stderr=subprocess2.STDOUT,
@@ -563,10 +504,10 @@ def CheckCallAndFilter(args, stdout=None, filter_fn=None,
 
     # Also, we need to forward stdout to prevent weird re-ordering of output.
     # This has to be done on a per byte basis to make sure it is not buffered:
-    # normally buffering is done for each line, but if svn requests input, no
-    # end-of-line character is output after the prompt and it would not show up.
+    # normally buffering is done for each line, but if the process requests
+    # input, no end-of-line character is output after the prompt and it would
+    # not show up.
     try:
-      timeout_killer = _KillTimer(kill_timeout, kid, debug_kid_info)
       in_byte = kid.stdout.read(1)
       if in_byte:
         if call_filter_on_first_line:
@@ -587,7 +528,6 @@ def CheckCallAndFilter(args, stdout=None, filter_fn=None,
         if len(in_line):
           filter_fn(in_line)
       rv = kid.wait()
-      timeout_killer.cancel()
 
       # Don't put this in a 'finally,' since the child may still run if we get
       # an exception.
@@ -601,12 +541,8 @@ def CheckCallAndFilter(args, stdout=None, filter_fn=None,
       return output.getvalue()
     if not retry:
       break
-    print ("WARNING: subprocess %s failed; will retry after a short nap..." %
-           debug_kid_info)
-    if timeout_killer.killing_attempted:
-      print('The subprocess above was likely killed because it looked hung. '
-            'Output thus far:\n> %s' %
-            ('\n> '.join(output.getvalue().splitlines())))
+    print ("WARNING: subprocess '%s' in %s failed; will retry after a short "
+           'nap...' % (' '.join('"%s"' % x for x in args), run_cwd))
     time.sleep(sleep_interval)
     sleep_interval *= 2
   raise subprocess2.CalledProcessError(
@@ -1135,7 +1071,7 @@ def GetEditor(git_editor=None):
   """Returns the most plausible editor to use.
 
   In order of preference:
-  - GIT_EDITOR/SVN_EDITOR environment variable
+  - GIT_EDITOR environment variable
   - core.editor git configuration variable (if supplied by git-cl)
   - VISUAL environment variable
   - EDITOR environment variable
@@ -1244,9 +1180,9 @@ def NumLocalCpus():
     try:
       import multiprocessing
       return multiprocessing.cpu_count()
-    except NotImplementedError:  # pylint: disable=W0702
+    except NotImplementedError:  # pylint: disable=bare-except
       # (UNIX) Query 'os.sysconf'.
-      # pylint: disable=E1101
+      # pylint: disable=no-member
       if hasattr(os, 'sysconf') and 'SC_NPROCESSORS_ONLN' in os.sysconf_names:
         return int(os.sysconf('SC_NPROCESSORS_ONLN'))
 

data/vendor/depot_tools/gerrit_client.py

@@ -0,0 +1,105 @@
+#!/usr/bin/python
+# Copyright 2017 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+"""Simple client for the Gerrit REST API.
+
+Example usage:
+  ./gerrit_client.py [command] [args]""
+"""
+
+from __future__ import print_function
+
+import json
+import logging
+import optparse
+import subcommand
+import sys
+import urllib
+import urlparse
+
+from third_party import colorama
+import fix_encoding
+import gerrit_util
+import setup_color
+
+__version__ = '0.1'
+# Shortcut since it quickly becomes redundant.
+Fore = colorama.Fore
+
+
+def write_result(result, opt):
+  if opt.json_file:
+    with open(opt.json_file, 'w') as json_file:
+      json_file.write(json.dumps(result))
+
+
+@subcommand.usage('[args ...]')
+def CMDbranchinfo(parser, args):
+  parser.add_option('--branch', dest='branch', help='branch name')
+
+  (opt, args) = parser.parse_args(args)
+  host = urlparse.urlparse(opt.host).netloc
+  project = urllib.quote_plus(opt.project)
+  branch = urllib.quote_plus(opt.branch)
+  result = gerrit_util.GetGerritBranch(host, project, branch)
+  logging.info(result)
+  write_result(result, opt)
+
+@subcommand.usage('[args ...]')
+def CMDbranch(parser, args):
+  parser.add_option('--branch', dest='branch', help='branch name')
+  parser.add_option('--commit', dest='commit', help='commit hash')
+
+  (opt, args) = parser.parse_args(args)
+
+  project = urllib.quote_plus(opt.project)
+  host = urlparse.urlparse(opt.host).netloc
+  branch = urllib.quote_plus(opt.branch)
+  commit = urllib.quote_plus(opt.commit)
+  result = gerrit_util.CreateGerritBranch(host, project, branch, commit)
+  logging.info(result)
+  write_result(result, opt)
+
+
+class OptionParser(optparse.OptionParser):
+  """Creates the option parse and add --verbose support."""
+  def __init__(self, *args, **kwargs):
+    optparse.OptionParser.__init__(
+        self, *args, prog='git cl', version=__version__, **kwargs)
+    self.add_option(
+        '--verbose', action='count', default=0,
+        help='Use 2 times for more debugging info')
+    self.add_option('--host', dest='host', help='Url of host.')
+    self.add_option('--project', dest='project', help='project name')
+    self.add_option(
+        '--json_file', dest='json_file', help='output json filepath')
+
+  def parse_args(self, args=None, values=None):
+    options, args = optparse.OptionParser.parse_args(self, args, values)
+    levels = [logging.WARNING, logging.INFO, logging.DEBUG]
+    logging.basicConfig(level=levels[min(options.verbose, len(levels) - 1)])
+    return options, args
+
+
+def main(argv):
+  if sys.hexversion < 0x02060000:
+    print('\nYour python version %s is unsupported, please upgrade.\n'
+          %(sys.version.split(' ', 1)[0],),
+          file=sys.stderr)
+    return 2
+  dispatcher = subcommand.CommandDispatcher(__name__)
+  return dispatcher.execute(OptionParser(), argv)
+
+
+if __name__ == '__main__':
+  # These affect sys.stdout so do it outside of main() to simplify mocks in
+  # unit testing.
+  fix_encoding.fix_encoding()
+  setup_color.init()
+  try:
+    sys.exit(main(sys.argv[1:]))
+  except KeyboardInterrupt:
+    sys.stderr.write('interrupted\n')
+    sys.exit(1)

data/vendor/depot_tools/gerrit_util.py

@@ -9,6 +9,7 @@ https://gerrit-review.googlesource.com/Documentation/rest-api.html
 """
 
 import base64
+import contextlib
 import cookielib
 import httplib
 import json
@@ -19,11 +20,13 @@ import re
 import socket
 import stat
 import sys
+import tempfile
 import time
 import urllib
 import urlparse
 from cStringIO import StringIO
 
+import gclient_utils
 
 LOGGER = logging.getLogger()
 TRY_LIMIT = 5
@@ -113,30 +116,46 @@ class CookiesAuthenticator(Authenticator):
 
   @classmethod
   def _get_netrc(cls):
+    # Buffer the '.netrc' path. Use an empty file if it doesn't exist.
     path = cls.get_netrc_path()
-    if not os.path.exists(path):
-      return netrc.netrc(os.devnull)
-
-    try:
-      return netrc.netrc(path)
-    except IOError:
-      print >> sys.stderr, 'WARNING: Could not read netrc file %s' % path
-      return netrc.netrc(os.devnull)
-    except netrc.NetrcParseError:
+    content = ''
+    if os.path.exists(path):
       st = os.stat(path)
       if st.st_mode & (stat.S_IRWXG | stat.S_IRWXO):
         print >> sys.stderr, (
             'WARNING: netrc file %s cannot be used because its file '
             'permissions are insecure.  netrc file permissions should be '
             '600.' % path)
-      else:
-        print >> sys.stderr, ('ERROR: Cannot use netrc file %s due to a '
-                              'parsing error.' % path)
-        raise
+      with open(path) as fd:
+        content = fd.read()
+
+    # Load the '.netrc' file. We strip comments from it because processing them
+    # can trigger a bug in Windows. See crbug.com/664664.
+    content = '\n'.join(l for l in content.splitlines()
+                        if l.strip() and not l.strip().startswith('#'))
+    with tempdir() as tdir:
+      netrc_path = os.path.join(tdir, 'netrc')
+      with open(netrc_path, 'w') as fd:
+        fd.write(content)
+      os.chmod(netrc_path, (stat.S_IRUSR | stat.S_IWUSR))
+      return cls._get_netrc_from_path(netrc_path)
+
+  @classmethod
+  def _get_netrc_from_path(cls, path):
+    try:
+      return netrc.netrc(path)
+    except IOError:
+      print >> sys.stderr, 'WARNING: Could not read netrc file %s' % path
+      return netrc.netrc(os.devnull)
+    except netrc.NetrcParseError as e:
+      print >> sys.stderr, ('ERROR: Cannot use netrc file %s due to a '
+                            'parsing error: %s' % (path, e))
       return netrc.netrc(os.devnull)
 
   @classmethod
   def get_gitcookies_path(cls):
+    if os.getenv('GIT_COOKIES_PATH'):
+      return os.getenv('GIT_COOKIES_PATH')
     return os.path.join(os.environ['HOME'], '.gitcookies')
 
   @classmethod
@@ -162,7 +181,7 @@ class CookiesAuthenticator(Authenticator):
             login, secret_token = value.split('=', 1)
             gitcookies[domain] = (login, secret_token)
         except (IndexError, ValueError, TypeError) as exc:
-          logging.warning(exc)
+          LOGGER.warning(exc)
 
     return gitcookies
 
@@ -179,6 +198,7 @@ class CookiesAuthenticator(Authenticator):
       return 'Basic %s' % (base64.b64encode('%s:%s' % (auth[0], auth[2])))
     return None
 
+
 # Backwards compatibility just in case somebody imports this outside of
 # depot_tools.
 NetrcAuthenticator = CookiesAuthenticator
@@ -199,6 +219,8 @@ class GceAuthenticator(Authenticator):
 
   @classmethod
   def is_gce(cls):
+    if os.getenv('SKIP_GCE_AUTH_FOR_GIT'):
+      return False
     if cls._cache_is_gce is None:
       cls._cache_is_gce = cls._test_is_gce()
     return cls._cache_is_gce
@@ -232,7 +254,6 @@ class GceAuthenticator(Authenticator):
       if resp.status < httplib.INTERNAL_SERVER_ERROR:
         return resp
 
-
   @classmethod
   def _get_token_dict(cls):
     if cls._token_cache:
@@ -254,7 +275,6 @@ class GceAuthenticator(Authenticator):
     return '%(token_type)s %(access_token)s' % token_dict
 
 
-
 def CreateHttpConn(host, path, reqtype='GET', headers=None, body=None):
   """Opens an https connection to a gerrit service, and sends a request."""
   headers = headers or {}
@@ -266,10 +286,11 @@ def CreateHttpConn(host, path, reqtype='GET', headers=None, body=None):
   else:
     LOGGER.debug('No authorization found for %s.' % bare_host)
 
-  if 'Authorization' in headers and not path.startswith('a/'):
-    url = '/a/%s' % path
-  else:
-    url = '/%s' % path
+  url = path
+  if not url.startswith('/'):
+    url = '/' + url
+  if 'Authorization' in headers and not url.startswith('/a/'):
+    url = '/a%s' % url
 
   if body:
     body = json.JSONEncoder().encode(body)
@@ -322,17 +343,18 @@ def ReadHttpResponse(conn, expect_status=200, ignore_404=True):
 
     # If response.status < 500 then the result is final; break retry loop.
     if response.status < 500:
+      LOGGER.debug('got response %d for %s %s', response.status,
+                   conn.req_params['method'], conn.req_params['url'])
       break
     # A status >=500 is assumed to be a possible transient error; retry.
     http_version = 'HTTP/%s' % ('1.1' if response.version == 11 else '1.0')
-    msg = (
-        'A transient error occurred while querying %s:\n'
-        '%s %s %s\n'
-        '%s %d %s' % (
-            conn.host, conn.req_params['method'], conn.req_params['url'],
-            http_version, http_version, response.status, response.reason))
+    LOGGER.warn('A transient error occurred while querying %s:\n'
+                '%s %s %s\n'
+                '%s %d %s',
+                conn.host, conn.req_params['method'], conn.req_params['url'],
+                http_version, http_version, response.status, response.reason)
     if TRY_LIMIT - idx > 1:
-      msg += '\n... will retry %d more times.' % (TRY_LIMIT - idx - 1)
+      LOGGER.warn('... will retry %d more times.', TRY_LIMIT - idx - 1)
       time.sleep(sleep_time)
       sleep_time = sleep_time * 2
       req_host = conn.req_host
@@ -341,7 +363,6 @@ def ReadHttpResponse(conn, expect_status=200, ignore_404=True):
       conn.req_host = req_host
       conn.req_params = req_params
       conn.request(**req_params)
-    LOGGER.warn(msg)
   if ignore_404 and response.status == 404:
     return StringIO()
   if response.status != expect_status:
@@ -475,27 +496,19 @@ def GetChange(host, change):
   return ReadHttpJsonResponse(CreateHttpConn(host, path))
 
 
-def GetChangeDetail(host, change, o_params=None):
+def GetChangeDetail(host, change, o_params=None, ignore_404=True):
   """Query a gerrit server for extended information about a single change."""
+  # TODO(tandrii): cahnge ignore_404 to False by default.
   path = 'changes/%s/detail' % change
   if o_params:
     path += '?%s' % '&'.join(['o=%s' % p for p in o_params])
-  return ReadHttpJsonResponse(CreateHttpConn(host, path))
+  return ReadHttpJsonResponse(CreateHttpConn(host, path), ignore_404=ignore_404)
 
 
-def GetChangeDescriptionFromGitiles(url, revision):
-  """Query Gitiles for actual commit message for a given url and ref.
-
-  url must be obtained from call to GetChangeDetail for a specific
-  revision (patchset) under 'fetch' key.
-  """
-  parsed = urlparse.urlparse(url)
-  path = '%s/+/%s?format=json' % (parsed.path, revision)
-  # Note: Gerrit instances that Chrome infrastructure uses thus far have all
-  # enabled Gitiles, which allowes us to execute this call. This isn't true for
-  # all Gerrit instances out there. Thus, if line below fails, consider adding a
-  # fallback onto actually fetching ref from remote using pure git.
-  return ReadHttpJsonResponse(CreateHttpConn(parsed.netloc, path))['message']
+def GetChangeCommit(host, change, revision='current'):
+  """Query a gerrit server for a revision associated with a change."""
+  path = 'changes/%s/revisions/%s/commit?links' % (change, revision)
+  return ReadHttpJsonResponse(CreateHttpConn(host, path))
 
 
 def GetChangeCurrentRevision(host, change):
@@ -545,36 +558,72 @@ def SubmitChange(host, change, wait_for_merge=True):
   return ReadHttpJsonResponse(conn, ignore_404=False)
 
 
-def SetCommitMessage(host, change, description):
-  """Updates a commit message."""
-  # First, edit the commit message in a draft.
-  path = 'changes/%s/edit:message' % change
-  body = {'message': description}
-  conn = CreateHttpConn(host, path, reqtype='PUT', body=body)
+def HasPendingChangeEdit(host, change):
+  conn = CreateHttpConn(host, 'changes/%s/edit' % change)
   try:
     ReadHttpResponse(conn, ignore_404=False)
   except GerritError as e:
     # On success, gerrit returns status 204; anything else is an error.
     if e.http_status != 204:
       raise
+    return False
   else:
-    raise GerritError(
-        'Unexpectedly received a 200 http status while editing message in '
-        'change %s' % change)
+    return True
+
 
-  # And then publish it.
-  path = 'changes/%s/edit:publish' % change
-  conn = CreateHttpConn(host, path, reqtype='POST', body={})
+def DeletePendingChangeEdit(host, change):
+  conn = CreateHttpConn(host, 'changes/%s/edit' % change, reqtype='DELETE')
   try:
     ReadHttpResponse(conn, ignore_404=False)
   except GerritError as e:
-    # On success, gerrit returns status 204; anything else is an error.
-    if e.http_status != 204:
+    # On success, gerrit returns status 204; if the edit was already deleted it
+    # returns 404.  Anything else is an error.
+    if e.http_status not in (204, 404):
       raise
-  else:
-    raise GerritError(
-        'Unexpectedly received a 200 http status while publishing message '
-        'change in %s' % change)
+
+
+def SetCommitMessage(host, change, description, notify='ALL'):
+  """Updates a commit message."""
+  try:
+    assert notify in ('ALL', 'NONE')
+    # First, edit the commit message in a draft.
+    path = 'changes/%s/edit:message' % change
+    body = {'message': description}
+    conn = CreateHttpConn(host, path, reqtype='PUT', body=body)
+    try:
+      ReadHttpResponse(conn, ignore_404=False)
+    except GerritError as e:
+      # On success, gerrit returns status 204; anything else is an error.
+      if e.http_status != 204:
+        raise
+    else:
+      raise GerritError(
+          'Unexpectedly received a 200 http status while editing message in '
+          'change %s' % change)
+
+    # And then publish it.
+    path = 'changes/%s/edit:publish' % change
+    conn = CreateHttpConn(host, path, reqtype='POST', body={'notify': notify})
+    try:
+      ReadHttpResponse(conn, ignore_404=False)
+    except GerritError as e:
+      # On success, gerrit returns status 204; anything else is an error.
+      if e.http_status != 204:
+        raise
+    else:
+      raise GerritError(
+          'Unexpectedly received a 200 http status while publishing message '
+          'change in %s' % change)
+  except (GerritError, KeyboardInterrupt) as e:
+    # Something went wrong with one of the two calls, so we want to clean up
+    # after ourselves before returning.
+    try:
+      DeletePendingChangeEdit(host, change)
+    except GerritError:
+      LOGGER.error('Encountered error while cleaning up after failed attempt '
+                   'to set the CL description. You may have to delete the '
+                   'pending change edit yourself in the web UI.')
+    raise e
 
 
 def GetReviewers(host, change):
@@ -589,18 +638,32 @@ def GetReview(host, change, revision):
   return ReadHttpJsonResponse(CreateHttpConn(host, path))
 
 
-def AddReviewers(host, change, add=None):
+def AddReviewers(host, change, add=None, is_reviewer=True, notify=True):
   """Add reviewers to a change."""
+  errors = None
   if not add:
-    return
+    return None
   if isinstance(add, basestring):
     add = (add,)
   path = 'changes/%s/reviewers' % change
   for r in add:
-    body = {'reviewer': r}
-    conn = CreateHttpConn(host, path, reqtype='POST', body=body)
-    jmsg = ReadHttpJsonResponse(conn, ignore_404=False)
-  return jmsg
+    state = 'REVIEWER' if is_reviewer else 'CC'
+    notify = 'ALL' if notify else 'NONE'
+    body = {
+      'reviewer': r,
+      'state': state,
+      'notify': notify,
+    }
+    try:
+      conn = CreateHttpConn(host, path, reqtype='POST', body=body)
+      _ = ReadHttpJsonResponse(conn, ignore_404=False)
+    except GerritError as e:
+      if e.http_status == 422:  # "Unprocessable Entity"
+        LOGGER.warn('Failed to add "%s" as a %s' % (r, state.lower()))
+        errors = True
+      else:
+        raise
+  return errors
 
 
 def RemoveReviewers(host, change, remove=None):
@@ -689,3 +752,47 @@ def ResetReviewLabels(host, change, label, value='0', message=None,
   elif jmsg[0]['current_revision'] != revision:
     raise GerritError(200, 'While resetting labels on change "%s", '
                    'a new patchset was uploaded.' % change)
+
+
+def CreateGerritBranch(host, project, branch, commit):
+  """
+  Create a new branch from given project and commit
+  https://gerrit-review.googlesource.com/Documentation/rest-api-projects.html#create-branch
+
+  Returns:
+    A JSON with 'ref' key
+  """
+  path = 'projects/%s/branches/%s' % (project, branch)
+  body = {'revision': commit}
+  conn = CreateHttpConn(host, path, reqtype='PUT', body=body)
+  response = ReadHttpJsonResponse(conn)
+  if response:
+    return response
+  raise GerritError(200, 'Unable to create gerrit branch')
+
+
+def GetGerritBranch(host, project, branch):
+  """
+  Get a branch from given project and commit
+  https://gerrit-review.googlesource.com/Documentation/rest-api-projects.html#get-branch
+
+  Returns:
+    A JSON object with 'revision' key
+  """
+  path = 'projects/%s/branches/%s' % (project, branch)
+  conn = CreateHttpConn(host, path, reqtype='GET')
+  response = ReadHttpJsonResponse(conn)
+  if response:
+    return response
+  raise GerritError(200, 'Unable to get gerrit branch')
+
+
+@contextlib.contextmanager
+def tempdir():
+  tdir = None
+  try:
+    tdir = tempfile.mkdtemp(suffix='gerrit_util')
+    yield tdir
+  finally:
+    if tdir:
+      gclient_utils.rmtree(tdir)