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

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

Issue 2198173002: Re-organize Skia recipes (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Fix missing dependency 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 | « infra/bots/recipe_modules/flavor/valgrind_flavor.py ('k') | infra/bots/recipe_modules/run/__init__.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: infra/bots/recipe_modules/flavor/xsan_flavor.py
diff --git a/infra/bots/recipe_modules/skia/xsan_flavor.py b/infra/bots/recipe_modules/flavor/xsan_flavor.py
similarity index 70%
rename from infra/bots/recipe_modules/skia/xsan_flavor.py
rename to infra/bots/recipe_modules/flavor/xsan_flavor.py
index ac4cac9b04c38ec904b681def2dbf0c1e5eeb9d1..89169b77c1baba2c25e619fb357d3e0a5d854ff8 100644
--- a/infra/bots/recipe_modules/skia/xsan_flavor.py
+++ b/infra/bots/recipe_modules/flavor/xsan_flavor.py
@@ -10,8 +10,8 @@ import default_flavor
class XSanFlavorUtils(default_flavor.DefaultFlavorUtils):
- def __init__(self, *args, **kwargs):
- super(XSanFlavorUtils, self).__init__(*args, **kwargs)
+ def __init__(self, m):
+ super(XSanFlavorUtils, self).__init__(m)
self._sanitizer = {
# We'd love to just pass 'address,undefined' and get all the checks, but
# we're not anywhere close to being able to do that. Instead we start
@@ -24,37 +24,37 @@ class XSanFlavorUtils(default_flavor.DefaultFlavorUtils):
# MSAN and TSAN can't run together with ASAN, so they're their own bots.
'MSAN': 'memory',
'TSAN': 'thread',
- }[self._skia_api.builder_cfg['extra_config'].replace('Swarming', '')]
+ }[self.m.vars.builder_cfg['extra_config'].replace('Swarming', '')]
def compile(self, target):
- cmd = [self._skia_api.skia_dir.join('tools', 'xsan_build'),
+ cmd = [self.m.vars.skia_dir.join('tools', 'xsan_build'),
self._sanitizer, target]
- self._skia_api.run(self._skia_api.m.step, 'build %s' % target, cmd=cmd,
- cwd=self._skia_api.skia_dir)
+ self.m.run(self.m.step, 'build %s' % target, cmd=cmd,
+ cwd=self.m.vars.skia_dir)
def copy_extra_build_products(self, swarming_out_dir):
# Include msan_out if MSAN.
- if 'MSAN' in self._skia_api.builder_cfg['extra_config']:
- msan_out = self._skia_api.m.path.join(
+ if 'MSAN' in self.m.vars.builder_cfg['extra_config']:
+ msan_out = self.m.path.join(
'third_party', 'externals', 'llvm', 'msan_out')
- self._skia_api.m.file.copytree(
+ self.m.file.copytree(
'copy msan_out',
- self._skia_api.skia_dir.join(msan_out),
+ self.m.vars.skia_dir.join(msan_out),
swarming_out_dir.join(msan_out),
symlinks=True)
# Include llvm_symbolizer from the Chromium DEPS so that suppressions work
# by symbol name.
# TODO(benjaminwagner): Figure out how to add this to Skia DEPS for
# target_os 'llvm'.
- self._skia_api.m.file.copytree(
+ self.m.file.copytree(
'copy llvm-build',
- self._skia_api.checkout_root.join('src', 'third_party', 'llvm-build'),
+ self.m.vars.checkout_root.join('src', 'third_party', 'llvm-build'),
swarming_out_dir.join('llvm-build'),
symlinks=True)
def step(self, name, cmd, env=None, **kwargs):
"""Wrapper for the Step API; runs a step as appropriate for this flavor."""
- skia_dir = self._skia_api.skia_dir
+ skia_dir = self.m.vars.skia_dir
lsan_suppressions = skia_dir.join('tools', 'lsan.supp')
tsan_suppressions = skia_dir.join('tools', 'tsan.supp')
ubsan_suppressions = skia_dir.join('tools', 'ubsan.supp')
@@ -64,13 +64,12 @@ class XSanFlavorUtils(default_flavor.DefaultFlavorUtils):
lsan_suppressions)
env['TSAN_OPTIONS'] = 'suppressions=%s' % tsan_suppressions
env['UBSAN_OPTIONS'] = 'suppressions=%s' % ubsan_suppressions
- self._skia_api.default_env['PATH'] = '%%(PATH)s:%s' % (
- self._skia_api.slave_dir.join('llvm-build', 'Release+Asserts', 'bin'))
- env['LD_LIBRARY_PATH'] = self._skia_api.slave_dir.join(
+ self.m.vars.default_env['PATH'] = '%%(PATH)s:%s' % (
+ self.m.vars.slave_dir.join('llvm-build', 'Release+Asserts', 'bin'))
+ env['LD_LIBRARY_PATH'] = self.m.vars.slave_dir.join(
'third_party', 'externals', 'llvm', 'msan_out', 'lib')
path_to_app = self.out_dir.join(cmd[0])
new_cmd = [path_to_app]
new_cmd.extend(cmd[1:])
- return self._skia_api.run(self._skia_api.m.step, name, cmd=new_cmd, env=env,
- **kwargs)
+ return self.m.run(self.m.step, name, cmd=new_cmd, env=env, **kwargs)
« no previous file with comments | « infra/bots/recipe_modules/flavor/valgrind_flavor.py ('k') | infra/bots/recipe_modules/run/__init__.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698