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: |