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

Unified Diff: infra/bots/recipe_modules/flavor/coverage_flavor.py

Issue 2217523002: [recipes] Fixes for Coverage bot (Closed) Base URL: https://skia.googlesource.com/skia.git@merge_buildbot_spec_sub2
Patch Set: Fix compile env Created 4 years, 4 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 | infra/bots/recipe_modules/vars/api.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: infra/bots/recipe_modules/flavor/coverage_flavor.py
diff --git a/infra/bots/recipe_modules/flavor/coverage_flavor.py b/infra/bots/recipe_modules/flavor/coverage_flavor.py
index a3bb5828ce847ddfe5cbf8283fb4a329950663a3..732da7fdd66d5420d9796f8c1a468e8f78cd8afb 100644
--- a/infra/bots/recipe_modules/flavor/coverage_flavor.py
+++ b/infra/bots/recipe_modules/flavor/coverage_flavor.py
@@ -18,10 +18,21 @@ class CoverageFlavorUtils(default_flavor.DefaultFlavorUtils):
compile_target = 'dm'
build_cmd = [self.m.vars.skia_dir.join('tools', 'llvm_coverage_build'),
compile_target]
- self.m.run(self.m.step,
- 'build %s' % compile_target,
- cmd=build_cmd,
- cwd=self.m.path['checkout'])
+ build_env = kwargs.pop('env', {})
+ # We have to use Clang 3.6 because earlier versions do not support the
+ # compile flags we use and 3.7 and 3.8 hit asserts during compilation.
+ build_env['CC'] = '/usr/bin/clang-3.6'
+ build_env['CXX'] = '/usr/bin/clang++-3.6'
+ build_env['GYP_DEFINES'] = (
+ 'skia_arch_type=x86_64 '
+ 'skia_clang_build=1 '
+ 'skia_gpu=0 '
+ 'skia_warnings_as_errors=0')
+ self.m.step('build %s' % compile_target,
+ cmd=build_cmd,
+ cwd=self.m.path['checkout'],
+ env=build_env,
+ **kwargs)
# Slice out the 'key' and 'properties' arguments to be reused.
key = []
@@ -48,7 +59,7 @@ class CoverageFlavorUtils(default_flavor.DefaultFlavorUtils):
self.m.vars.skia_dir.join('tools', 'llvm_coverage_run.py'),
] + cmd + ['--outResultsFile', report_file]
self.m.run(self.m.step, name=name, cmd=args,
- cwd=self.m.path['checkout'], **kwargs)
+ cwd=self.m.path['checkout'], **kwargs)
# Generate nanobench-style JSON output from the coverage report.
nanobench_json = results_dir.join('nanobench_%s.json' % (
« no previous file with comments | « no previous file | infra/bots/recipe_modules/vars/api.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698