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

Side by Side Diff: scripts/slave/recipe_modules/v8/chromium_config.py

Issue 2351963004: V8: Remove xcode-ninja bot (Closed)
Patch Set: Remove bot Created 4 years, 3 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 recipe_engine.config import BadConf 5 from recipe_engine.config import BadConf
6 from recipe_engine.config_types import Path 6 from recipe_engine.config_types import Path
7 from recipe_engine import config as recipe_config 7 from recipe_engine import config as recipe_config
8 8
9 import DEPS 9 import DEPS
10 CONFIG_CTX = DEPS['chromium'].CONFIG_CTX 10 CONFIG_CTX = DEPS['chromium'].CONFIG_CTX
(...skipping 95 matching lines...) Expand 10 before | Expand all | Expand 10 after
106 @CONFIG_CTX(includes=['ninja']) 106 @CONFIG_CTX(includes=['ninja'])
107 def v8_ninja(c): 107 def v8_ninja(c):
108 c.gyp_env.GYP_GENERATORS.add('ninja') 108 c.gyp_env.GYP_GENERATORS.add('ninja')
109 109
110 if c.HOST_PLATFORM == 'win' and c.TARGET_BITS == 64: 110 if c.HOST_PLATFORM == 'win' and c.TARGET_BITS == 64:
111 # Windows requires 64-bit builds to be in <dir>_x64 with ninja. See 111 # Windows requires 64-bit builds to be in <dir>_x64 with ninja. See
112 # crbug.com/470681. 112 # crbug.com/470681.
113 c.build_config_fs = c.BUILD_CONFIG + '_x64' 113 c.build_config_fs = c.BUILD_CONFIG + '_x64'
114 114
115 115
116 @CONFIG_CTX(includes=['ninja'])
117 def v8_xcode_ninja(c):
118 c.gyp_env.GYP_GENERATORS.add('ninja')
119 c.gyp_env.GYP_GENERATORS.add('xcode-ninja')
120
121
122 # Work-around for obtaining the right build dir on linux slave that trigger 116 # Work-around for obtaining the right build dir on linux slave that trigger
123 # windows 64 bit swarming jobs. 117 # windows 64 bit swarming jobs.
124 @CONFIG_CTX(includes=['v8']) 118 @CONFIG_CTX(includes=['v8'])
125 def use_windows_swarming_slaves(c): 119 def use_windows_swarming_slaves(c):
126 if c.TARGET_BITS == 64: 120 if c.TARGET_BITS == 64:
127 c.build_config_fs = c.BUILD_CONFIG + '_x64' 121 c.build_config_fs = c.BUILD_CONFIG + '_x64'
128 122
129 123
130 @CONFIG_CTX(includes=['v8', 'dcheck']) 124 @CONFIG_CTX(includes=['v8', 'dcheck'])
131 def no_dcheck(c): 125 def no_dcheck(c):
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
200 def vtunejit(c): 194 def vtunejit(c):
201 c.gyp_env.GYP_DEFINES['v8_enable_vtunejit'] = 1 195 c.gyp_env.GYP_DEFINES['v8_enable_vtunejit'] = 1
202 196
203 197
204 @CONFIG_CTX(includes=['v8']) 198 @CONFIG_CTX(includes=['v8'])
205 def x87(c): 199 def x87(c):
206 # TODO(machenbach): Chromium does not support x87 yet. With the current 200 # TODO(machenbach): Chromium does not support x87 yet. With the current
207 # configuration, target_arch can't be set through a parameter as ARCH=intel 201 # configuration, target_arch can't be set through a parameter as ARCH=intel
208 # and BITS=32 is ambigue with x87. 202 # and BITS=32 is ambigue with x87.
209 c.gyp_env.GYP_DEFINES['v8_target_arch'] = 'x87' 203 c.gyp_env.GYP_DEFINES['v8_target_arch'] = 'x87'
OLDNEW
« no previous file with comments | « scripts/slave/recipe_modules/v8/builders.py ('k') | scripts/slave/recipes/v8.expected/full_client_v8_V8_Mac64___xcode.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698