| OLD | NEW |
| 1 # Copyright 2014 The Chromium Authors. All rights reserved. | 1 # Copyright 2014 The Chromium Authors. All rights reserved. |
| 2 # Use of this source code is governed by a BSD-style license that can be | 2 # Use of this source code is governed by a BSD-style license that can be |
| 3 # found in the LICENSE file. | 3 # found in the LICENSE file. |
| 4 | 4 |
| 5 import ast | 5 import ast |
| 6 import contextlib | 6 import contextlib |
| 7 import copy | 7 import copy |
| 8 import itertools | 8 import itertools |
| 9 import json | 9 import json |
| 10 | 10 |
| (...skipping 450 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 461 | 461 |
| 462 if bot_config.get('archive_build') and not self.m.tryserver.is_tryserver: | 462 if bot_config.get('archive_build') and not self.m.tryserver.is_tryserver: |
| 463 self.m.chromium.archive_build( | 463 self.m.chromium.archive_build( |
| 464 'archive_build', | 464 'archive_build', |
| 465 bot_config['gs_bucket'], | 465 bot_config['gs_bucket'], |
| 466 bot_config.get('gs_acl'), | 466 bot_config.get('gs_acl'), |
| 467 mode='dev' | 467 mode='dev' |
| 468 ) | 468 ) |
| 469 | 469 |
| 470 def run_mb_and_compile(self, compile_targets, isolated_targets, name_suffix): | 470 def run_mb_and_compile(self, compile_targets, isolated_targets, name_suffix): |
| 471 # TODO(phajdan.jr): Also enable error on android. |
| 472 if not compile_targets and self.m.chromium.c.TARGET_PLATFORM != 'android': |
| 473 # TODO(phajdan.jr): Mark as infra failure (make python module support it). |
| 474 self.m.python.failing_step( |
| 475 'compile', 'Please add explicit compile targets for the build') |
| 476 |
| 471 if self.m.chromium.c.project_generator.tool == 'mb': | 477 if self.m.chromium.c.project_generator.tool == 'mb': |
| 472 # We don't use the mastername and buildername passed in, because | 478 # We don't use the mastername and buildername passed in, because |
| 473 # those may be the values of the continuous builder the trybot may | 479 # those may be the values of the continuous builder the trybot may |
| 474 # be configured to match; we need to use the actual mastername | 480 # be configured to match; we need to use the actual mastername |
| 475 # and buildername we're running on, because it may be configured | 481 # and buildername we're running on, because it may be configured |
| 476 # with different MB settings. | 482 # with different MB settings. |
| 477 real_mastername = self.m.properties['mastername'] | 483 real_mastername = self.m.properties['mastername'] |
| 478 real_buildername = self.m.properties['buildername'] | 484 real_buildername = self.m.properties['buildername'] |
| 479 self.m.chromium.run_mb(real_mastername, real_buildername, | 485 self.m.chromium.run_mb(real_mastername, real_buildername, |
| 480 isolated_targets=isolated_targets, | 486 isolated_targets=isolated_targets, |
| (...skipping 333 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 814 result_text = 'MB is enabled for this builder at this revision.' | 820 result_text = 'MB is enabled for this builder at this revision.' |
| 815 log_name = 'Builder MB-ready' | 821 log_name = 'Builder MB-ready' |
| 816 self.m.step.active_result.presentation.logs[log_name] = [result_text] | 822 self.m.step.active_result.presentation.logs[log_name] = [result_text] |
| 817 return False | 823 return False |
| 818 except (self.m.step.StepFailure, KeyError): | 824 except (self.m.step.StepFailure, KeyError): |
| 819 result_text = 'MB is not enabled for this builder at this revision.' | 825 result_text = 'MB is not enabled for this builder at this revision.' |
| 820 log_name = 'Builder NOT MB-ready' | 826 log_name = 'Builder NOT MB-ready' |
| 821 self.m.step.active_result.presentation.logs[log_name] = [result_text] | 827 self.m.step.active_result.presentation.logs[log_name] = [result_text] |
| 822 self.m.step.active_result.presentation.status = self.m.step.WARNING | 828 self.m.step.active_result.presentation.status = self.m.step.WARNING |
| 823 return True | 829 return True |
| OLD | NEW |