Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1900)

Unified Diff: bin/cros_mark_chrome_as_stable.py

Issue 6339018: Revert "Have the ability for the PFQ to both rev Chrome and ..." (Closed) Base URL: ssh://git@gitrw.chromium.org:9222/crosutils.git@master
Patch Set: Put the right revert patch Created 9 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « bin/cbuildbot_config.py ('k') | cros_mark_as_stable.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: bin/cros_mark_chrome_as_stable.py
diff --git a/bin/cros_mark_chrome_as_stable.py b/bin/cros_mark_chrome_as_stable.py
index 2d88d58b628c7db85adcd36ef4b9ec075e609382..0b17218180ecf1971cc472fc0b71421178eb421c 100755
--- a/bin/cros_mark_chrome_as_stable.py
+++ b/bin/cros_mark_chrome_as_stable.py
@@ -345,12 +345,18 @@ def main():
work_branch = cros_mark_as_stable.GitBranch(
cros_mark_as_stable.STABLE_BRANCH_NAME, options.tracking_branch)
work_branch.CreateBranch()
- chrome_version_atom = MarkChromeEBuildAsStable(
- stable_candidate, unstable_ebuild, chrome_rev, version_to_uprev,
- commit_to_use, overlay_dir, sticky_ebuild)
- # Explicit print to communicate to caller.
- if chrome_version_atom:
- print 'CHROME_VERSION_ATOM=%s' % chrome_version_atom
+ try:
+ chrome_version_atom = MarkChromeEBuildAsStable(
+ stable_candidate, unstable_ebuild, chrome_rev, version_to_uprev,
+ commit_to_use, overlay_dir, sticky_ebuild)
+ # Explicit print to communicate to caller.
+ if chrome_version_atom:
+ print 'CHROME_VERSION_ATOM=%s' % chrome_version_atom
+ else:
+ work_branch.Delete()
+ except:
+ work_branch.Delete()
+ raise
if __name__ == '__main__':
« no previous file with comments | « bin/cbuildbot_config.py ('k') | cros_mark_as_stable.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698