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

Unified Diff: native_client_sdk/src/build_tools/build_sdk.py

Issue 10271013: [NaCl SDK] Push sdk_tools.tgz to commondatastorage on linux buildbots. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: feedback Created 8 years, 8 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: native_client_sdk/src/build_tools/build_sdk.py
diff --git a/native_client_sdk/src/build_tools/build_sdk.py b/native_client_sdk/src/build_tools/build_sdk.py
index 48ba1e59bc15a5429ab142b739d3e43f06c7e3e5..feec61e9ca78a4144f9276ad10f769a0921592dd 100755
--- a/native_client_sdk/src/build_tools/build_sdk.py
+++ b/native_client_sdk/src/build_tools/build_sdk.py
@@ -682,12 +682,20 @@ def main(args):
cwd=os.path.abspath(dirnode), shell=True)
# Archive on non-trybots.
- if options.archive or '-sdk' in os.environ.get('BUILDBOT_BUILDERNAME', ''):
+ buildername = os.environ.get('BUILDBOT_BUILDERNAME', '')
+ if options.archive or '-sdk' in buildername:
buildbot_common.BuildStep('Archive build')
bucket_path = 'nativeclient-mirror/nacl/nacl_sdk/%s' % \
build_utils.ChromeVersion()
buildbot_common.Archive(tarname, bucket_path, os.path.dirname(tarfile))
+ if not skip_update:
+ # Only push up sdk_tools.tgz on the linux buildbot.
+ if buildername == 'linux-sdk-multi':
+ sdk_tools = os.path.join(OUT_DIR, 'sdk_tools.tgz')
+ buildbot_common.Archive('sdk_tools.tgz', bucket_path, OUT_DIR,
+ step_link=False)
+
# generate "manifest snippet" for this archive.
if not skip_test_updater:
archive = bundle.GetArchive(manifest_util.GetHostOS())
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698