Replace ssrlia with mczify
Since Prosa now requires Coq >= 8.13 and mczify is also available for Coq >= 8.13, we could replace the home made ssrlia by the more powerful lia offered by mczify.
Merge request reports
Activity
- Resolved by Björn Brandenburg
Hi Pierre, thanks for the MR.
I have a question about CI. Is it generally a good idea to re-install packages on each CI build, or would it be better to create custom Docker images that include this dependency? Of course, creating the images and keeping them up-to-date is tedious and creates some maintenance hassle.
Edited by Björn Brandenburg
added 1 commit
- ebf848f1 - CI: don't remove dependencies of coq-mathcomp-zify
- Resolved by Björn Brandenburg
BTW, @Mbodin I don't know how to update the esy package, could you have a look? (you can take inspiration on the
coq-prosa.opam
file changes) Side question: how does esy compare to Nix which seems more commonly used in the Coq community?
added 1 commit
- 11a96574 - update documentation to mention the mczify dependency
added 13 commits
-
11a96574...be40b98b - 10 commits from branch
master
- e5305cf7 - Replace ssrlia with mczify
- 7e65d6c8 - CI: don't remove dependencies of coq-mathcomp-zify
- e8d0c3c9 - update documentation to mention the mczify dependency
Toggle commit list-
11a96574...be40b98b - 10 commits from branch
- Resolved by Björn Brandenburg
Pierre, I see that you left the old
ssrlia
only inclassic/analysis/uni/basic/tdma_wcrt_analysis.v
, while replacing it everywhere else. Is there a reason for that or just something that I should take care of?
added 1 commit
- 1383a681 - fixup! CI: run `opam update` before `opam install`
marked this merge request as draft from 1383a681
added 9 commits
-
1383a681...0b7a65db - 3 commits from branch
master
- 0254d72b - Replace ssrlia with mczify
- fa3c6444 - CI: don't remove dependencies of coq-mathcomp-zify
- 471c39e6 - update documentation to mention the mczify dependency
- ca88eec7 - remove ssrlia entirely
- 66b3310f - CI: run `opam update` before `opam install`
- 44bb743e - fixup! CI: run `opam update` before `opam install`
Toggle commit list-
1383a681...0b7a65db - 3 commits from branch
mentioned in issue #88 (closed)
enabled an automatic merge when the pipeline for 8d997a10 succeeds