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

Unified Diff: experimental/benchtools/rebase.py

Issue 292623005: fix bench rebase tool to cope with wider range of bots. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 6 years, 7 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: experimental/benchtools/rebase.py
diff --git a/experimental/benchtools/rebase.py b/experimental/benchtools/rebase.py
index 5eee51630dcc1f7b92ebde8abcffd94e067a9337..3f89791eb461eca0d5b417be01b5a894467c7041 100755
--- a/experimental/benchtools/rebase.py
+++ b/experimental/benchtools/rebase.py
@@ -84,7 +84,7 @@ def download_gs_files(p, h, gs_dir):
os.remove(os.path.join(gs_dir, p, f))
else:
files += 1
- if files == 4:
+ if files:
return True
return False
@@ -220,7 +220,7 @@ def main():
hash_to_use = ''
for h in reversed(hashes):
li = get_gs_filelist(p, h)
- if len(li) != 4: # no or partial data
+ if not len(li): # no data
continue
if download_gs_files(p, h, gs_dir):
print 'Copied %s/%s' % (p, h)
« 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