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

Side by Side Diff: scripts/slave/recipes/skia/swarm_compile.py

Issue 1917243002: Revert "build: roll infra_paths changes" (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 8 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 2016 The Chromium Authors. All rights reserved. 1 # Copyright 2016 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 5
6 # Recipe module for Skia Swarming compile. 6 # Recipe module for Skia Swarming compile.
7 7
8 8
9 DEPS = [ 9 DEPS = [
10 'depot_tools/infra_paths',
11 'recipe_engine/json', 10 'recipe_engine/json',
12 'recipe_engine/path', 11 'recipe_engine/path',
13 'recipe_engine/platform', 12 'recipe_engine/platform',
14 'recipe_engine/properties', 13 'recipe_engine/properties',
15 'skia', 14 'skia',
16 ] 15 ]
17 16
18 17
19 TEST_BUILDERS = { 18 TEST_BUILDERS = {
20 'client.skia.compile': { 19 'client.skia.compile': {
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
52 for builder in builders_by_slave: 51 for builder in builders_by_slave:
53 test = ( 52 test = (
54 api.test(builder) + 53 api.test(builder) +
55 api.properties(buildername=builder, 54 api.properties(buildername=builder,
56 mastername=mastername, 55 mastername=mastername,
57 slavename=slavename, 56 slavename=slavename,
58 buildnumber=5, 57 buildnumber=5,
59 revision='abc123', 58 revision='abc123',
60 swarm_out_dir='[SWARM_OUT_DIR]') + 59 swarm_out_dir='[SWARM_OUT_DIR]') +
61 api.path.exists( 60 api.path.exists(
62 api.infra_paths['slave_build'].join('skia'), 61 api.path['slave_build'].join('skia'),
63 api.infra_paths['slave_build'].join('tmp', 'uninteresting_hashes.t xt') 62 api.path['slave_build'].join('tmp', 'uninteresting_hashes.txt')
64 ) 63 )
65 ) 64 )
66 if 'Win' in builder: 65 if 'Win' in builder:
67 test += api.platform('win', 64) 66 test += api.platform('win', 64)
68 67
69 if 'Android' in builder: 68 if 'Android' in builder:
70 ccache = '/usr/bin/ccache' if 'Appurify' in builder else None 69 ccache = '/usr/bin/ccache' if 'Appurify' in builder else None
71 test += api.step_data('has ccache?', 70 test += api.step_data('has ccache?',
72 stdout=api.json.output({'ccache':ccache})) 71 stdout=api.json.output({'ccache':ccache}))
73 if 'Trybot' in builder: 72 if 'Trybot' in builder:
74 test += api.properties(issue=500, 73 test += api.properties(issue=500,
75 patchset=1, 74 patchset=1,
76 rietveld='https://codereview.chromium.org') 75 rietveld='https://codereview.chromium.org')
77 76
78 yield test 77 yield test
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698