Revision: 23678
Author: [email protected]
Date: Thu Sep 4 09:56:29 2014 UTC
Log: Make lookup of last v8 roll work with pure git.
BUG=410721
LOG=n
[email protected]
Review URL: https://codereview.chromium.org/540843002
https://code.google.com/p/v8/source/detail?r=23678
Modified:
/branches/bleeding_edge/tools/push-to-trunk/auto_roll.py
/branches/bleeding_edge/tools/push-to-trunk/test_scripts.py
=======================================
--- /branches/bleeding_edge/tools/push-to-trunk/auto_roll.py Thu Sep 4
08:42:21 2014 UTC
+++ /branches/bleeding_edge/tools/push-to-trunk/auto_roll.py Thu Sep 4
09:56:29 2014 UTC
@@ -19,7 +19,6 @@
CLUSTERFUZZ_API_KEY_FILE: ".cf_api_key",
}
-CR_DEPS_URL = 'http://src.chromium.org/svn/trunk/src/DEPS'
class CheckActiveRoll(Step):
MESSAGE = "Check active roll."
@@ -60,9 +59,11 @@
def RunStep(self):
# Interpret the DEPS file to retrieve the v8 revision.
+ # TODO(machenbach): This should be part or the roll-deps api of
+ # depot_tools.
Var = lambda var: '%s'
- exec(self.ReadURL(CR_DEPS_URL))
- last_roll = vars['v8_revision']
+ exec(FileToText(os.path.join(self._options.chromium, "DEPS")))
+ last_roll = self.GetCommitPositionNumber(vars['v8_revision'])
# FIXME(machenbach): When rolling from bleeding edge and from trunk
there
# be different commit numbers here. Better use version?
if int(last_roll) >= int(self["last_push"]):
=======================================
--- /branches/bleeding_edge/tools/push-to-trunk/test_scripts.py Thu Sep 4
08:42:21 2014 UTC
+++ /branches/bleeding_edge/tools/push-to-trunk/test_scripts.py Thu Sep 4
09:56:29 2014 UTC
@@ -804,6 +804,12 @@
git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@22624 123
+"""
+
+ C_V8_123455_LOG = """V8 CL.
+
+git-svn-id: https://v8.googlecode.com/svn/branches/bleeding_edge@123455 123
+
"""
C_V8_123456_LOG = """V8 CL.
@@ -954,7 +960,7 @@
# Snippet from the original DEPS file.
FAKE_DEPS = """
vars = {
- "v8_revision": "123455",
+ "v8_revision": "abcd123455",
}
deps = {
"src/v8":
@@ -964,6 +970,8 @@
"""
def testAutoRollUpToDate(self):
+ os.makedirs(TEST_CONFIG[CHROMIUM])
+ TextToFile(self.FAKE_DEPS, os.path.join(TEST_CONFIG[CHROMIUM], "DEPS"))
self.Expect([
URL("https://codereview.chromium.org/search";,
"owner=author%40chromium.org&limit=30&closed=3&format=json",
@@ -972,8 +980,7 @@
"\"^Version [[:digit:]]*\.[[:digit:]]*\.[[:digit:]]*\" "
"origin/master"), "push_hash\n"),
Cmd("git log -1 --format=%B push_hash", self.C_V8_22624_LOG),
- URL("http://src.chromium.org/svn/trunk/src/DEPS";,
- self.FAKE_DEPS),
+ Cmd("git log -1 --format=%B abcd123455", self.C_V8_123455_LOG),
])
result = auto_roll.AutoRoll(TEST_CONFIG, self).Run(
@@ -981,6 +988,8 @@
self.assertEquals(1, result)
def testAutoRoll(self):
+ os.makedirs(TEST_CONFIG[CHROMIUM])
+ TextToFile(self.FAKE_DEPS, os.path.join(TEST_CONFIG[CHROMIUM], "DEPS"))
TEST_CONFIG[CLUSTERFUZZ_API_KEY_FILE] = self.MakeEmptyTempFile()
TextToFile("fake key", TEST_CONFIG[CLUSTERFUZZ_API_KEY_FILE])
@@ -992,8 +1001,7 @@
"\"^Version [[:digit:]]*\.[[:digit:]]*\.[[:digit:]]*\" "
"origin/master"), "push_hash\n"),
Cmd("git log -1 --format=%B push_hash", self.C_V8_123456_LOG),
- URL("http://src.chromium.org/svn/trunk/src/DEPS";,
- self.FAKE_DEPS),
+ Cmd("git log -1 --format=%B abcd123455", self.C_V8_123455_LOG),
])
result = auto_roll.AutoRoll(TEST_CONFIG, self).Run(
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
