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
ocamlPackages.janeStreet: 0.13 → 0.14 #106386
Conversation
e0e4bd6
to
9f36ebb
Compare
9f36ebb
to
0e8a8ef
Compare
Rebased on top of #106752. |
I think this update is missing |
114d673
to
5e5f541
Compare
Thanks. I’ve just added some of the missing libraries. |
Are there objections to merging this? |
I've dabbled around a bit and went through the opam files of the packages. I've noticed the following things
I've implemented changes based on that, you can see the changes comparing to your branch here. I've also changed some dependencies around based on my testing and the opam files, I'm not 100% sure about those, maybe I missed a propagation here or there, I guess a second pair of eyes would help there :) |
Also refactor to ease selection of a custom version
ocamlPackages.biocaml: 0.10.1 → 0.11.1 ocamlPackages.bistro: fix for Core 0.14 ocamlPackages.js_of_ocaml: 3.7.0 → 3.7.1 ocamlPackages.ocaml-r: 0.2.0 → 0.4.0 ocamlPackages.phylogenetics: 2020-01-25 → 2020-11-23 Co-authored-by: sternenseemann <0rpkxez4ksa01gb3typccl0i@systemli.org>
5e5f541
to
9d4d232
Compare
Thanks a lot! I’ve added your changes to the last commit of this PR. |
Result of 244 packages built:
|
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.
No objections from my side.
Motivation for this change
#106356
This removes
sqlexpr
as it cannot be built with dune 2.Closes #106344.
Closes #102451.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)