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

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

Issue 1187863002: Use simpler "analyze" logic for chromium_trybot recipe on non-Windows (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: fixes Created 5 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 unified diff | Download patch
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 408 matching lines...) Expand 10 before | Expand all | Expand 10 after
419 bot_config['buildername'], 419 bot_config['buildername'],
420 master_dict, 420 master_dict,
421 override_bot_type='builder_tester', 421 override_bot_type='builder_tester',
422 override_tests=tests) 422 override_tests=tests)
423 423
424 requires_compile, matching_exes, compile_targets = \ 424 requires_compile, matching_exes, compile_targets = \
425 api.chromium_tests.analyze( 425 api.chromium_tests.analyze(
426 affected_files, 426 affected_files,
427 all_compile_targets(api, tests + tests_including_triggered), 427 all_compile_targets(api, tests + tests_including_triggered),
428 compile_targets, 428 compile_targets,
429 'trybot_analyze_config.json') 429 'trybot_analyze_config.json',
430 legacy_postprocess=False)
430 431
431 if not requires_compile: 432 if not requires_compile:
432 return 433 return
433 434
434 tests = tests_in_compile_targets(api, matching_exes, tests) 435 tests = tests_in_compile_targets(api, matching_exes, tests)
435 tests_including_triggered = tests_in_compile_targets( 436 tests_including_triggered = tests_in_compile_targets(
436 api, matching_exes, tests_including_triggered) 437 api, matching_exes, tests_including_triggered)
437 438
438 api.chromium_tests.compile_specific_targets( 439 api.chromium_tests.compile_specific_targets(
439 bot_config['mastername'], 440 bot_config['mastername'],
(...skipping 549 matching lines...) Expand 10 before | Expand all | Expand 10 after
989 990
990 yield ( 991 yield (
991 api.test('swarming_paths') + 992 api.test('swarming_paths') +
992 api.properties.tryserver( 993 api.properties.tryserver(
993 mastername='tryserver.chromium.linux', 994 mastername='tryserver.chromium.linux',
994 buildername='linux_chromium_rel_ng', 995 buildername='linux_chromium_rel_ng',
995 path_config='swarming', 996 path_config='swarming',
996 ) + 997 ) +
997 api.platform.name('linux') 998 api.platform.name('linux')
998 ) 999 )
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698