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

Unified Diff: infra/bots/recipes/swarm_compile.py

Issue 2186553006: GN: use ccache if available and the requested compiler. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: quotes Created 4 years, 5 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
Index: infra/bots/recipes/swarm_compile.py
diff --git a/infra/bots/recipes/swarm_compile.py b/infra/bots/recipes/swarm_compile.py
index e33a0365bfffcec2ea12affcc0520cbce39eacad..50bbcbe0fe971b29cfb89c561e030243a50b4800 100644
--- a/infra/bots/recipes/swarm_compile.py
+++ b/infra/bots/recipes/swarm_compile.py
@@ -27,6 +27,7 @@ TEST_BUILDERS = {
'Build-Ubuntu-GCC-Arm7-Release-Android_Vulkan',
'Build-Ubuntu-GCC-x86-Debug',
'Build-Ubuntu-GCC-x86_64-Debug-MSAN',
+ 'Build-Ubuntu-Clang-x86_64-Debug-GN',
'Build-Ubuntu-GCC-x86_64-Debug-GN',
'Build-Ubuntu-GCC-x86_64-Release-CMake',
'Build-Ubuntu-GCC-x86_64-Release-PDFium',
@@ -70,10 +71,11 @@ def GenTests(api):
test += api.platform('mac', 64)
else:
test += api.platform('linux', 64)
- if 'Android' in builder:
+ if 'Android' in builder or ('GN' in builder and 'Win' not in builder):
ccache = '/usr/bin/ccache'
test += api.step_data('has ccache?',
stdout=api.json.output({'ccache':ccache}))
+ if 'Android' in builder:
test += api.step_data(
'which adb',
retcode=1)

Powered by Google App Engine
This is Rietveld 408576698