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

Unified Diff: tools/rebaseline.py

Issue 16306010: rebaseline.py: --tests and --configs are now FILTERS within json results (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Patch Set: Created 7 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: tools/rebaseline.py
===================================================================
--- tools/rebaseline.py (revision 9460)
+++ tools/rebaseline.py (working copy)
@@ -173,23 +173,12 @@
expectations_subdir + '/' + builder_name + '/' +
expectations_subdir + '/' + infilename)
- # Try to download this file, but if that fails, keep going...
+ # Try to download this file.
#
- # This not treated as a fatal failure because not all
- # platforms generate all configs (e.g., Android does not
- # generate PDF).
- #
- # We could tweak the list of configs within this tool to
- # reflect which combinations the bots actually generate, and
- # then fail if any of those expected combinations are
- # missing... but then this tool would become useless every
- # time someone tweaked the configs on the bots without
- # updating this script.
- try:
- self._DownloadFile(source_url=url, dest_filename=outfilename)
- except CommandFailedException:
- print '# Couldn\'t fetch ' + url
- return
+ # If the download fails, this will raise an exception and halt the
+ # rebaseline process. Since the JSON results summary told us that
+ # this file needed rebaselining, we ought to be able to download it...
+ self._DownloadFile(source_url=url, dest_filename=outfilename)
epoger 2013/06/06 17:53:29 Patchset 1 makes the script fail early if any down
# Add this file to version control (if it isn't already).
if self._is_svn_checkout:
« 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