-
Notifications
You must be signed in to change notification settings - Fork 358
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
Erroneous message/return code with opam remove when are unavailable #4890
Comments
related to #4759 (comment) |
Thanks for the report!
In fact, it is not failing, the error message is misleading (and wrong) and the error code too. There is no removal possible as the package is not installed; If the package were installed, it would be another message, a warning that system dependency is no more in the system, and then package removal.
It is a virtual package, and this is not handled for alpine yet, as pointed in linked issue.
This one is an expected behavior. |
Thank you for your answer!
Ok, I think I understand better now. Any chance that opam would exit with error code
Indeed. Thank you very much for the reference to the other issue.
You are right, I made a confusion between |
|
unavailable system package
error with llvm-dev
on alpine
whereas apk add llvm-dev
succeeds
(somewhat) related to #4493 |
fixed by #4995 |
I am facing some failures with
opam-ci
while trying to packagelibclang
(ocaml/opam-repository#19824), and I don't understand theopam
behavior here.In the docker container created by
opam-ci
(based on the imageocaml/opam:alpine-3.13-ocaml-4.13
), the commandopam remove conf-libclang.12
fails with the following error:I am a bit surprised that
opam remove
can fail on an uninstalled package for an unavailable dependency, but more surprisingly this package is available on the system:sudo apk add llvm-dev
successfully installs llvm!Moreover, if I try
opam install --deps-only conf-libclang.12
(before installing thellvm-dev
package manually viaapk
), the command answersNothing to do.
.Any hint about how to fix that? Thank you very much.
The text was updated successfully, but these errors were encountered: