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

.install files and executable bit #2430

Closed
dbuenzli opened this issue Feb 10, 2016 · 4 comments
Closed

.install files and executable bit #2430

dbuenzli opened this issue Feb 10, 2016 · 4 comments

Comments

@dbuenzli
Copy link
Contributor

Currently we have:

File permissions are set to 0644, except for libexec, bin, sbin, stublibs for 
which it’s 0755. umask may apply.

Is there a strong reason in doing so ?

This is quite annoying for some software I'm developing. I'm storing arbitary files in the etc directory of a package. These files can be copied over somewhere else on user request, but the executable bit of these file is an important bit of information.

So my proposal would be to always preserve the executable bit, whatever the section.

@dbuenzli
Copy link
Contributor Author

Other use case would include having sample ocaml scripts for a library in a share or doc directory.

@AltGr
Copy link
Member

AltGr commented Feb 12, 2016

This originally mimicked the behaviour of the UNIX install command; not sure it's strictly needed, but it may help with files coming from a filesystem or archive without file permissions. Another option would be a way to specify the desired permissions in the .install file.

@dra27
Copy link
Member

dra27 commented Oct 16, 2020

It's hard to see what the story could be for compatibility for this change - it could conceivably be done by having a package flag to opt-in to a perms-preserving installation, but this couldn't be relied on without constraining a package to a newer opam client (and that also wouldn't work if using opam-installer directly). It can - at least for the foreseeable future - be done with manual installation commands. Is this still an issue, or can this can be closed, therefore?

@dbuenzli
Copy link
Contributor Author

I think it could be, you may want to install scripts in share that you don't want to have in bin. But in that case the package can install the files itself rather than rely on .install files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants