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

Unified Diff: scripts/slave/recipe_modules/chromium/config.py

Issue 75163006: WebRTC Android APK trybot recipe (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Addressed comments and refactored 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 side-by-side diff with in-line comments
Download patch
Index: scripts/slave/recipe_modules/chromium/config.py
diff --git a/scripts/slave/recipe_modules/chromium/config.py b/scripts/slave/recipe_modules/chromium/config.py
index 8fdbf2dd5a8dc7a06a4e4d0691136f18c6043ef1..42e64f3f316dc68443a7a17f315d6d947009a9c1 100644
--- a/scripts/slave/recipe_modules/chromium/config.py
+++ b/scripts/slave/recipe_modules/chromium/config.py
@@ -219,3 +219,15 @@ def blink(c):
@config_ctx(includes=['chromium_clang'])
def blink_clang(c):
c.compile_py.default_targets = ['blink_tests']
+
+@config_ctx(includes=['ninja', 'static_library', 'default_compiler', 'goma'])
+def android(c):
+ c.gyp_env.GYP_CROSSCOMPILE = 1
iannucci 2013/11/21 00:33:59 Should we just set this in BASE() any time HOST_PL
kjellander_chromium 2013/11/22 12:49:09 That is an excellent idea. I'm not 100% sure it's
+ gyp_defs = c.gyp_env.GYP_DEFINES
+ gyp_defs['fastbuild'] = 1
+ gyp_defs['OS'] = c.TARGET_PLATFORM
+ gyp_defs['host_os'] = c.HOST_PLATFORM
+ gyp_defs['gcc_version'] = 46
+ gyp_defs['order_text_section'] = Path(
+ '[CHECKOUT]', 'orderfiles', 'orderfile.out')
+ gyp_defs['target_arch'] = c.TARGET_ARCH
iannucci 2013/11/21 00:33:59 I think target_arch is now automatically handled f
kjellander_chromium 2013/11/22 12:49:09 Ah, that's great.

Powered by Google App Engine
This is Rietveld 408576698