Revision: 17926
Author: [email protected]
Date: Wed Nov 20 13:53:11 2013 UTC
Log: Force 'git cl upload' in push-to-trunk forced mode.
[email protected]
Review URL: https://codereview.chromium.org/77853003
http://code.google.com/p/v8/source/detail?r=17926
Modified:
/branches/bleeding_edge/tools/push-to-trunk/common_includes.py
/branches/bleeding_edge/tools/push-to-trunk/push_to_trunk.py
/branches/bleeding_edge/tools/push-to-trunk/test_scripts.py
=======================================
--- /branches/bleeding_edge/tools/push-to-trunk/common_includes.py Wed Nov
20 08:49:42 2013 UTC
+++ /branches/bleeding_edge/tools/push-to-trunk/common_includes.py Wed Nov
20 13:53:11 2013 UTC
@@ -370,15 +370,18 @@
Step.__init__(self, "Upload for code review.")
def RunStep(self):
- if self._options and self._options.r:
+ if self._options.r:
print "Using account %s for review." % self._options.r
reviewer = self._options.r
else:
print "Please enter the email address of a V8 reviewer for your
patch: ",
self.DieInForcedMode("A reviewer must be specified in forced mode.")
reviewer = self.ReadLine()
- args = "cl upload -r \"%s\" --send-mail" % reviewer
- if self.Git(args,pipe=False) is None:
+ force_flag = " -f" if self._options.f else ""
+ args = "cl upload -r \"%s\" --send-mail%s" % (reviewer, force_flag)
+ # TODO(machenbach): Check output in forced mode. Verify that all
required
+ # base files were uploaded, if not retry.
+ if self.Git(args, pipe=False) is None:
self.Die("'git cl upload' failed, please try again.")
=======================================
--- /branches/bleeding_edge/tools/push-to-trunk/push_to_trunk.py Wed Nov 20
08:49:42 2013 UTC
+++ /branches/bleeding_edge/tools/push-to-trunk/push_to_trunk.py Wed Nov 20
13:53:11 2013 UTC
@@ -457,7 +457,8 @@
args = "commit -am \"Update V8 to version %s.\n\nTBR=%s\"" % (ver, rev)
if self.Git(args) is None:
self.Die("'git commit' failed.")
- if self.Git("cl upload --send-mail", pipe=False) is None:
+ force_flag = " -f" if self._options.f else ""
+ if self.Git("cl upload --send-mail%s" % force_flag, pipe=False) is
None:
self.Die("'git cl upload' failed, please try again.")
print "CL uploaded."
=======================================
--- /branches/bleeding_edge/tools/push-to-trunk/test_scripts.py Wed Nov 20
08:49:42 2013 UTC
+++ /branches/bleeding_edge/tools/push-to-trunk/test_scripts.py Wed Nov 20
13:53:11 2013 UTC
@@ -461,6 +461,7 @@
self.assertTrue(re.search(r"#define PATCH_LEVEL\s+0", version))
self.assertTrue(re.search(r"#define IS_CANDIDATE_VERSION\s+0",
version))
+ force_flag = " -f" if force else ""
self._git_recipe = [
["status -s -uno", ""],
["status -s -b -uno", "## some_branch\n"],
@@ -481,7 +482,8 @@
"Now working on version 3.22.6.\""),
" 2 files changed\n",
CheckPreparePush],
- ["cl upload -r \"[email protected]\" --send-mail", "done\n"],
+ ["cl upload -r \"[email protected]\" --send-mail%s" % force_flag,
+ "done\n"],
["cl dcommit -f", "Closing issue\n"],
["svn fetch", "fetch result\n"],
["checkout svn/bleeding_edge", ""],
@@ -502,7 +504,7 @@
[("commit -am \"Update V8 to version 3.22.5.\n\n"
"[email protected]\""),
""],
- ["cl upload --send-mail", ""],
+ ["cl upload --send-mail%s" % force_flag, ""],
["checkout -f some_branch", ""],
["branch -D %s" % TEST_CONFIG[TEMP_BRANCH], ""],
["branch -D %s" % TEST_CONFIG[BRANCHNAME], ""],
--
--
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/groups/opt_out.
