discover: Sanity check compilation before checking for missing dependencies#313
Conversation
|
(heh, somehow I manage to keep hitting this - from three years ago: #33) As a related matter, I just checked the opam-repository and it's unfortunately very divided on whether I know |
|
I know the toplevel is deprecated and probably only on life support at this point. But it would be nice if it was its own opam sub package as well as I'm not sure how long we'd wait for 3.0. |
|
Thanks for this, and the other PR. I'm pretty sure 3.0.0 is coming out this weekend (hopefully). |
I have a bit of a novel environment in opam2nix where only the (transitive) dependencies are made available to each package while building. Because of this,
lwtdoes not havencursesavailable unless it depends transitively on on anncursesdepext (orconf-ncurses). There's been a lot of back and forth on where the dependency onncursesbelongs, but it hopefully won't be contentious to fail earlier if a basic toplevel can't be compiled.This PR gives a better error message - currently
lwttells me that "pthread" isn't available, because that's just the first thing it attempts to compile. This is misleading, since it would tell me everything is unavailable except for the real problem (in my case, ncurses).