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

Unified Diff: apply_issue.py

Issue 1194063003: Comment out applying patchset dependencies in apply_issue.py (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/depot_tools.git@master
Patch Set: Lint Created 5 years, 6 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: apply_issue.py
diff --git a/apply_issue.py b/apply_issue.py
index ba389878fbacb717296276fcbc534713401ecae4..aa696d28483a1f49db2f2ea8b75b9bed8eff959f 100755
--- a/apply_issue.py
+++ b/apply_issue.py
@@ -167,22 +167,27 @@ def main():
print('No patchset specified. Using patchset %d' % options.patchset)
issues_patchsets_to_apply = [(options.issue, options.patchset)]
- depends_on_info = obj.get_depends_on_patchset(options.issue, options.patchset)
- while depends_on_info:
- depends_on_issue = int(depends_on_info['issue'])
- depends_on_patchset = int(depends_on_info['patchset'])
- try:
- depends_on_info = obj.get_depends_on_patchset(depends_on_issue,
- depends_on_patchset)
- issues_patchsets_to_apply.insert(0, (depends_on_issue,
- depends_on_patchset))
- except urllib2.HTTPError:
- print ('The patchset that was marked as a dependency no longer '
- 'exists: %s/%d/#ps%d' % (
- options.server, depends_on_issue, depends_on_patchset))
- print 'Therefore it is likely that this patch will not apply cleanly.'
- print
- depends_on_info = None
+
+ # Temporary comment out the below while we try to figure out what to do with
+ # patchset dependencies on trybots.
+ # TODO(rmistry): Uncomment the below after a solution is found.
+ # depends_on_info = obj.get_depends_on_patchset(options.issue,
+ # options.patchset)
+ # while depends_on_info:
+ # depends_on_issue = int(depends_on_info['issue'])
+ # depends_on_patchset = int(depends_on_info['patchset'])
+ # try:
+ # depends_on_info = obj.get_depends_on_patchset(depends_on_issue,
+ # depends_on_patchset)
+ # issues_patchsets_to_apply.insert(0, (depends_on_issue,
+ # depends_on_patchset))
+ # except urllib2.HTTPError:
+ # print ('The patchset that was marked as a dependency no longer '
+ # 'exists: %s/%d/#ps%d' % (
+ # options.server, depends_on_issue, depends_on_patchset))
+ # print 'Therefore it is likely that this patch will not apply cleanly.'
+ # print
+ # depends_on_info = None
num_issues_patchsets_to_apply = len(issues_patchsets_to_apply)
if num_issues_patchsets_to_apply > 1:
« 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