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

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

Issue 2302803003: Remove empty {lsan,ubsan}.supp files. (Closed)
Patch Set: Created 4 years, 3 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/recipe_modules/flavor/xsan_flavor.py
diff --git a/infra/bots/recipe_modules/flavor/xsan_flavor.py b/infra/bots/recipe_modules/flavor/xsan_flavor.py
index b4bff4911d4fcb20f1450f802e37307422072e8f..2567b01a0d498812c12aa23ff2d63c4364cdb342 100644
--- a/infra/bots/recipe_modules/flavor/xsan_flavor.py
+++ b/infra/bots/recipe_modules/flavor/xsan_flavor.py
@@ -55,15 +55,11 @@ class XSanFlavorUtils(default_flavor.DefaultFlavorUtils):
def step(self, name, cmd, env=None, **kwargs):
"""Wrapper for the Step API; runs a step as appropriate for this flavor."""
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')
env = dict(env or {})
env['ASAN_OPTIONS'] = 'symbolize=1 detect_leaks=1'
- env['LSAN_OPTIONS'] = ('symbolize=1 print_suppressions=1 suppressions=%s' %
- lsan_suppressions)
+ env['LSAN_OPTIONS'] = 'symbolize=1 print_suppressions=1'
env['TSAN_OPTIONS'] = 'suppressions=%s' % tsan_suppressions
- env['UBSAN_OPTIONS'] = 'suppressions=%s' % ubsan_suppressions
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(

Powered by Google App Engine
This is Rietveld 408576698