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

Unified Diff: dart/tools/bots/get_chromium_build.py

Issue 553823002: Revert change r37732. Remove copies of scripts from old location. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/
Patch Set: Created 6 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 | « no previous file | dart/tools/dartium/fetch_reference_build.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dart/tools/bots/get_chromium_build.py
===================================================================
--- dart/tools/bots/get_chromium_build.py (revision 39949)
+++ dart/tools/bots/get_chromium_build.py (working copy)
@@ -14,7 +14,6 @@
$ get_chromium_build.py -r <revision> -t <target>
"""
-import json
import logging
import optparse
import os
@@ -50,7 +49,7 @@
self._zipfiles = platform_data['zipfiles']
self._folder = platform_data['folder']
self._archive_path = platform_data['archive_path']
- self._revision = options.revision
+ self._revision = int(options.revision)
self._target_dir = options.target_dir
self._download_dir = os.path.join(self._target_dir, 'downloads')
@@ -58,9 +57,12 @@
return CHROMIUM_URL_FMT % (self._archive_path, revision, filename)
def _FindBuildRevision(self, revision, filename):
- git_hash = json.loads(revision)[platform.system()]
- if self._DoesBuildExist(git_hash, filename):
- return git_hash
+ MAX_REVISIONS_PER_BUILD = 100
+ for revision_guess in xrange(revision, revision + MAX_REVISIONS_PER_BUILD):
+ if self._DoesBuildExist(revision_guess, filename):
+ return revision_guess
+ else:
+ time.sleep(.1)
return None
def _DoesBuildExist(self, revision_guess, filename):
« no previous file with comments | « no previous file | dart/tools/dartium/fetch_reference_build.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698