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

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

Issue 59233015: Make gyp target_arch setting native to chromium/config.py (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: rebase again Created 7 years, 1 month 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.
2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file.
4
1 from slave.recipe_config_types import Path 5 from slave.recipe_config_types import Path
6 from slave import recipe_config
2 from RECIPE_MODULES.chromium import CONFIG_CTX 7 from RECIPE_MODULES.chromium import CONFIG_CTX
3 8
4 9
5 @CONFIG_CTX() 10 @CONFIG_CTX()
6 def v8(c): 11 def v8(c):
7 if c.TARGET_ARCH == 'arm': 12 targ_arch = c.gyp_env.GYP_DEFINES.get('target_arch')
8 v8_target_arch = 'arm' 13 if not targ_arch: # pragma: no cover
9 elif c.TARGET_ARCH == 'mips': 14 raise recipe_config.BadConf('v8 must have a valid target_arch.')
10 v8_target_arch = 'mips' 15 c.gyp_env.GYP_DEFINES['v8_target_arch'] = targ_arch
11 elif c.TARGET_BITS == 64:
12 v8_target_arch = 'x64'
13 else:
14 v8_target_arch = 'ia32'
15 c.gyp_env.GYP_DEFINES['v8_target_arch'] = v8_target_arch
16 del c.gyp_env.GYP_DEFINES['component'] 16 del c.gyp_env.GYP_DEFINES['component']
17 c.build_config_fs = c.BUILD_CONFIG 17 c.build_config_fs = c.BUILD_CONFIG
18 c.build_dir = Path('[CHECKOUT]') 18 c.build_dir = Path('[CHECKOUT]')
19 19
20 c.compile_py.build_tool = 'make' 20 c.compile_py.build_tool = 'make'
21 c.compile_py.default_targets = ['buildbot'] 21 c.compile_py.default_targets = ['buildbot']
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698