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

Unified Diff: prebuilt.py

Issue 5501001: Update prebuilt.py to set permissions correctly on created Packages file. (Closed) Base URL: ssh://git@gitrw.chromium.org:9222/crosutils.git@master
Patch Set: Created 10 years, 1 month 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: prebuilt.py
diff --git a/prebuilt.py b/prebuilt.py
index 607397ee8dcbc730a91011f6bd22a80be6af66d9..2bada2ea35517ac6597ce9a46f7ef1b816ab9a3a 100755
--- a/prebuilt.py
+++ b/prebuilt.py
@@ -441,11 +441,12 @@ def UploadPrebuilt(build_path, upload_location, version, binhost_base_url,
ssh_server, remote_path = remote_location.split(':', 1)
d = { 'pkg_index': tmp_packages_file.name,
'pkgs': pkgs,
+ 'remote_packages': '%s/Packages' % remote_location.rstrip('/'),
'remote_path': remote_path,
'remote_location': remote_location,
'ssh_server': ssh_server }
cmds = ['ssh %(ssh_server)s mkdir -p %(remote_path)s' % d,
- 'rsync -av %(pkg_index)s %(remote_location)s/Packages' % d]
+ 'rsync -av --chmod=a+r %(pkg_index)s %(remote_packages)s' % d]
if pkgs:
cmds.append('rsync -Rav %(pkgs)s %(remote_location)s/' % d)
for cmd in cmds:
« 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