Revision: 12635
Author:   [email protected]
Date:     Mon Oct  1 02:05:27 2012
Log: Test runner: Send SVN revision instead of git commit hash in work packet

Review URL: https://codereview.chromium.org/11013007
http://code.google.com/p/v8/source/detail?r=12635

Modified:
 /branches/bleeding_edge/tools/testrunner/network/network_execution.py
 /branches/bleeding_edge/tools/testrunner/server/work_handler.py

=======================================
--- /branches/bleeding_edge/tools/testrunner/network/network_execution.py Mon Sep 24 02:38:46 2012 +++ /branches/bleeding_edge/tools/testrunner/network/network_execution.py Mon Oct 1 02:05:27 2012
@@ -67,7 +67,13 @@
     self.pubkey_fingerprint = None  # Fetched later.
     self.base_rev = subprocess.check_output(
         "cd %s; git log -1 --format=%%H --grep=git-svn-id" % workspace,
-        shell=True)
+        shell=True).strip()
+    self.base_svn_rev = subprocess.check_output(
+        "cd %s; git log -1 %s"          # Get commit description.
+        " | grep -e '^\s*git-svn-id:'"  # Extract "git-svn-id" line.
+ " | awk '{print $2}'" # Extract "repository@revision" part.
+        " | sed -e 's/.*@//'" %         # Strip away "repository@".
+        (workspace, self.base_rev), shell=True).strip()
     self.patch = subprocess.check_output(
         "cd %s; git diff %s" % (workspace, self.base_rev), shell=True)
     self.binaries = {}
@@ -168,7 +174,7 @@
         peer.runtime = None
         start_time = time.time()
         packet = workpacket.WorkPacket(peer=peer, context=self.context,
-                                       base_revision=self.base_rev,
+                                       base_revision=self.base_svn_rev,
                                        patch=self.patch,
                                        pubkey=self.pubkey_fingerprint)
         data, test_map = packet.Pack(self.binaries)
=======================================
--- /branches/bleeding_edge/tools/testrunner/server/work_handler.py Fri Sep 28 08:11:33 2012 +++ /branches/bleeding_edge/tools/testrunner/server/work_handler.py Mon Oct 1 02:05:27 2012
@@ -102,11 +102,22 @@
     os.chmod(target, stat.S_IRWXU)
     return True

-  def _CheckoutRevision(self, base_revision):
-    code = self._Call("git checkout -f %s" % base_revision)
-    if code != 0:
+  def _CheckoutRevision(self, base_svn_revision):
+    get_hash_cmd = (
+        "git log -1 --format=%%H --remotes --grep='^git-svn-id:.*@%s'" %
+        base_svn_revision)
+    try:
+      base_revision = subprocess.check_output(get_hash_cmd, shell=True)
+      if not base_revision: raise ValueError
+    except:
       self._Call("git fetch")
-      code = self._Call("git checkout -f %s" % base_revision)
+      try:
+        base_revision = subprocess.check_output(get_hash_cmd, shell=True)
+        if not base_revision: raise ValueError
+      except:
+        self._SendResponse("Base revision not found.")
+        return False
+    code = self._Call("git checkout -f %s" % base_revision)
     if code != 0:
       self._SendResponse("Error trying to check out base revision.")
       return False

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to