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

ERROR while installing lwt.2.4.5 #2102

Closed
UnixJunkie opened this issue May 20, 2014 · 4 comments
Closed

ERROR while installing lwt.2.4.5 #2102

UnixJunkie opened this issue May 20, 2014 · 4 comments
Labels

Comments

@UnixJunkie
Copy link
Contributor

Am I the only one getting this?
No lwt means no utop. That's not cool.
Here is the OPAM trace:
https://gist.github.com/UnixJunkie/9eab4cffa084ede9c6ff

@UnixJunkie
Copy link
Contributor Author

Easy to reproduce:

opam remove base-threads
opam install lwt # fails

opam install base-threads
opam install lwt # works

So, to me, it looks like base-threads is a hard dependency of lwt, not an optional one.
If it is really supposed to be an optional dependency of lwt, then the lwt package or lwt
itself should be corrected so that it can compile and install properly even after an

opam remove base-threads

@samoht
Copy link
Member

samoht commented Nov 1, 2014

patchs are always welcome

@aantron
Copy link
Contributor

aantron commented Jun 29, 2016

This was fixed by 878ed9d in OPAM, and current Lwt itself doesn't have this issue anymore because the "extra" library that was causing trouble has since been removed.

With the linked patch, Lwt 2.4.5 correctly compiles without base-threads.

EDIT: "extra" was removed in 2.4.7, so the linked patch is exhaustive.

@dbuenzli
Copy link
Collaborator

According to @aantron, this can be closed.

@samoht samoht closed this as completed Sep 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants