-
Notifications
You must be signed in to change notification settings - Fork 166
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
Update several packages for Coq 8.19 and recent releases #2993
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andrew-appel standard advice I give these days: since GitHub no longer guarantees stability of generated archives such as https://github.com/VeriNum/vcfloat/archive/refs/tags/v2.2.tar.gz
, you may want to manually upload an archive to GitHub as is done for Coq: https://github.com/coq/coq/releases/tag/V8.19.1 ( coq-8.19.1.tar.gz
). Essentially, you can just download a generated archive and upload it again to ensure GitHub doesn't change the checksum over time.
@andrew-appel the package |
coq-vst.2.14 now (correctly) depends on coq-vst-zlist.2.13 which did not need any updating (except that its opam now permits coq 8.19). Now use a special asset for coq-vst-2.14 and for coq-vcfloat-2.2 as suggested. |
@andrew-appel there is the following build error in CI for
This is for the following possibly-relevant dependencies:
Any ideas? |
We have a bunch of OOM kills and timeouts in CI. So due to CI limitations, doesn't look like we can check everything here, I'm going to merge and hope for the best... |
I think this may have broken vcfloat on Coq 8.17, which presumably now needs an upper bound on the vst version.
|
coq-vcfloat doesn't depend on coq-vst, so it probably doesn't need an upper bound on the vst version. Just looking at the source code of Prune.v in github (without seeing the proof state in Coq), I'd guess one possibility is that coq-vcfloat.2.1.1 needs an upper bound on the coq-interval version, maybe |
Update the following packages for Coq 8.19 and for recent releases of other packages on which they depend: