ocaml / ocamlfind

The OCaml findlib library manager
Other
30 stars 30 forks source link

Using `install` as `INSTALLFILE` sets the executable bit on all installed files #59

Closed glondu closed 3 weeks ago

glondu commented 1 year ago

Discovered while updating the Debian package. As a workaround, I set INSTALLFILE to cp.

Leonidas-from-XIV commented 1 month ago

I can reproduce, if I install -c INSTALL installed then installed is +x, also on Fedora.

The configure script writes these unconditionally into Makefile.config with a fallback comment to use cp, maybe it would make sense use cp in that case.

Looks like install only really makes sense for executables given it has options for stripping etc, so maybe cp is the better choice. What do you think @gerdstolpmann?