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

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

Issue 2235773002: chromium: run GN in the checkout directory (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: rebase 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 unified diff | Download patch
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/cronet/example.expected/gn_test.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 508 matching lines...) Expand 10 before | Expand all | Expand 10 after
519 def run_gyp_chromium(self): # pragma: no cover 519 def run_gyp_chromium(self): # pragma: no cover
520 gyp_chromium_path = self.m.path['checkout'].join('build', 'gyp_chromium.py') 520 gyp_chromium_path = self.m.path['checkout'].join('build', 'gyp_chromium.py')
521 env = self.get_env() 521 env = self.get_env()
522 env.update(self.c.gyp_env.as_jsonish()) 522 env.update(self.c.gyp_env.as_jsonish())
523 self.m.python(name='gyp_chromium', script=gyp_chromium_path, env=env); 523 self.m.python(name='gyp_chromium', script=gyp_chromium_path, env=env);
524 524
525 def run_gn(self, use_goma=False, gn_path=None, build_dir=None, **kwargs): 525 def run_gn(self, use_goma=False, gn_path=None, build_dir=None, **kwargs):
526 if not gn_path: 526 if not gn_path:
527 gn_path = self.m.depot_tools.gn_py_path 527 gn_path = self.m.depot_tools.gn_py_path
528 528
529 kwargs.setdefault('cwd', self.m.path['checkout'])
530
529 gn_args = list(self.c.gn_args) 531 gn_args = list(self.c.gn_args)
530 532
531 # TODO(dpranke): Figure out if we should use the '_x64' thing to 533 # TODO(dpranke): Figure out if we should use the '_x64' thing to
532 # consistent w/ GYP, or drop it to be consistent w/ the other platforms. 534 # consistent w/ GYP, or drop it to be consistent w/ the other platforms.
533 build_dir = build_dir or '//out/%s' % self.c.build_config_fs 535 build_dir = build_dir or '//out/%s' % self.c.build_config_fs
534 536
535 if self.c.BUILD_CONFIG == 'Debug': 537 if self.c.BUILD_CONFIG == 'Debug':
536 gn_args.append('is_debug=true') 538 gn_args.append('is_debug=true')
537 if self.c.BUILD_CONFIG == 'Release': 539 if self.c.BUILD_CONFIG == 'Release':
538 gn_args.append('is_debug=false') 540 gn_args.append('is_debug=false')
(...skipping 250 matching lines...) Expand 10 before | Expand all | Expand 10 after
789 })) 791 }))
790 792
791 def get_annotate_by_test_name(self, test_name): 793 def get_annotate_by_test_name(self, test_name):
792 return 'graphing' 794 return 'graphing'
793 795
794 def download_lto_plugin(self): 796 def download_lto_plugin(self):
795 return self.m.python( 797 return self.m.python(
796 name='download LTO plugin', 798 name='download LTO plugin',
797 script=self.m.path['checkout'].join( 799 script=self.m.path['checkout'].join(
798 'build', 'download_gold_plugin.py')) 800 'build', 'download_gold_plugin.py'))
OLDNEW
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/cronet/example.expected/gn_test.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698