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

Unified Diff: masters/master.chromium.webkit/master_linux_gpu_latest_cfg.py

Issue 11516008: Switch the Linux bots on chromium.webkit to use Ninja. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build/
Patch Set: Created 8 years 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: masters/master.chromium.webkit/master_linux_gpu_latest_cfg.py
===================================================================
--- masters/master.chromium.webkit/master_linux_gpu_latest_cfg.py (revision 172130)
+++ masters/master.chromium.webkit/master_linux_gpu_latest_cfg.py (working copy)
@@ -43,11 +43,16 @@
'gpu_tests',
'gpu_throughput',
],
- options=['--compiler=goma', 'chromium_gpu_builder'],
+ options=[
+ '--build-tool=ninja',
+ '--compiler=goma',
+ 'chromium_gpu_builder',
iannucci 2012/12/10 22:08:15 Not to be pedantic, but I think we're trying to co
tony 2012/12/10 22:15:17 Ok, I've added '--' before any build targets.
+ ],
factory_properties={
- 'generate_gtest_json': True,
- 'perf_id': 'gpu-webkit-linux-nvidia',
- 'show_perf_results': True,
+ 'generate_gtest_json': True,
+ 'perf_id': 'gpu-webkit-linux-nvidia',
+ 'show_perf_results': True,
+ 'gclient_env': { 'GYP_GENERATORS': 'ninja' },
}))
################################################################################
@@ -67,8 +72,15 @@
'gl_tests',
'gpu_tests',
],
- options=['--compiler=goma', 'chromium_gpu_debug_builder'],
- factory_properties={'generate_gtest_json': True}))
+ options=[
+ '--build-tool=ninja',
+ '--compiler=goma',
+ 'chromium_gpu_debug_builder'
+ ],
+ factory_properties={
+ 'generate_gtest_json': True,
+ 'gclient_env': { 'GYP_GENERATORS': 'ninja' },
+ }))
def Update(config, active_master, c):

Powered by Google App Engine
This is Rietveld 408576698