[Findit] Add local_git_repository

appengine/findit/lib/gitiles/repo_util.py

 # Copyright 2016 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.
 
+import json
+import logging
 import re
+import subprocess
+import urllib2
 
 CODE_REVIEW_URL_PATTERN = re.compile(
     '^(?:Review URL|Review-Url): (.*\d+).*$', re.IGNORECASE)
 COMMIT_POSITION_PATTERN = re.compile(
     '^Cr-Commit-Position: refs/heads/master@{#(\d+)}$', re.IGNORECASE)
 REVERTED_REVISION_PATTERN = re.compile(
     '^> Committed: https://.+/([0-9a-fA-F]{40})$', re.IGNORECASE)
 START_OF_CR_COMMIT_POSITION = -5
 
 
@@ skipping 46 matching lines @@
   """Parse message to get the reverted revision if there is one."""
   lines = message.strip().splitlines()
   if not lines[0].lower().startswith('revert'):
     return None
 
   for line in reversed(lines):  # pragma: no cover
     # TODO: Handle cases where no reverted_revision in reverting message.
     reverted_revision_match = REVERTED_REVISION_PATTERN.match(line)
     if reverted_revision_match:
       return reverted_revision_match.group(1)
+
+
+# TODO(katesonia): Add local cache for this function.
+def GetCommandOutput(command):
+  """Gets the output stream of executable command.
+
+  Args:
+    command (str): Command to execute to get output.
+
+  Return:
+    Output steam of the command.
+  """
+  p = subprocess.Popen(
+      command, stdout=subprocess.PIPE, stderr=subprocess.PIPE, shell=True)
+  stdoutdata, stderrdata = p.communicate()
+
+  if p.returncode != 0:
+    logging.error('Error running command %s: %s', command, stderrdata)
+    return None

[wrengr, 2016/11/01 20:26:53]

You probably want to throw an exception packaging up the ``p.returncode``,
``command``, and ``stderrdata``; that way callers can decide whether to eat the
error and return None, vs do something else instead (retry, recover some other
way, crash the program).

[Sharu Jiang, 2016/11/05 01:18:15]

Crashing the program working for local testing, but if this is in the production
code, I think we'd better use logging.error.

Let's decide the location of those local_* files first.

+
+  return stdoutdata