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
coqPackages: fix + add multinomials 1.3 + coqeal 1.0.0 #62718
Conversation
@vbgl can you build the appropriate packages (coqPackages_8_8.coqeal, coqPackages_8_9.coqeal, coqPackages_8_9.multinomials) |
Evaluation fails:
Also (unrelated to this PR, I guess, but worth mentioning), the naming is wrong: this |
d73e9cb
to
f3a8f41
Compare
I could not reproduce this,
and all the variants I could think of give the right versions. |
This should be fixed now |
|
@vbgl what would be the recommended name instead of |
@vbgl, would it be |
CoqEAL does not build with Coq 8.7 or 8.9, e.g.: builder for '/nix/store/n57zd4031f9y3xkjfcxz122m938hhaha-coq8.8-mathcomp-1.9.0-coqeal-1.0.0.drv' failed with exit code 2; last 10 log lines: About naming, something looks wrong: it is OK to have different versions of a library for different versions of Coq. But for a given version of Coq, one version of the library should be enough. Otherwise you are bound to have to deal with an exponential mess. |
It's not enough, some libraries are compatible only with a restricted set of versions of the others... |
f3a8f41
to
e23d4bb
Compare
This should be fixed now... |
e23d4bb
to
2c6c22b
Compare
@vbgl is it okay to merge now? |
mathcomp-analysis does not build with Coq 8.7. |
2c6c22b
to
58f5121
Compare
@vbgl should be fixed now |
pkgs/top-level/coq-packages.nix
Outdated
|
||
mathcomp_1_7-finmap mathcomp_1_7-bigenough mathcomp_1_7-analysis | ||
mathcomp_1_7-multinomials mathcomp_1_7-real-closed | ||
mathcomp_1_7-finmap_1_0 | ||
mathcomp_1_7-finmap_1_0 mathcomp_1_7-coqeal |
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.
This guy does not exist (mathcomp_1_7-coqeal)
pkgs/top-level/coq-packages.nix
Outdated
|
||
mathcomp_1_9-finmap mathcomp_1_9-bigenough mathcomp_1_9-analysis | ||
mathcomp_1_9-multinomials mathcomp_1_9-real-closed; | ||
mathcomp_1_9-multinomials mathcomp_1_9-real-closed mathcomp_1_9-coqeal; |
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.
This one neither (mathcomp_1_9-coqeal).
58f5121
to
5135858
Compare
pkgs/top-level/coq-packages.nix
Outdated
|
||
mathcomp_1_9-finmap mathcomp_1_9-bigenough mathcomp_1_9-analysis | ||
mathcomp_1_9-multinomials mathcomp_1_9-real-closed; | ||
mathcomp_1_9-multinomials mathcomp_1_9-real-closed |
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.
missing semicolon?
5135858
to
7d099c1
Compare
@GrahamcOfBorg build coqPackages_8_9.coqeal |
Motivation for this change
Fixing my buggy implementation of generation of additional packages for mathcomp and added multinomials release 1.3 and coqeal release 1.0.0
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)