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

Unified Diff: tools/dartium/upload_steps.py

Issue 440233005: Added sha256 sums for build outputs (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
Index: tools/dartium/upload_steps.py
diff --git a/tools/dartium/upload_steps.py b/tools/dartium/upload_steps.py
index 154acacdfeb44b80532e01a4bcc663aa9df13e04..941299a8dce2aaa16b5310beee9edb3b82c7549c 100755
--- a/tools/dartium/upload_steps.py
+++ b/tools/dartium/upload_steps.py
@@ -206,9 +206,10 @@ def UploadFile(local_path, remote_path, create_md5sum=False):
# to make sure the *.md5sum file contains the correct name.
assert '/' in remote_path and not remote_path.endswith('/')
mangled_filename = remote_path[remote_path.rfind('/') + 1:]
- local_md5sum = bot_utils.CreateChecksumFile(local_path, mangled_filename)
+ local_md5sum = bot_utils.CreateMD5ChecksumFile(local_path, mangled_filename)
gsutil.upload(local_md5sum, remote_path + '.md5sum', public=True)
-
+ local_sha256 = bot_utils.CreateSha256ChecksumFile(local_path, mangled_filename)
kevmoo 2014/08/06 01:03:57 long line -> we try to keep all lines <= 80 chars
+ gsutil.upload(local_sha256, remote_path + '.sha256', public=True)
def ExecuteCommand(cmd):
"""Execute a command in a subprocess.

Powered by Google App Engine
This is Rietveld 408576698