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

Side by Side Diff: scripts/slave/recipe_modules/chromium/api.py

Issue 2161673002: chromium.compile: set cwd to checkout for TARGET_CROS_BOARD builds (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 5 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 unified diff | Download patch
« no previous file with comments | « no previous file | scripts/slave/recipes/chromium.expected/full_chromium_chromiumos_ChromiumOS_amd64_generic_Compile.json » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 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 import re 5 import re
6 6
7 from recipe_engine import recipe_api 7 from recipe_engine import recipe_api
8 from recipe_engine import util as recipe_util 8 from recipe_engine import util as recipe_util
9 9
10 class TestLauncherFilterFileInputPlaceholder(recipe_util.InputPlaceholder): 10 class TestLauncherFilterFileInputPlaceholder(recipe_util.InputPlaceholder):
(...skipping 206 matching lines...) Expand 10 before | Expand all | Expand 10 after
217 args.append('--ninja-ensure-up-to-date') 217 args.append('--ninja-ensure-up-to-date')
218 if self.c.TARGET_CROS_BOARD: 218 if self.c.TARGET_CROS_BOARD:
219 args += ['--cros-board', self.c.TARGET_CROS_BOARD] 219 args += ['--cros-board', self.c.TARGET_CROS_BOARD]
220 220
221 args.append('--') 221 args.append('--')
222 args.extend(targets) 222 args.extend(targets)
223 223
224 if self.c.TARGET_CROS_BOARD: 224 if self.c.TARGET_CROS_BOARD:
225 # Wrap 'compile' through 'cros chrome-sdk' 225 # Wrap 'compile' through 'cros chrome-sdk'
226 kwargs['wrapper'] = self.get_cros_chrome_sdk_wrapper() 226 kwargs['wrapper'] = self.get_cros_chrome_sdk_wrapper()
227 kwargs.setdefault('cwd', self.m.path['checkout'])
227 228
228 env = self.get_env() 229 env = self.get_env()
229 env.update(kwargs.pop('env', {})) 230 env.update(kwargs.pop('env', {}))
230 231
231 try: 232 try:
232 self.m.python( 233 self.m.python(
233 name or 'compile', 234 name or 'compile',
234 self.package_repo_resource('scripts', 'tools', 'runit.py'), 235 self.package_repo_resource('scripts', 'tools', 'runit.py'),
235 args, 236 args,
236 env=env, 237 env=env,
(...skipping 540 matching lines...) Expand 10 before | Expand all | Expand 10 after
777 })) 778 }))
778 779
779 def get_annotate_by_test_name(self, test_name): 780 def get_annotate_by_test_name(self, test_name):
780 return 'graphing' 781 return 'graphing'
781 782
782 def download_lto_plugin(self): 783 def download_lto_plugin(self):
783 return self.m.python( 784 return self.m.python(
784 name='download LTO plugin', 785 name='download LTO plugin',
785 script=self.m.path['checkout'].join( 786 script=self.m.path['checkout'].join(
786 'build', 'download_gold_plugin.py')) 787 'build', 'download_gold_plugin.py'))
OLDNEW
« no previous file with comments | « no previous file | scripts/slave/recipes/chromium.expected/full_chromium_chromiumos_ChromiumOS_amd64_generic_Compile.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698