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

Unified Diff: masters/master.tryserver.chromium/master.cfg

Issue 10260008: Add all the linux tests to linux_clang but keep them off by default. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: now works Created 8 years, 8 months 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: masters/master.tryserver.chromium/master.cfg
diff --git a/masters/master.tryserver.chromium/master.cfg b/masters/master.tryserver.chromium/master.cfg
index 56f0edfa6dbcce2523a35a8d350c7bbf43b17a22..873be723133d42859c3a9eea742a978017ee2bf8 100644
--- a/masters/master.tryserver.chromium/master.cfg
+++ b/masters/master.tryserver.chromium/master.cfg
@@ -78,13 +78,14 @@ def CreateBuilder(platform, builder_name, target,
tests=None, options=None, mode=None, timeout=2400,
slavebuilddir=None, extra_gyp_defines=None,
gclient_env=None, sharded_tests=None, gyp_generators=None,
- asan=False):
+ asan=False, non_default=None):
"""Generates and register a builder along with its slave(s)."""
if platform not in ('win32', 'win64', 'linux', 'mac',
'android', 'android_test'):
raise Exception(platform + ' is not a known os type')
+
factory_properties = {
- 'non_default': [
+ 'non_default': non_default or [
'check_licenses',
'courgette_unittests',
'googleurl_unittests',
@@ -874,9 +875,11 @@ b_linux_clang = CreateBuilder(
platform='linux',
target='Debug',
options=['--build-tool=make', '--compiler=goma-clang'],
- tests=[],
+ tests=linux_tests,
extra_gyp_defines='clang=1 clang_use_chrome_plugins=1',
- builder_name='linux_clang')
+ builder_name='linux_clang',
+ # Keep all tests off by default.
+ non_default=linux_tests)
nsylvain 2012/04/30 15:39:38 i thought non_default took the display name, and n
b_linux_clang_no_goma = CreateBuilder(
platform='linux',
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698