-
-
Notifications
You must be signed in to change notification settings - Fork 12.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
isabelle: 2022 -> 2023 #243497
isabelle: 2022 -> 2023 #243497
Conversation
86b99bb
to
ca5a436
Compare
c7723aa
to
3ed5f67
Compare
3ed5f67
to
b4f1822
Compare
b4f1822
to
1f7cb6b
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/2684 |
1f7cb6b
to
ce9429b
Compare
ce9429b
to
7c17ab6
Compare
@SuperSandro2000 can this be merged? |
7c17ab6
to
d329d58
Compare
In the meantime a release of the linter was tagged, updated the commit accordingly |
Would it make sense to also include the AFP as a component? I defined this derivation for it: https://github.com/lukasstevens/dotfiles/blob/a971a4c0bee283d27351413e1cc47e239812ad17/nix/isabelle/components/afp.nix |
@lukasstevens the AFP doesn't sound like a component that should be included in nixpkgs. It's important but afaik is changing too often and not really something that fits into a software package collection. |
pkgs/applications/science/logic/isabelle/components/isabelle-linter.nix
Outdated
Show resolved
Hide resolved
d329d58
to
f507b81
Compare
@SuperSandro2000 can we merge this now? |
At least the server component of Isabelle 2023 (from this MR) had also been tested on arm64 (rp4). |
@SuperSandro2000 sorry for the ping, but this has been sitting for a while now |
f507b81
to
3e90293
Compare
@ofborg build tlaps tlaps.passthru.tests |
Thanks! |
Description of changes
Changelog file
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)