-
-
Notifications
You must be signed in to change notification settings - Fork 14.7k
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.19.1 → 8.19.2 #318799
coq: 8.19.1 → 8.19.2 #318799
Conversation
There is now an optional dependency on memprof-limits. Should we add this as a dependency for versions > 8.19.2? I've added memprof-limits here: #318800 |
Can you please elaborate on this new (optional) dependency? Should it be opt-in or opt-out for nixpkgs users? |
I think for the time being lets ignore it. We will need to think a bit harder about how we want to depend on it in the future. |
This comment has been minimized.
This comment has been minimized.
Result of 72 packages built:
|
Successfully created backport PR for |
Update after NixOS/nixpkgs#318799
Update after NixOS/nixpkgs#318799
Description of changes
https://coq.inria.fr/doc/V8.19.2/refman/changes.html#changes-in-8-19-2
Tested there: coq-community/coq-nix-toolbox#223
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.