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

add new variable %{ld_conf}% #198

Closed
tuong opened this issue Sep 25, 2012 · 4 comments
Closed

add new variable %{ld_conf}% #198

tuong opened this issue Sep 25, 2012 · 4 comments

Comments

@tuong
Copy link

tuong commented Sep 25, 2012

When using the existing OCaml (system), opam install P could fail if the following conditions are encountered :

  • P tries to write in $(ocamlc -where)/ld.conf,
  • $(ocamlc -where)/ld.conf can only be read.

Here, opam init should copy $(ocamlc -where)/ld.conf to a writable file so that packagers could later refer to this file as %{ld_conf}%.

@samoht
Copy link
Member

samoht commented Sep 25, 2012

Do you know which package is trying to do that (ocamlfind apart)

Thomas

Le 25 sept. 2012 à 17:53, Frédéric Tuong notifications@github.com a écrit :

When using the existing OCaml (system), opam install P could fail if the following conditions are encountered :

P tries to write in $(ocamlc -where)/ld.conf,
$(ocamlc -where)/ld.conf can only be read.
Here, opam init should copy $(ocamlc -where)/ld.conf to a writable file so that packagers could later refer to this file as %{ld_conf}%.


Reply to this email directly or view it on GitHub.

@tuong
Copy link
Author

tuong commented Sep 25, 2012

For instance camlzip which has the empty list as dependencies.

@samoht
Copy link
Member

samoht commented Sep 27, 2012

OPAM set LD_LIBRARY_PATH to point to ~/.opam/<alias>/lib/stublibs so normally you do not need to change ld.conf at all (ie. we should patch these packages to not do that).

For camlzip, I've added a META file, and I guess we should use a more complete .install file instead of using make install. So I'm closing this issue for now on, as copying ld.conf elsewhere is not the right solution to this problem.

@xavierleroy
Copy link

For the record: I think OPAM setting CAML_LD_LIBRARY_PATH is harmful and useless, see my comment here ocaml/ocaml#6532 (comment)

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