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

Side by Side Diff: scripts/slave/recipes/chromium_trybot.py

Issue 946403002: Fix "analyze" on linux_chromium_clobber_rel_ng always compiling everything (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Created 5 years, 10 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 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 from infra.libs.infra_types import freeze 5 from infra.libs.infra_types import freeze
6 6
7 DEPS = [ 7 DEPS = [
8 'bot_update', 8 'bot_update',
9 'chromium', 9 'chromium',
10 'chromium_android', 10 'chromium_android',
(...skipping 426 matching lines...) Expand 10 before | Expand all | Expand 10 after
437 _sanitize_nonalpha(buildername), 437 _sanitize_nonalpha(buildername),
438 analyze) 438 analyze)
439 yield ( 439 yield (
440 api.test(test_name) + 440 api.test(test_name) +
441 api.chromium_tests.platform( 441 api.chromium_tests.platform(
442 bot_config['mastername'], bot_config['buildername']) + 442 bot_config['mastername'], bot_config['buildername']) +
443 (api.empty_test_data() if analyze else suppress_analyze()) + 443 (api.empty_test_data() if analyze else suppress_analyze()) +
444 props(mastername=mastername, buildername=buildername) 444 props(mastername=mastername, buildername=buildername)
445 ) 445 )
446 446
447 # Regression test for http://crbug.com/453471#c16
448 yield (
449 api.test('clobber_analyze') +
450 props(buildername='linux_chromium_clobber_rel_ng') +
451 api.platform.name('linux') +
452 api.override_step_data(
453 'analyze',
454 api.json.output({'status': 'Found dependency', 'targets': [],
455 'build_targets': ['base_unittests', 'net_unittests']}))
456 )
457
447 yield ( 458 yield (
448 api.test('invalid_results') + 459 api.test('invalid_results') +
449 props() + 460 props() +
450 api.platform.name('linux') + 461 api.platform.name('linux') +
451 api.override_step_data('read test spec', api.json.output({ 462 api.override_step_data('read test spec', api.json.output({
452 'Linux Tests': { 463 'Linux Tests': {
453 'gtest_tests': ['base_unittests'], 464 'gtest_tests': ['base_unittests'],
454 }, 465 },
455 })) + 466 })) +
456 suppress_analyze() + 467 suppress_analyze() +
(...skipping 407 matching lines...) Expand 10 before | Expand all | Expand 10 after
864 tryserver=True) 875 tryserver=True)
865 yield step_failure(mastername='tryserver.chromium.linux', 876 yield step_failure(mastername='tryserver.chromium.linux',
866 buildername='android_clang_dbg_recipe', 877 buildername='android_clang_dbg_recipe',
867 steps=['findbugs (with patch)'], 878 steps=['findbugs (with patch)'],
868 tryserver=True) 879 tryserver=True)
869 yield step_failure(mastername='tryserver.chromium.linux', 880 yield step_failure(mastername='tryserver.chromium.linux',
870 buildername='android_clang_dbg_recipe', 881 buildername='android_clang_dbg_recipe',
871 steps=['findbugs (with patch)', 882 steps=['findbugs (with patch)',
872 'findbugs (without patch)'], 883 'findbugs (without patch)'],
873 tryserver=True) 884 tryserver=True)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698