| OLD | NEW |
| 1 # Copyright 2016 The Chromium Authors. All rights reserved. | 1 # Copyright 2016 The Chromium Authors. All rights reserved. |
| 2 # Use of this source code is governed by a BSD-style license that can be | 2 # Use of this source code is governed by a BSD-style license that can be |
| 3 # found in the LICENSE file. | 3 # found in the LICENSE file. |
| 4 | 4 |
| 5 | 5 |
| 6 # Recipe which runs the Skia presubmit. | 6 # Recipe which runs the Skia presubmit. |
| 7 | 7 |
| 8 | 8 |
| 9 DEPS = [ | 9 DEPS = [ |
| 10 'build/file', |
| 10 'core', | 11 'core', |
| 11 'depot_tools/depot_tools', | 12 'depot_tools/depot_tools', |
| 12 'recipe_engine/path', | 13 'recipe_engine/path', |
| 13 'recipe_engine/properties', | 14 'recipe_engine/properties', |
| 14 'recipe_engine/step', | 15 'recipe_engine/step', |
| 15 'recipe_engine/uuid', | 16 'recipe_engine/uuid', |
| 16 'vars', | 17 'vars', |
| 17 ] | 18 ] |
| 18 | 19 |
| 19 | 20 |
| 20 def RunSteps(api): | 21 def RunSteps(api): |
| 21 api.vars.setup() | 22 api.vars.setup() |
| 22 api.core.checkout_steps() | 23 api.core.checkout_steps() |
| 23 | 24 |
| 25 # Create a local GOPATH. |
| 26 gopath = api.vars.slave_dir.join('gopath') |
| 27 if not api.path.exists(gopath): |
| 28 api.file.makedirs('gopath', gopath) |
| 29 |
| 24 api.step('git status', | 30 api.step('git status', |
| 25 cmd=['git', 'status'], | 31 cmd=['git', 'status'], |
| 26 cwd=api.vars.skia_dir) | 32 cwd=api.vars.skia_dir) |
| 27 | 33 |
| 28 depot_tools_path = api.depot_tools.package_repo_resource() | 34 depot_tools_path = api.depot_tools.package_repo_resource() |
| 29 script = depot_tools_path.join('presubmit_support.py') | 35 script = depot_tools_path.join('presubmit_support.py') |
| 30 env = {'PATH': api.path.pathsep.join([str(depot_tools_path), '%(PATH)s'])} | 36 env = { |
| 37 'PATH': api.path.pathsep.join([str(depot_tools_path), '%(PATH)s']), |
| 38 'GOPATH': str(gopath), |
| 39 } |
| 31 # TODO(borenet): --upstream=HEAD^ is a hack to force presubmit_support to | 40 # TODO(borenet): --upstream=HEAD^ is a hack to force presubmit_support to |
| 32 # find a diff. Otherwise, it quits early with: | 41 # find a diff. Otherwise, it quits early with: |
| 33 # "Warning, no PRESUBMIT.py found." | 42 # "Warning, no PRESUBMIT.py found." |
| 34 api.step('presubmit', | 43 api.step('presubmit', |
| 35 cmd=[script, '--commit', '--upstream=HEAD^', '-v', '-v'], | 44 cmd=[script, '--commit', '--upstream=HEAD^', '-v', '-v'], |
| 36 cwd=api.vars.skia_dir, | 45 cwd=api.vars.skia_dir, |
| 37 env=env) | 46 env=env) |
| 38 | 47 |
| 39 | 48 |
| 40 def GenTests(api): | 49 def GenTests(api): |
| 41 yield ( | 50 yield ( |
| 42 api.test('presubmit') + | 51 api.test('presubmit') + |
| 43 api.properties(buildername='Housekeeper-PerCommit-Presubmit', | 52 api.properties(buildername='Housekeeper-PerCommit-Presubmit', |
| 44 mastername='client.skia.fyi', | 53 mastername='client.skia.fyi', |
| 45 slavename='dummy-slave', | 54 slavename='dummy-slave', |
| 46 buildnumber=5, | 55 buildnumber=5, |
| 47 revision='abc123', | 56 revision='abc123', |
| 48 path_config='kitchen', | 57 path_config='kitchen', |
| 49 swarm_out_dir='[SWARM_OUT_DIR]') | 58 swarm_out_dir='[SWARM_OUT_DIR]') |
| 50 ) | 59 ) |
| OLD | NEW |