2f2419b936
This reverts commit e1cb4778c7
.
Reason for revert: Need to revert dependency https://skia-review.googlesource.com/c/skia/+/114093
Original change's description:
> Switch Win Exceptions Build to Clang.
>
> No-Try: true
> Change-Id: I119c7853299b6eef65f991b41cbb2afb0a35f431
> Reviewed-on: https://skia-review.googlesource.com/113673
> Commit-Queue: Ben Wagner <benjaminwagner@google.com>
> Reviewed-by: Mike Klein <mtklein@google.com>
TBR=mtklein@google.com,benjaminwagner@google.com
Change-Id: I36f4c6cdf890043d4b77b8d5347b4752f58ebd33
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://skia-review.googlesource.com/114420
Reviewed-by: Ben Wagner <benjaminwagner@google.com>
Commit-Queue: Ben Wagner <benjaminwagner@google.com>
219 lines
6.8 KiB
Python
219 lines
6.8 KiB
Python
# Copyright 2016 The Chromium Authors. All rights reserved.
|
|
# Use of this source code is governed by a BSD-style license that can be
|
|
# found in the LICENSE file.
|
|
|
|
|
|
# Recipe module for Skia Swarming compile.
|
|
|
|
|
|
DEPS = [
|
|
'core',
|
|
'recipe_engine/context',
|
|
'recipe_engine/json',
|
|
'recipe_engine/path',
|
|
'recipe_engine/platform',
|
|
'recipe_engine/properties',
|
|
'recipe_engine/python',
|
|
'recipe_engine/step',
|
|
'flavor',
|
|
'run',
|
|
'vars',
|
|
]
|
|
|
|
|
|
def build_targets_from_builder_dict(builder_dict):
|
|
"""Return a list of targets to build, depending on the builder type."""
|
|
return ['most']
|
|
|
|
|
|
def get_extra_env_vars(vars_api):
|
|
env = {}
|
|
if vars_api.builder_cfg.get('compiler') == 'Clang':
|
|
env['CC'] = '/usr/bin/clang'
|
|
env['CXX'] = '/usr/bin/clang++'
|
|
|
|
# SKNX_NO_SIMD, SK_USE_DISCARDABLE_SCALEDIMAGECACHE, etc.
|
|
# TODO(benjaminwagner): Same appears in gn_flavor.py to set extra_cflags. Are
|
|
# both needed?
|
|
if (len(vars_api.extra_tokens) == 1 and
|
|
vars_api.extra_tokens[0].startswith('SK')):
|
|
env['CPPFLAGS'] = '-D' + vars_api.extra_tokens[0]
|
|
|
|
return env
|
|
|
|
|
|
def RunSteps(api):
|
|
bot_update=True
|
|
if 'NoDEPS' in api.properties['buildername']:
|
|
bot_update = False
|
|
api.core.setup(bot_update=bot_update)
|
|
|
|
env = get_extra_env_vars(api.vars)
|
|
build_targets = build_targets_from_builder_dict(api.vars.builder_cfg)
|
|
|
|
try:
|
|
for target in build_targets:
|
|
with api.context(env=env):
|
|
api.flavor.compile(target)
|
|
api.run.copy_build_products(
|
|
api.flavor.out_dir,
|
|
api.vars.swarming_out_dir.join(
|
|
'out', api.vars.configuration))
|
|
api.flavor.copy_extra_build_products(api.vars.swarming_out_dir)
|
|
finally:
|
|
if 'Win' in api.vars.builder_cfg.get('os', ''):
|
|
api.python.inline(
|
|
name='cleanup',
|
|
program='''import psutil
|
|
for p in psutil.process_iter():
|
|
try:
|
|
if p.name in ('mspdbsrv.exe', 'vctip.exe', 'cl.exe', 'link.exe'):
|
|
p.kill()
|
|
except psutil._error.AccessDenied:
|
|
pass
|
|
''',
|
|
infra_step=True)
|
|
|
|
api.flavor.cleanup_steps()
|
|
api.run.check_failure()
|
|
|
|
|
|
TEST_BUILDERS = [
|
|
'Build-Debian9-Clang-arm-Release-Chromebook_GLES',
|
|
'Build-Debian9-Clang-arm64-Release-Android',
|
|
'Build-Debian9-Clang-arm64-Release-Android_Vulkan',
|
|
'Build-Debian9-Clang-arm64-Release-Android_ASAN',
|
|
'Build-Debian9-Clang-x86_64-Debug',
|
|
'Build-Debian9-Clang-x86_64-Debug-ASAN',
|
|
'Build-Debian9-Clang-x86_64-Debug-Coverage',
|
|
'Build-Debian9-Clang-x86_64-Debug-MSAN',
|
|
'Build-Debian9-Clang-x86_64-Debug-SK_USE_DISCARDABLE_SCALEDIMAGECACHE',
|
|
'Build-Debian9-Clang-x86_64-Release-Chromebook_GLES',
|
|
'Build-Debian9-Clang-x86_64-Release-Fast',
|
|
'Build-Debian9-Clang-x86_64-Release-Mini',
|
|
'Build-Debian9-Clang-x86_64-Release-NoDEPS',
|
|
'Build-Debian9-Clang-x86_64-Release-Vulkan',
|
|
'Build-Debian9-Clang-x86_64-Release-Vulkan_Coverage',
|
|
'Build-Debian9-EMCC-wasm-Release',
|
|
'Build-Debian9-GCC-arm-Release-Chromecast',
|
|
'Build-Debian9-GCC-x86-Debug',
|
|
'Build-Debian9-GCC-x86_64-Debug-NoGPU',
|
|
'Build-Debian9-GCC-x86_64-Release-ANGLE',
|
|
'Build-Debian9-GCC-x86_64-Release-Flutter_Android',
|
|
'Build-Debian9-GCC-x86_64-Release-PDFium',
|
|
'Build-Debian9-GCC-x86_64-Release-PDFium_SkiaPaths',
|
|
'Build-Debian9-GCC-x86_64-Release-Shared',
|
|
'Build-Mac-Clang-arm64-Debug-Android',
|
|
'Build-Mac-Clang-arm64-Debug-iOS',
|
|
'Build-Mac-Clang-x64-Release-iOS',
|
|
'Build-Mac-Clang-x86_64-Debug-CommandBuffer',
|
|
'Build-Mac-Clang-x86_64-Release',
|
|
'Build-Win-Clang-arm64-Release-Android',
|
|
'Build-Win-Clang-x86_64-Release-Vulkan',
|
|
'Build-Win-MSVC-x86-Debug',
|
|
'Build-Win-MSVC-x86-Debug-ANGLE',
|
|
'Build-Win-MSVC-x86-Debug-Exceptions',
|
|
'Build-Win-MSVC-x86-Release-GDI',
|
|
]
|
|
|
|
|
|
def GenTests(api):
|
|
for builder in TEST_BUILDERS:
|
|
test = (
|
|
api.test(builder) +
|
|
api.properties(buildername=builder,
|
|
repository='https://skia.googlesource.com/skia.git',
|
|
revision='abc123',
|
|
path_config='kitchen',
|
|
swarm_out_dir='[SWARM_OUT_DIR]') +
|
|
api.path.exists(
|
|
api.path['start_dir'].join('tmp', 'uninteresting_hashes.txt')
|
|
)
|
|
)
|
|
if 'Win' in builder:
|
|
test += api.platform('win', 64)
|
|
elif 'Mac' in builder:
|
|
test += api.platform('mac', 64)
|
|
else:
|
|
test += api.platform('linux', 64)
|
|
|
|
yield test
|
|
|
|
|
|
buildername = 'Build-Win-Clang-x86_64-Release-Vulkan'
|
|
yield (
|
|
api.test('trybot') +
|
|
api.properties(buildername=buildername,
|
|
repository='https://skia.googlesource.com/skia.git',
|
|
revision='abc123',
|
|
path_config='kitchen',
|
|
swarm_out_dir='[SWARM_OUT_DIR]') +
|
|
api.path.exists(
|
|
api.path['start_dir'].join('tmp', 'uninteresting_hashes.txt')
|
|
) +
|
|
api.properties(patch_storage='gerrit') +
|
|
api.properties.tryserver(
|
|
buildername=buildername,
|
|
gerrit_project='skia',
|
|
gerrit_url='https://skia-review.googlesource.com/',
|
|
)
|
|
)
|
|
|
|
yield (
|
|
api.test('alternate_repo') +
|
|
api.properties(buildername=buildername,
|
|
repository='https://skia.googlesource.com/other_repo.git',
|
|
revision='abc123',
|
|
path_config='kitchen',
|
|
swarm_out_dir='[SWARM_OUT_DIR]') +
|
|
api.path.exists(
|
|
api.path['start_dir'].join('tmp', 'uninteresting_hashes.txt')
|
|
)
|
|
)
|
|
|
|
buildername = 'Build-Debian9-GCC-x86_64-Release-PDFium'
|
|
yield (
|
|
api.test('pdfium_trybot') +
|
|
api.properties(
|
|
repository='https://skia.googlesource.com/skia.git',
|
|
buildername=buildername,
|
|
path_config='kitchen',
|
|
swarm_out_dir='[SWARM_OUT_DIR]',
|
|
revision='abc123',
|
|
patch_issue=500,
|
|
patch_repo='https://skia.googlesource.com/skia.git',
|
|
patch_set=1,
|
|
patch_storage='gerrit') +
|
|
api.properties.tryserver(
|
|
buildername=buildername,
|
|
gerrit_project='skia',
|
|
gerrit_url='https://skia-review.googlesource.com/',
|
|
) +
|
|
api.path.exists(
|
|
api.path['start_dir'].join('tmp', 'uninteresting_hashes.txt')
|
|
)
|
|
)
|
|
|
|
buildername = 'Build-Debian9-GCC-x86_64-Release-Flutter_Android'
|
|
yield (
|
|
api.test('flutter_trybot') +
|
|
api.properties(
|
|
repository='https://skia.googlesource.com/skia.git',
|
|
buildername=buildername,
|
|
path_config='kitchen',
|
|
swarm_out_dir='[SWARM_OUT_DIR]',
|
|
revision='abc123',
|
|
patch_issue=500,
|
|
patch_repo='https://skia.googlesource.com/skia.git',
|
|
patch_set=1,
|
|
patch_storage='gerrit') +
|
|
api.properties.tryserver(
|
|
buildername=buildername,
|
|
gerrit_project='skia',
|
|
gerrit_url='https://skia-review.googlesource.com/',
|
|
) +
|
|
api.path.exists(
|
|
api.path['start_dir'].join('tmp', 'uninteresting_hashes.txt')
|
|
)
|
|
)
|