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

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

Issue 1420993007: Simplify config for Clang ToT builders. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: remove Windows config delta Created 5 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
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/chromium_tests/chromium_fyi.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 e3ac8a74e67248b5c141a9816586e7227d6ca379..3d86e9011f114ff9c47e19d3f598011f0ec5850e 100644
--- a/scripts/slave/recipe_modules/chromium/config.py
+++ b/scripts/slave/recipe_modules/chromium/config.py
@@ -517,6 +517,16 @@ def gn_component_build(c):
def gn_minimal_symbols(c):
c.gn_args.append('symbol_level=1')
+@config_ctx()
+def clang_tot(c):
+ c.env.LLVM_FORCE_HEAD_REVISION = 'YES'
+ # Plugin flags often need to be changed when using a plugin newer than
+ # the latest Clang package, so disable plugins.
+ # TODO(pcc): Investigate whether this should be consistent between Windows and
+ # non-Windows.
+ if c.TARGET_PLATFORM != 'win':
+ c.gyp_env.GYP_DEFINES['clang_use_chrome_plugins'] = 0
+
#### 'Full' configurations
@config_ctx(includes=['ninja', 'default_compiler'])
def chromium_no_goma(c):
@@ -531,17 +541,17 @@ def chromium(c):
def chromium_win_clang(c):
fastbuild(c, final=False) # final=False so win_clang_asan can override it.
-@config_ctx(includes=['chromium_win_clang'])
+@config_ctx(includes=['chromium_win_clang', 'clang_tot'])
def chromium_win_clang_tot(c):
- c.env.LLVM_FORCE_HEAD_REVISION = 'YES'
+ pass
@config_ctx(includes=['chromium_win_clang', 'official'])
def chromium_win_clang_official(c):
pass
-@config_ctx(includes=['chromium_win_clang', 'official'])
+@config_ctx(includes=['chromium_win_clang', 'official', 'clang_tot'])
def chromium_win_clang_official_tot(c):
- c.env.LLVM_FORCE_HEAD_REVISION = 'YES'
+ pass
@config_ctx(includes=['chromium_win_clang', 'asan', 'static_library'])
def chromium_win_clang_asan(c):
@@ -551,9 +561,9 @@ def chromium_win_clang_asan(c):
c.gyp_env.GYP_DEFINES['enable_ipc_fuzzer'] = 1
c.gyp_env.GYP_DEFINES['v8_enable_verify_heap'] = 1
-@config_ctx(includes=['chromium_win_clang_asan'])
+@config_ctx(includes=['chromium_win_clang_asan', 'clang_tot'])
def chromium_win_clang_asan_tot(c):
- c.env.LLVM_FORCE_HEAD_REVISION = 'YES'
+ pass
@config_ctx(includes=['chromium_win_clang_asan_tot', 'sanitizer_coverage'])
def chromium_win_clang_asan_tot_coverage(c):
@@ -563,15 +573,12 @@ def chromium_win_clang_asan_tot_coverage(c):
def chromium_win_clang_goma(c):
pass
-@config_ctx(includes=['ninja', 'clang']) # No goma.
+@config_ctx(includes=['ninja', 'clang', 'clang_tot']) # No goma.
def clang_tot_linux(c):
- # Use ToT Clang.
- c.env.LLVM_FORCE_HEAD_REVISION = 'YES'
+ pass
-@config_ctx(includes=['ninja', 'clang']) # No goma.
+@config_ctx(includes=['ninja', 'clang', 'clang_tot']) # No goma.
def clang_tot_mac(c):
- # Use ToT Clang and fastbuild.
- c.env.LLVM_FORCE_HEAD_REVISION = 'YES'
c.gyp_env.GYP_DEFINES['fastbuild'] = 1
@config_ctx()
@@ -595,11 +602,10 @@ def clang_tot_mac_asan(c):
# Clear lsan configuration for mac.
del c.gyp_env.GYP_DEFINES['lsan']
-@config_ctx(includes=['android_common', 'ninja', 'clang', 'asan'])
+@config_ctx(includes=['android_common', 'ninja', 'clang', 'asan', 'clang_tot'])
def clang_tot_android_asan(c):
# Like android_clang, minus goma, minus static_libarary, plus asan.
- # Use ToT Clang.
- c.env.LLVM_FORCE_HEAD_REVISION = 'YES'
+ pass
# GYP_DEFINES must not include 'asan' or 'clang', else the tester bot will try
# to compile clang.
@@ -696,10 +702,9 @@ def chromium_ios_ninja(c):
gyp_defs = c.gyp_env.GYP_DEFINES
gyp_defs['clang_xcode'] = 0
-@config_ctx(includes=['chromium_ios_ninja'])
+@config_ctx(includes=['chromium_ios_ninja', 'clang_tot'])
def clang_tot_ios(c):
- # Use ToT Clang.
- c.env.LLVM_FORCE_HEAD_REVISION = 'YES'
+ pass
@config_ctx(includes=['chromium', 'official'])
def chromium_official(c):
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/chromium_tests/chromium_fyi.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698