getbinaries: make .AppImage files executable

shouldn't be a security issue, since they are always in subdirectories
openSUSE · Feb 27, 2017 · cf656e3 · cf656e3
1 parent df8e7d2
commit cf656e3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/NEWS b/NEWS
@@ -1,6 +1,7 @@
 0.157
   - add unpublish command (requires OBS 2.8)
   - results: show multibuild results by default
+  - getbinaries: make .AppImage files executable
 
 0.156
   - highlight scheduled jobs with dispatch problems (due to constraints)

diff --git a/osc/core.py b/osc/core.py
@@ -4512,6 +4512,8 @@ def get_binary_file(apiurl, prj, repo, arch,
     where = package or '_repository'
     u = makeurl(apiurl, ['build', prj, repo, arch, where, filename])
     download(u, target_filename, progress_obj, target_mtime)
+    if target_filename.endswith('.AppImage'):
+        os.chmod(target_filename, 0o755)
 
 def dgst_from_string(str):
     # Python 2.5 depracates the md5 modules