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
Remove camlp5 transitional #48159
Remove camlp5 transitional #48159
Conversation
pkgs/top-level/ocaml-packages.nix
Outdated
@@ -674,6 +665,8 @@ let | |||
|
|||
process = callPackage ../development/ocaml-modules/process { }; | |||
|
|||
prooftree = callPackage ../applications/science/logic/prooftree { }; |
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 does not belong to ocamlPackages
; please leave it in all-packages.nix
.
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.
It is an OCaml package though, isn't it? Many applications are defined there then exported in all-packages.nix
. Having it there avoids listing its arguments...
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.
When we do not have it there we only have to worry about one ocaml version. You can also use ocamlPackages.callPackage
in all-packages.nix for convenience.
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.
I see.
d8b2874
to
2fdd38e
Compare
Updated. I didn't rebuild everything but there should be no functional change. |
You can also `ocamlPackages.callPackage` for convenience
How do you (sanely) override a package defined like this?
|
The |
What you usually want to override, its the full |
Ah. Yes, if you want to change a dependency for all libraries and not just the current package, then |
Motivation for this change
Remove annoying
camlp5_transitional
/camlp5_strict
duality in favor of a (strict)camlp5
. We take care of updating / removing the last bits that still require the transitional version.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)Currently runningnow-review
but testing along the way seems to indicate that everything is fine.EDIT: done.