monashmicroimaging / podocytes

GNU General Public License v3.0
2 stars 3 forks source link

Safari download of Podo executable erroneously adds a .dms extension to file #25

Closed GenevieveBuckley closed 5 years ago

GenevieveBuckley commented 6 years ago

Safari downloads of the Podo executable file from the release page: https://github.com/monashmicroimaging/podocytes/releases end up having a weird ".dms" extension added to the file name (which shouldn't have an extension).

Downloads using Chrome or Firefox are not affected. This is an Apple bug, and has apparently been around for a while (macrumors discussion here).

Suggested fix: I think we should zip both the executable Podo as well as Podo.app before uploading to the release page. The .zip extension should prevent ".dms" being added where it shouldn't, and users can just unzip it after downloading and run the file.

GenevieveBuckley commented 6 years ago

Zipping the executable will probably also preserve the file permissions (whereas downloading a bare file tends to give you rw-r--r-- permissions by default, it seems.