-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Switches are configured with OCAML_TOPLEVEL_PATH
but the variable name is OCAMLTOP_INCLUDE_PATH
#25819
Comments
Not OCAML_TOPLEVEL_PATH. See ocaml/opam-repository#25819
This reverts commit b2d98ea. We can't do that before opam switches are configured correctly see: ocaml/opam-repository#25819
Suggests that I really should move on getting the precise value in dra27@7813e70 sorted out and upstreamed 🙂 |
@dra27 do you have an ETA on that one ? I should likely do an |
Sorry for the slowness, I've at least updated ocaml.5.3.0 to support it (would you be able to test that with omod and down versions which do install to |
@dra27 thanks for your answer which it seems I missed (or didn't get notified of). I will try to test that things work when the first 5.3.0 betas get out. |
I was trying to change
omod
anddown
to stop poopingomod.top
anddown.top
intoocaml
's library directory and leave it for$(opam var lib)/toplevel
to handle it.This should be possible since OCaml 4.08.0 when ocaml/ocaml#7808 was resolved by @nojb's ocaml/ocaml#1841.
Unfortunately the compilers packages set the
OCAML_TOPLEVEL_PATH
instead ofOCAMLTOP_INCLUDE_PATH
so that doesn't work.Not sure exactly who's responsible for opam compiler packages so I tentatively cc @Octachron and @dra27.
The text was updated successfully, but these errors were encountered: