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
Makefile: opam install the lock file #1415
Comments
Indeed we should install the |
Reopening this as #1462 was reverted. It seems given the current state of opam-monorepo we should not opam install the lock file but also we should(?!) it would be good to have clarified the expectations of opam-monorepo and what the workflow should look like. |
When installing the lockfile, we ended up in #1476 which adds lots of pins to the current opam switch. An option would be to |
According to this comment we need to opam install the lockfile:
tarides/opam-monorepo#378 (comment)
But in the code we install the .opam file (and not the .opam.locked file):
https://github.com/mirage/mirage/blob/main/lib/functoria/makefile.ml#L138-L140
CC @TheLortex @samoht
The text was updated successfully, but these errors were encountered: