Backed out changeset 7f3cd5974457 (bug 1283879) for failing "all_kinds.py:68:15 | multiple spaces after keyword (E271)" as requested by dustin. r=backout

This commit is contained in:
Sebastian Hengst 2016-08-08 23:39:50 +02:00
parent 4ab8a736fb
commit a2c2fa9153
2 changed files with 2 additions and 6 deletions

View File

@ -59,14 +59,10 @@ def set_expires_after(config, tests):
@transforms.add
def set_download_symbols(config, tests):
"""In general, we download symbols immediately for debug builds, but only
on demand for everything else. ASAN builds shouldn't download
symbols since they don't product symbol zips see bug 1283879"""
on demand for everything else."""
for test in tests:
if test['test-platform'].split('/')[-1] == 'debug':
test['mozharness']['download-symbols'] = True
elif test['build-platform'] == 'linux64-asan/opt':
if 'download-symbols' in test['mozharness']:
del test['mozharness']['download-symbols']
else:
test['mozharness']['download-symbols'] = 'ondemand'
yield test

View File

@ -145,7 +145,7 @@ test_description_schema = Schema({
# The setting for --download-symbols (if omitted, the option will not
# be passed to mozharness)
Optional('download-symbols'): Any(True, 'ondemand'),
Optional('download-symbols'): Any(True, False, 'ondemand'),
# If set, then MOZ_NODE_PATH=/usr/local/bin/node is included in the
# environment. This is more than just a helpful path setting -- it