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

Unified Diff: scripts/slave/recipe_modules/archive/api.py

Issue 2393293002: Make clusterfuzz_archive include llvm-symbolizer and sancov. (Closed)
Patch Set: address comments 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/archive/example.expected/cf_archiving_component.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipe_modules/archive/api.py
diff --git a/scripts/slave/recipe_modules/archive/api.py b/scripts/slave/recipe_modules/archive/api.py
index b451d1b9169b90d18a0bec72916984174de16e5a..e4870936de9361f8bc49683e130b2db3b798125d 100644
--- a/scripts/slave/recipe_modules/archive/api.py
+++ b/scripts/slave/recipe_modules/archive/api.py
@@ -270,6 +270,17 @@ class ArchiveApi(recipe_api.RecipeApi):
else:
staging_dir = self.m.path.mkdtemp('chrome_staging')
+ llvm_tools_to_copy = ['llvm-symbolizer', 'sancov']
+ llvm_bin_dir = self.m.path['checkout'].join('third_party', 'llvm-build',
+ 'Release+Asserts', 'bin')
+ ext = ''
Nico 2016/10/06 18:54:47 nit: This can be ext = '.exe' if self.m.platfor
Oliver Chang 2016/10/06 19:01:32 Done.
+ if self.m.platform.name == 'win':
+ ext = '.exe'
+
+ for tool in llvm_tools_to_copy:
+ self.m.file.copy('Copy ' + tool,
+ self.m.path.join(llvm_bin_dir, tool + ext), build_dir)
+
# Build the list of files to archive.
zip_file_list = [f for f in self.m.file.listdir('build_dir', build_dir)
if self._cf_should_package_file(f)]
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/archive/example.expected/cf_archiving_component.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698