Make gyp target_arch setting native to chromium/config.py

scripts/slave/recipe_modules/chromium_android/chromium_config.py

 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 from slave.recipe_config_types import Path
+from slave import recipe_config
 
 from RECIPE_MODULES.chromium import CONFIG_CTX
 
-@CONFIG_CTX(includes=['ninja', 'static_library'])
+@CONFIG_CTX(includes=['ninja', 'static_library'],
+            config_vars={'TARGET_ARCH': 'arm', 'TARGET_BITS': 32})
 def android_defaults(c):
   c.compile_py.default_targets=['All']
   c.gyp_env.GYP_CROSSCOMPILE = 1
   c.gyp_env.GYP_GENERATORS.add('ninja')
   c.gyp_env.GYP_GENERATOR_FLAGS['default_target'] = 'All'
   gyp_defs = c.gyp_env.GYP_DEFINES
   gyp_defs['fastbuild'] = 1
   gyp_defs['OS'] = 'android'
   gyp_defs['host_os'] = 'linux'
   gyp_defs['gcc_version'] = 46
   gyp_defs['order_text_section'] = Path(
     '[CHECKOUT]', 'orderfiles', 'orderfile.out')
-  gyp_defs['target_arch'] = 'arm'
+
+  if c.HOST_PLATFORM != 'linux':
+    raise recipe_config.BadConf('Can only build android on linux.')
+  if c.TARGET_BITS != 32:
+    raise recipe_config.BadConf('Android cannot targit %d bits' % c.TARGET_BITS)

[ghost stip (do not use), 2013/11/08 20:11:18]

target

[iannucci, 2013/11/12 02:16:36]

Lol. Done.

 
 
 @CONFIG_CTX(includes=['android_defaults', 'default_compiler', 'goma'])
 def main_builder(c):

[ghost stip (do not use), 2013/11/08 20:11:18]

maybe rename this to arm_builder?

[iannucci, 2013/11/12 02:16:36]

I'd prefer to leave that to another CL. main_builder is a
chromium-on-androidism.

But I agree, it's not a good name (esp. since we're not namespacing configs
yet).

-  pass
+  if c.TARGET_ARCH != 'arm':
+    raise recipe_config.BadConf(
+      'Cannot target arm with TARGET_ARCH == %s' % c.TARGET_ARCH)
 
 @CONFIG_CTX(includes=['android_defaults', 'clang', 'goma'])
 def clang_builder(c):
   pass
 
 @CONFIG_CTX(includes=['main_builder'])
 def component_builder(c):
   c.gyp_env.GYP_DEFINES['component'] = 'shared_library'
 
-@CONFIG_CTX(includes=['main_builder'])
+@CONFIG_CTX(includes=['android_defaults', 'default_compiler', 'goma'],
+            config_vars={'TARGET_ARCH': 'intel'})
 def x86_builder(c):
-  c.gyp_env.GYP_DEFINES['target_arch'] = 'ia32'
+  if c.TARGET_ARCH != 'intel':
+    raise recipe_config.BadConf(
+      'Cannot target x86 with TARGET_ARCH == %s' % c.TARGET_ARCH)
 
 @CONFIG_CTX(includes=['main_builder'])
 def klp_builder(c):
   gyp_defs = c.gyp_env.GYP_DEFINES
   gyp_defs['android_sdk_version'] = '4.4'
   gyp_defs['android_sdk_root'] = Path(
     '[CHECKOUT]', 'third_party', 'android_tools_internal', 'sdk')
 
 @CONFIG_CTX(includes=['main_builder'])
 def try_builder(c):
   pass
 
-@CONFIG_CTX(includes=['x86_builder', 'try_builder'])

[ghost stip (do not use), 2013/11/08 20:11:18]

won't this mean x86_try_builder won't get apply_issue?

[iannucci, 2013/11/12 02:16:36]

no, the recipes check to see if 'try' is in the builder name (which it still
is). You can also tell from the expectations that the apply_issue still runs.

+@CONFIG_CTX(includes=['x86_builder'])
 def x86_try_builder(c):
   pass
 
 @CONFIG_CTX(includes=['android_defaults'])
 def tests_base(c):
   pass
 
 @CONFIG_CTX(includes=['tests_base'])
 def main_tests(c):
   pass
 
 @CONFIG_CTX(includes=['tests_base'])
 def clang_tests(c):
   pass
 
 @CONFIG_CTX(includes=['tests_base'])
 def enormous_tests(c):
   pass
 
 @CONFIG_CTX(includes=['tests_base'])
 def try_instrumentation_tests(c):
   pass
 
-@CONFIG_CTX(includes=['tests_base'])
+@CONFIG_CTX(includes=['x86_builder'])
 def x86_try_instrumentation_tests(c):
   pass