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

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

Issue 2418233002: Add IGNORE_DEPS_CHANGES flag and coverage builder. (Closed)
Patch Set: Created 4 years, 2 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/chromium/config.py » ('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 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 ret['ADB_VENDOR_KEYS'] = self.c.env.ADB_VENDOR_KEYS 72 ret['ADB_VENDOR_KEYS'] = self.c.env.ADB_VENDOR_KEYS
73 if self.c.env.LLVM_FORCE_HEAD_REVISION: 73 if self.c.env.LLVM_FORCE_HEAD_REVISION:
74 ret['LLVM_FORCE_HEAD_REVISION'] = self.c.env.LLVM_FORCE_HEAD_REVISION 74 ret['LLVM_FORCE_HEAD_REVISION'] = self.c.env.LLVM_FORCE_HEAD_REVISION
75 if self.c.env.GOMA_STUBBY_PROXY_IP_ADDRESS: 75 if self.c.env.GOMA_STUBBY_PROXY_IP_ADDRESS:
76 ret['GOMA_STUBBY_PROXY_IP_ADDRESS'] = \ 76 ret['GOMA_STUBBY_PROXY_IP_ADDRESS'] = \
77 self.c.env.GOMA_STUBBY_PROXY_IP_ADDRESS 77 self.c.env.GOMA_STUBBY_PROXY_IP_ADDRESS
78 ret['GOMA_SERVICE_ACCOUNT_JSON_FILE'] = \ 78 ret['GOMA_SERVICE_ACCOUNT_JSON_FILE'] = \
79 self.m.goma.service_account_json_path 79 self.m.goma.service_account_json_path
80 if self.c.env.FORCE_MAC_TOOLCHAIN: 80 if self.c.env.FORCE_MAC_TOOLCHAIN:
81 ret['FORCE_MAC_TOOLCHAIN'] = self.c.env.FORCE_MAC_TOOLCHAIN 81 ret['FORCE_MAC_TOOLCHAIN'] = self.c.env.FORCE_MAC_TOOLCHAIN
82 if self.c.env.IGNORE_DEPS_CHANGES:
83 ret['IGNORE_DEPS_CHANGES'] = '1'
Dirk Pranke 2016/10/15 22:40:31 We shouldn't be using environment variables for an
dnj 2016/10/16 16:44:27 Agreed. IMO the thing to do here is to make an act
82 return ret 84 return ret
83 85
84 @property 86 @property
85 def build_properties(self): 87 def build_properties(self):
86 return self._build_properties 88 return self._build_properties
87 89
88 @property 90 @property
89 def output_dir(self): 91 def output_dir(self):
90 """Return the path to the built executable directory.""" 92 """Return the path to the built executable directory."""
91 return self.c.build_dir.join(self.c.build_config_fs) 93 return self.c.build_dir.join(self.c.build_config_fs)
(...skipping 732 matching lines...) Expand 10 before | Expand all | Expand 10 after
824 })) 826 }))
825 827
826 def get_annotate_by_test_name(self, test_name): 828 def get_annotate_by_test_name(self, test_name):
827 return 'graphing' 829 return 'graphing'
828 830
829 def download_lto_plugin(self): 831 def download_lto_plugin(self):
830 return self.m.python( 832 return self.m.python(
831 name='download LTO plugin', 833 name='download LTO plugin',
832 script=self.m.path['checkout'].join( 834 script=self.m.path['checkout'].join(
833 'build', 'download_gold_plugin.py')) 835 'build', 'download_gold_plugin.py'))
OLDNEW
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/chromium/config.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698