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

The executable bit is lost for scripts in data directory #68

Closed
frostming opened this issue Aug 19, 2021 · 6 comments · Fixed by #90
Closed

The executable bit is lost for scripts in data directory #68

frostming opened this issue Aug 19, 2021 · 6 comments · Fixed by #90
Labels
type: bug A confirmed bug or unintended behavior

Comments

@frostming
Copy link
Contributor

installer seems only to copy streams and won't copy the executable bit when installing

@FFY00
Copy link
Member

FFY00 commented Aug 19, 2021

Oh, right, because we only work with streams for the abstraction. We will have to add something for the permissions/mode. Do we need this in anything other than the user-provided scripts? If not, would it be reasonable to simply set the executable on all installed scripts?

@FFY00
Copy link
Member

FFY00 commented Aug 19, 2021

Nevermind, users can ship executables as package resources 😅

@pradyunsg
Copy link
Member

Got an example zip file for this?

@pradyunsg pradyunsg added the type: bug A confirmed bug or unintended behavior label Sep 26, 2021
@pradyunsg
Copy link
Member

I've got a patch for this in pradyunsg@d2cd484, since I wrote that as part of #74. Gonna split that out though, since that's unrelated to that broader PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A confirmed bug or unintended behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants