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

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: 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
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..4a1b134de66d05279a2e86ea616740b95ebbb8d9 100644
--- a/infra/bots/recipes/swarm_compile.py
+++ b/infra/bots/recipes/swarm_compile.py
@@ -7,11 +7,15 @@
DEPS = [
+ 'core',
'recipe_engine/json',
'recipe_engine/path',
'recipe_engine/platform',
'recipe_engine/properties',
- 'skia',
+ 'recipe_engine/python',
+ 'flavor',
+ 'run',
+ 'vars',
]
@@ -42,10 +46,32 @@ TEST_BUILDERS = {
def RunSteps(api):
- api.skia.setup()
- api.skia.compile_steps()
- api.skia.cleanup_steps()
- api.skia.check_failure()
+ api.core.setup()
+
+ 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.core.cleanup_steps()
+ api.run.check_failure()
def GenTests(api):

Powered by Google App Engine
This is Rietveld 408576698