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

Unified Diff: masters/master.chromium.webkit/master_linux_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_latest_cfg.py
===================================================================
--- masters/master.chromium.webkit/master_linux_latest_cfg.py (revision 172146)
+++ masters/master.chromium.webkit/master_linux_latest_cfg.py (working copy)
@@ -34,81 +34,108 @@
B('Linux Tests', 'f_linux_tests_rel', scheduler='s9_webkit_rel')
F('f_linux_tests_rel', linux().ChromiumWebkitLatestFactory(
tests=[
- 'browser_tests',
- 'content_browsertests',
- 'interactive_ui',
- 'unit',
+ 'browser_tests',
+ 'content_browsertests',
+ 'interactive_ui',
+ 'unit',
],
- options=['--compiler=goma'],
- factory_properties={'generate_gtest_json': True}))
+ options=[
+ '--build-tool=ninja',
+ '--compiler=goma'
+ ],
+ factory_properties={
+ 'generate_gtest_json': True,
+ 'gclient_env': { 'GYP_GENERATORS': 'ninja' },
+ }))
-linux_aura_options = [
- 'aura_builder',
- 'base_unittests',
- 'browser_tests',
- 'cacheinvalidation_unittests',
- 'compositor_unittests',
- 'content_browsertests',
- 'content_unittests',
- 'crypto_unittests',
- 'device_unittests',
- 'googleurl_unittests',
- 'gpu_unittests',
- 'interactive_ui_tests',
- 'ipc_tests',
- 'jingle_unittests',
- 'media_unittests',
- 'net_unittests',
- 'ppapi_unittests',
- 'printing_unittests',
- 'remoting_unittests',
- 'sql_unittests',
- 'ui_unittests',
+linux_aura_build_targets = [
+ 'aura_builder',
+ 'base_unittests',
+ 'browser_tests',
+ 'cacheinvalidation_unittests',
+ 'compositor_unittests',
+ 'content_browsertests',
+ 'content_unittests',
+ 'crypto_unittests',
+ 'device_unittests',
+ 'googleurl_unittests',
+ 'gpu_unittests',
+ 'interactive_ui_tests',
+ 'ipc_tests',
+ 'jingle_unittests',
+ 'media_unittests',
+ 'net_unittests',
+ 'ppapi_unittests',
+ 'printing_unittests',
+ 'remoting_unittests',
+ 'sql_unittests',
+ 'ui_unittests',
]
B('Linux Aura', 'f_linux_aura_rel', scheduler='s9_webkit_rel')
F('f_linux_aura_rel', linux().ChromiumWebkitLatestFactory(
tests=[
- 'aura',
- # This seems to have many failures
- #'content_browsertests',
- 'unit',
+ 'aura',
+ # This seems to have many failures
+ #'content_browsertests',
+ 'unit',
],
- options=['--compiler=goma'] + linux_aura_options,
+ options=[
+ '--build-tool=ninja',
+ '--compiler=goma',
+ '--',
+ ] + linux_aura_build_targets,
factory_properties={
- 'generate_gtest_json': True,
- 'gclient_env': {'GYP_DEFINES': 'use_aura=1'}
+ 'generate_gtest_json': True,
+ 'gclient_env': {'GYP_DEFINES': 'use_aura=1', 'GYP_GENERATORS': 'ninja'}
}))
B('Linux Perf', 'f_linux_perf_rel', scheduler='s9_webkit_rel')
F('f_linux_perf_rel', linux().ChromiumWebkitLatestFactory(
- options=['--compiler=goma', 'chromium_builder_perf'],
+ options=[
+ '--build-tool=ninja',
+ '--compiler=goma',
+ '--',
+ 'chromium_builder_perf'
+ ],
tests=[
- 'dom_perf',
- 'dromaeo',
- 'page_cycler_bloat-http',
- 'page_cycler_database',
- 'page_cycler_dhtml',
- 'page_cycler_indexeddb',
- 'page_cycler_intl1',
- 'page_cycler_intl2',
- 'page_cycler_morejs',
- 'page_cycler_moz',
- 'page_cycler_moz-http',
- 'startup',
- 'sunspider'
+ 'dom_perf',
+ 'dromaeo',
+ 'page_cycler_bloat-http',
+ 'page_cycler_database',
+ 'page_cycler_dhtml',
+ 'page_cycler_indexeddb',
+ 'page_cycler_intl1',
+ 'page_cycler_intl2',
+ 'page_cycler_morejs',
+ 'page_cycler_moz',
+ 'page_cycler_moz-http',
+ 'startup',
+ 'sunspider'
],
- factory_properties={'perf_id': 'chromium-rel-linux-webkit',
- 'show_perf_results': True,}))
+ factory_properties={
+ 'perf_id': 'chromium-rel-linux-webkit',
+ 'show_perf_results': True,
+ 'gclient_env': { 'GYP_GENERATORS': 'ninja' },
+ }))
valgrind_gyp_defines = chromium_factory.ChromiumFactory.MEMORY_TOOLS_GYP_DEFINES
B('Linux Valgrind', 'f_linux_valgrind_rel', scheduler='s9_webkit_rel')
F('f_linux_valgrind_rel', linux().ChromiumWebkitLatestFactory(
- options=['--compiler=goma', 'test_shell', 'test_shell_tests'],
+ options=[
+ '--build-tool=ninja',
+ '--compiler=goma',
+ '--',
+ 'test_shell',
+ 'test_shell_tests'],
tests=['valgrind_test_shell'],
factory_properties={
'needs_valgrind': True,
- 'gclient_env': {'GYP_DEFINES' : valgrind_gyp_defines,}}))
+ 'gclient_env': {
+ 'GYP_DEFINES' : valgrind_gyp_defines,
+ 'GYP_GENERATORS': 'ninja',
+ },
+ }))
def Update(config, active_master, c):

Powered by Google App Engine
This is Rietveld 408576698