Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

zip loses permission, renders unusable #2

Closed
aonez opened this issue Nov 25, 2016 · 7 comments
Closed

zip loses permission, renders unusable #2

aonez opened this issue Nov 25, 2016 · 7 comments
Assignees
Milestone

Comments

@aonez
Copy link
Owner

aonez commented Nov 25, 2016

As stated here:
https://twitter.com/blynxical/status/802136463446077440

Example file here:
https://t.co/UJJFTPSz0Y

PKG installer looses permissions and can't be executed.

@aonez aonez self-assigned this Nov 25, 2016
@aonez aonez changed the title zip looses permission, renders unusable zip loses permission, renders unusable Nov 27, 2016
@aonez aonez added this to the Future milestone Dec 1, 2016
@aonez aonez added the zip label Feb 6, 2017
@aonez
Copy link
Owner Author

aonez commented Mar 14, 2017

Hearthstore installer is also affected: https://eu.battle.net/account/download/?show=bnetapp

@jojoman02
Copy link

jojoman02 commented Apr 19, 2017

StarCraft installer is also affected: https://starcraft.com/en-us/articles/20674424

  • this is an .app file that doesn't have the necessary run permissions
  • manually adding execute permission to appropriate files within .app container (using chmod) fixes the issue
  • file from MacOS' built-in extractor (Archive Utility) work fine

@aonez
Copy link
Owner Author

aonez commented Jun 5, 2017

Usually adding the execute permission to all the binaries solves the issue. As a workaround:

chmod +x /PATH_TO_THE_APP_PKG/Contents/MacOS/*

@eriksneff
Copy link

FWIW - I would consider this bug to be a SHOWSTOPPER. I've probably spent 10 hours troubleshooting this, and now that I've finally figured out it was Keka all along, well, I'm super pissed, and depending on how quickly this gets resolved, may be uninstalling it for good.

A few more zips that I've personally confirmed lose their ability to be executed when Keka unzips them:

  1. The Sonarr app is affected: https://download.sonarr.tv/v2/master/latest/NzbDrone.master.osx.zip
  2. The Radarr app: https://github.com/Radarr/Radarr/releases/download/v0.2.0.778/Radarr.develop.0.2.0.778.osx-app.zip
  3. The Sophos installer (I can't link to it, but I can confirm that the entire thing was broken after being unzipped by Keka, and not broken when using the macOS native archive app).

@aonez
Copy link
Owner Author

aonez commented Jul 3, 2017

@eriksneff sorry about that one. I've also spent some time on this without any luck. I'll spend some time looking at the Radarr packaging process.

@aonez
Copy link
Owner Author

aonez commented Jul 5, 2017

This one is fixed now. Will come in a few days as Keka 1.0.11.

@aonez aonez closed this as completed Jul 5, 2017
@eriksneff
Copy link

Thanks @aonez !

aonez added a commit that referenced this issue Sep 13, 2017
aonez pushed a commit that referenced this issue Apr 10, 2018
aonez pushed a commit that referenced this issue Aug 21, 2018
aonez pushed a commit that referenced this issue Nov 2, 2018
aonez pushed a commit that referenced this issue Nov 23, 2018
aonez added a commit that referenced this issue Oct 14, 2019
aonez pushed a commit that referenced this issue Oct 14, 2019
aonez pushed a commit that referenced this issue Oct 14, 2019
aonez pushed a commit that referenced this issue Oct 14, 2019
aonez pushed a commit that referenced this issue Oct 14, 2019
aonez added a commit that referenced this issue Oct 14, 2019
aonez pushed a commit that referenced this issue Oct 14, 2019
aonez pushed a commit that referenced this issue Oct 14, 2019
aonez pushed a commit that referenced this issue Oct 14, 2019
aonez pushed a commit that referenced this issue Oct 14, 2019
aonez pushed a commit that referenced this issue Feb 29, 2020
aonez pushed a commit that referenced this issue Jul 28, 2020
aonez pushed a commit that referenced this issue Nov 27, 2020
Merge pull request #675 from Marc66/master
aonez pushed a commit that referenced this issue Apr 27, 2021
aonez added a commit that referenced this issue Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants