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

Unified Diff: scripts/slave/recipes/infra/publish_tarball.py

Issue 1210593002: publish_tarball: fetch nacl source dependencies in addition to pnacl (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 5 years, 6 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/recipes/infra/publish_tarball.expected/basic.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipes/infra/publish_tarball.py
diff --git a/scripts/slave/recipes/infra/publish_tarball.py b/scripts/slave/recipes/infra/publish_tarball.py
index f99354a085c9201c852bf1d1b5c7f3c074cd3a26..237c265e3d2eceb3c3767362d931359dbf74e136 100644
--- a/scripts/slave/recipes/infra/publish_tarball.py
+++ b/scripts/slave/recipes/infra/publish_tarball.py
@@ -133,9 +133,15 @@ def export_lite_tarball(api, version):
def export_nacl_tarball(api, version):
# Make destructive file operations on the copy of the checkout.
with copytree_checkout(api) as dest_dir:
+ api.step(
+ 'download nacl toolchain dependencies',
+ ['make',
+ '-C', api.path.join(dest_dir, 'native_client', 'tools'),
+ 'sync-pinned'])
+
# Based on instructions from https://sites.google.com/a/chromium.org/dev/nativeclient/pnacl/building-pnacl-components-for-distribution-packagers
api.python(
- 'download nacl toolchain dependencies',
+ 'download pnacl toolchain dependencies',
api.path.join(dest_dir, 'native_client', 'toolchain_build',
'toolchain_build_pnacl.py'),
['--verbose', '--sync', '--sync-only', '--disable-git-cache'])
« no previous file with comments | « no previous file | scripts/slave/recipes/infra/publish_tarball.expected/basic.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698