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

Unified Diff: rietveld.py

Issue 10963013: Do not enforce HEAD when a revision is not specified. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
Patch Set: Created 8 years, 3 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 | « git_cl.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: rietveld.py
diff --git a/rietveld.py b/rietveld.py
index 94f7c6f524ce3212e02d7496960a242a61af1563..e0bd7a54adc91b68155ab209ca83c7e7159c52e0 100644
--- a/rietveld.py
+++ b/rietveld.py
@@ -315,10 +315,11 @@ class Rietveld(object):
params = [
('reason', reason),
('clobber', 'True' if clobber else 'False'),
- ('revision', revision if revision else 'HEAD'),
('builders', json.dumps(builders_and_tests)),
('xsrf_token', self.xsrf_token()),
]
+ if revision:
+ params.append(('revision', revision))
return self.post('/%d/try/%d' % (issue, patchset), params)
def get_pending_try_jobs(self, cursor=None, limit=100):
« no previous file with comments | « git_cl.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698