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

Unified Diff: infra/bots/recipes/swarm_compile.py

Issue 2198173002: Re-organize Skia recipes (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Rename modules, add README.md 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
Index: infra/bots/recipes/swarm_compile.py
diff --git a/infra/bots/recipes/swarm_compile.py b/infra/bots/recipes/swarm_compile.py
index 50bbcbe0fe971b29cfb89c561e030243a50b4800..1349da41b36d7ad1de4b3edcb11ddf150133f860 100644
--- a/infra/bots/recipes/swarm_compile.py
+++ b/infra/bots/recipes/swarm_compile.py
@@ -11,7 +11,11 @@ DEPS = [
'recipe_engine/path',
'recipe_engine/platform',
'recipe_engine/properties',
+ 'recipe_engine/python',
'skia',
+ 'flavor',
+ 'run',
+ 'vars',
]
@@ -43,9 +47,31 @@ TEST_BUILDERS = {
def RunSteps(api):
api.skia.setup()
- api.skia.compile_steps()
+
+ try:
+ for target in api.vars.build_targets:
+ 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.skia.cleanup_steps()
- api.skia.check_failure()
+ api.run.check_failure()
def GenTests(api):
@@ -59,7 +85,7 @@ def GenTests(api):
slavename=slavename,
buildnumber=5,
revision='abc123',
- path_config='kitchen',
+ #path_config='kitchen',
rmistry 2016/08/03 14:15:16 Commented out?
borenet 2016/08/03 14:30:00 Fixed.
swarm_out_dir='[SWARM_OUT_DIR]') +
api.path.exists(
api.path['slave_build'].join('tmp', 'uninteresting_hashes.txt')

Powered by Google App Engine
This is Rietveld 408576698