Revision: 21147
Author: [email protected]
Date: Mon May 5 15:16:26 2014 UTC
Log: Remove old branch logic from push scripts.
The old temporary branch is not needed anymore. All scripts create a
different branch after creating the temporary branch.
This also fixes logging subprocesses and logs a warning to track down
script failures due to wrong branches.
BUG=
[email protected]
Review URL: https://codereview.chromium.org/266083003
http://code.google.com/p/v8/source/detail?r=21147
Modified:
/branches/bleeding_edge/tools/push-to-trunk/common_includes.py
/branches/bleeding_edge/tools/push-to-trunk/merge_to_branch.py
/branches/bleeding_edge/tools/push-to-trunk/push_to_trunk.py
/branches/bleeding_edge/tools/push-to-trunk/releases.py
/branches/bleeding_edge/tools/push-to-trunk/test_scripts.py
=======================================
--- /branches/bleeding_edge/tools/push-to-trunk/common_includes.py Wed Apr
9 14:30:02 2014 UTC
+++ /branches/bleeding_edge/tools/push-to-trunk/common_includes.py Mon May
5 15:16:26 2014 UTC
@@ -42,7 +42,6 @@
from git_recipes import GitFailedException
PERSISTFILE_BASENAME = "PERSISTFILE_BASENAME"
-TEMP_BRANCH = "TEMP_BRANCH"
BRANCHNAME = "BRANCHNAME"
DOT_GIT_LOCATION = "DOT_GIT_LOCATION"
VERSION_FILE = "VERSION_FILE"
@@ -176,6 +175,7 @@
# TODO(machenbach): Use timeout.
cmd_line = "%s %s %s" % (prefix, cmd, args)
print "Command: %s" % cmd_line
+ sys.stdout.flush()
try:
if pipe:
return subprocess.check_output(cmd_line, shell=True)
@@ -183,6 +183,9 @@
return subprocess.check_call(cmd_line, shell=True)
except subprocess.CalledProcessError:
return None
+ finally:
+ sys.stdout.flush()
+ sys.stderr.flush()
# Wrapper for side effects.
@@ -376,18 +379,11 @@
self.GitSVNFetch()
def PrepareBranch(self):
- # Get ahold of a safe temporary branch and check it out.
- if self["current_branch"] != self._config[TEMP_BRANCH]:
- self.DeleteBranch(self._config[TEMP_BRANCH])
- self.GitCreateBranch(self._config[TEMP_BRANCH])
-
# Delete the branch that will be created later if it exists already.
self.DeleteBranch(self._config[BRANCHNAME])
def CommonCleanup(self):
self.GitCheckout(self["current_branch"])
- if self._config[TEMP_BRANCH] != self["current_branch"]:
- self.GitDeleteBranch(self._config[TEMP_BRANCH])
if self._config[BRANCHNAME] != self["current_branch"]:
self.GitDeleteBranch(self._config[BRANCHNAME])
=======================================
--- /branches/bleeding_edge/tools/push-to-trunk/merge_to_branch.py Thu Apr
3 13:01:22 2014 UTC
+++ /branches/bleeding_edge/tools/push-to-trunk/merge_to_branch.py Mon May
5 15:16:26 2014 UTC
@@ -41,7 +41,6 @@
PERSISTFILE_BASENAME: "/tmp/v8-merge-to-branch-tempfile",
ALREADY_MERGING_SENTINEL_FILE:
"/tmp/v8-merge-to-branch-tempfile-already-merging",
- TEMP_BRANCH: "prepare-merge-temporary-branch-created-by-script",
DOT_GIT_LOCATION: ".git",
VERSION_FILE: "src/version.cc",
TEMPORARY_PATCH_FILE: "/tmp/v8-prepare-merge-tempfile-temporary-patch",
=======================================
--- /branches/bleeding_edge/tools/push-to-trunk/push_to_trunk.py Fri Mar 21
12:15:25 2014 UTC
+++ /branches/bleeding_edge/tools/push-to-trunk/push_to_trunk.py Mon May 5
15:16:26 2014 UTC
@@ -39,7 +39,6 @@
BRANCHNAME: "prepare-push",
TRUNKBRANCH: "trunk-push",
PERSISTFILE_BASENAME: "/tmp/v8-push-to-trunk-tempfile",
- TEMP_BRANCH: "prepare-push-temporary-branch-created-by-script",
DOT_GIT_LOCATION: ".git",
VERSION_FILE: "src/version.cc",
CHANGELOG_FILE: "ChangeLog",
@@ -58,6 +57,11 @@
def RunStep(self):
self.InitialEnvironmentChecks()
self.CommonPrepare()
+
+ if(self["current_branch"] == self.Config(TRUNKBRANCH)
+ or self["current_branch"] == self.Config(BRANCHNAME)):
+ print "Warning: Script started on branch %s" % self["current_branch"]
+
self.PrepareBranch()
self.DeleteBranch(self.Config(TRUNKBRANCH))
=======================================
--- /branches/bleeding_edge/tools/push-to-trunk/releases.py Fri Apr 25
11:33:19 2014 UTC
+++ /branches/bleeding_edge/tools/push-to-trunk/releases.py Mon May 5
15:16:26 2014 UTC
@@ -25,7 +25,6 @@
CONFIG = {
BRANCHNAME: "retrieve-v8-releases",
- TEMP_BRANCH: "unused-branch", # TODO(machenbach): Remove from
infrastructure.
PERSISTFILE_BASENAME: "/tmp/v8-releases-tempfile",
DOT_GIT_LOCATION: ".git",
VERSION_FILE: "src/version.cc",
=======================================
--- /branches/bleeding_edge/tools/push-to-trunk/test_scripts.py Fri Apr 25
11:33:19 2014 UTC
+++ /branches/bleeding_edge/tools/push-to-trunk/test_scripts.py Mon May 5
15:16:26 2014 UTC
@@ -53,7 +53,6 @@
BRANCHNAME: "test-prepare-push",
TRUNKBRANCH: "test-trunk-push",
PERSISTFILE_BASENAME: "/tmp/test-v8-push-to-trunk-tempfile",
- TEMP_BRANCH: "test-prepare-push-temporary-branch-created-by-script",
DOT_GIT_LOCATION: None,
VERSION_FILE: None,
CHANGELOG_FILE: None,
@@ -435,10 +434,8 @@
Git("status -s -uno", ""),
Git("status -s -b -uno", "## some_branch"),
Git("svn fetch", ""),
- Git("branch", " branch1\n* %s" % TEST_CONFIG[TEMP_BRANCH]),
- Git("branch -D %s" % TEST_CONFIG[TEMP_BRANCH], ""),
- Git("checkout -b %s" % TEST_CONFIG[TEMP_BRANCH], ""),
- Git("branch", ""),
+ Git("branch", " branch1\n* %s" % TEST_CONFIG[BRANCHNAME]),
+ Git("branch -D %s" % TEST_CONFIG[BRANCHNAME], ""),
])
self.ExpectReadline([RL("Y")])
self.MakeStep().CommonPrepare()
@@ -450,7 +447,7 @@
Git("status -s -uno", ""),
Git("status -s -b -uno", "## some_branch"),
Git("svn fetch", ""),
- Git("branch", " branch1\n* %s" % TEST_CONFIG[TEMP_BRANCH]),
+ Git("branch", " branch1\n* %s" % TEST_CONFIG[BRANCHNAME]),
])
self.ExpectReadline([RL("n")])
self.MakeStep().CommonPrepare()
@@ -462,8 +459,8 @@
Git("status -s -uno", ""),
Git("status -s -b -uno", "## some_branch"),
Git("svn fetch", ""),
- Git("branch", " branch1\n* %s" % TEST_CONFIG[TEMP_BRANCH]),
- Git("branch -D %s" % TEST_CONFIG[TEMP_BRANCH], None),
+ Git("branch", " branch1\n* %s" % TEST_CONFIG[BRANCHNAME]),
+ Git("branch -D %s" % TEST_CONFIG[BRANCHNAME], None),
])
self.ExpectReadline([RL("Y")])
self.MakeStep().CommonPrepare()
@@ -727,8 +724,6 @@
Git("status -s -b -uno", "## some_branch\n"),
Git("svn fetch", ""),
Git("branch", " branch1\n* branch2\n"),
- Git("checkout -b %s" % TEST_CONFIG[TEMP_BRANCH], ""),
- Git("branch", " branch1\n* branch2\n"),
Git("branch", " branch1\n* branch2\n"),
Git("checkout -b %s svn/bleeding_edge" %
TEST_CONFIG[BRANCHNAME], ""),
Git("svn find-rev r123455", "push_hash\n"),
@@ -761,7 +756,6 @@
Git("svn dcommit 2>&1", "Some output\nCommitted r123456\nSome
output\n"),
Git("svn tag 3.22.5 -m \"Tagging version 3.22.5\"", ""),
Git("checkout -f some_branch", ""),
- Git("branch -D %s" % TEST_CONFIG[TEMP_BRANCH], ""),
Git("branch -D %s" % TEST_CONFIG[BRANCHNAME], ""),
Git("branch -D %s" % TEST_CONFIG[TRUNKBRANCH], ""),
])
@@ -1062,8 +1056,6 @@
Git("status -s -uno", ""),
Git("status -s -b -uno", "## some_branch\n"),
Git("svn fetch", ""),
- Git("branch", " branch1\n* branch2\n"),
- Git("checkout -b %s" % TEST_CONFIG[TEMP_BRANCH], ""),
Git("branch", " branch1\n* branch2\n"),
Git("checkout -b %s svn/trunk" % TEST_CONFIG[BRANCHNAME], ""),
Git("log --format=%H --grep=\"Port r12345\" --reverse
svn/bleeding_edge",
@@ -1126,7 +1118,6 @@
"https://v8.googlecode.com/svn/tags/3.22.5.1 -m "
"\"Tagging version 3.22.5.1\""), ""),
Git("checkout -f some_branch", ""),
- Git("branch -D %s" % TEST_CONFIG[TEMP_BRANCH], ""),
Git("branch -D %s" % TEST_CONFIG[BRANCHNAME], ""),
])
@@ -1176,8 +1167,6 @@
Git("status -s -uno", ""),
Git("status -s -b -uno", "## some_branch\n"),
Git("svn fetch", ""),
- Git("branch", " branch1\n* branch2\n"),
- Git("checkout -b %s" % TEST_CONFIG[TEMP_BRANCH], ""),
Git("branch", " branch1\n* branch2\n"),
Git("checkout -b %s" % TEST_CONFIG[BRANCHNAME], ""),
Git("branch -r", " svn/3.21\n svn/3.3\n"),
@@ -1236,7 +1225,6 @@
Git("checkout -f master", ""),
Git("branch -D %s" % TEST_CONFIG[BRANCHNAME], ""),
Git("checkout -f some_branch", ""),
- Git("branch -D %s" % TEST_CONFIG[TEMP_BRANCH], ""),
Git("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/d/optout.
