-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
coq_8_10: 8.10+β3 → 8.10.0 #70642
coq_8_10: 8.10+β3 → 8.10.0 #70642
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.
✨
This comment has been minimized.
This comment has been minimized.
It has been merged in |
Ah right, I misunderstood what was meant by "land in". |
I don’t know how long it takes for staging to be merged into master, so I propose to (temporarily) disable CoqIDE for Coq 8.10. @GrahamcOfBorg build coqPackages_8_10.interval |
OK to me. Longer term we may want to split the Coq and CoqIDE packages, but this will probably be easier to do after Coq has switched to Dune as the primary build system. |
BTW, I just noticed that Coq 8.10+beta2 is in nixos-19.09. Given that this is a pre-release version, shouldn't the update to 8.10+beta3, and the upcoming one to 8.10.0 (maybe after CoqIDE is made available again) be backported? |
The release-19.09 branch has GTK3 at 3.24.10, so you can have CoqIDE there I guess. |
GTK3 at version 3.24.11 has broken pkg-config files
coqPackages.coq-elpi: master → 1.1.0
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.
Note that I cannot merge the PR for you.
Motivation for this change
Major release.
This PR is currently a draft, waiting for #70323 to land in
master
.Fixes #66217
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)