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

Unified Diff: drover.py

Issue 554054: Adding branch to branch merge support for drover. (Closed) Base URL: svn://chrome-svn/chrome/trunk/tools/depot_tools/
Patch Set: Created 10 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: drover.py
===================================================================
--- drover.py (revision 36869)
+++ drover.py (working copy)
@@ -23,6 +23,10 @@
--merge <revision> --branch <branch_num>
Example: %(app)s --merge 12345 --branch 187
+[Merge from branch to branch]
+--merge <revision> --sbranch <branch_num> --branch <branch_num>
+Example: %(app)s --merge 12345 --sbranch 248 --branch 249
+
[Revert from trunk]
--revert <revision>
Example: %(app)s --revert 12345
@@ -429,7 +433,9 @@
file_pattern_ = FILE_PATTERN
if options.revert and options.branch:
- url = BRANCH_URL.replace("$branch", options.branch)
+ url = BRANCH_URL.replace("$branch", options.branch)
+ elif options.merge and options.sbranch:
+ url = BRANCH_URL.replace("$branch", options.sbranch)
else:
url = TRUNK_URL
@@ -520,6 +526,8 @@
help='Revision to merge from trunk to branch')
option_parser.add_option('-b', '--branch',
help='Branch to revert or merge from')
+ option_parser.add_option('-s', '--sbranch',
+ help='Source branch for merge')
option_parser.add_option('-r', '--revert', type="int",
help='Revision to revert')
option_parser.add_option('-w', '--workdir',
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698