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: jackline: switch to mirage-crypto and update dependencies #85142
Conversation
See #84691 |
cf40c6e
to
f8f9f9a
Compare
Oh, that's a bit unfortunate! I've rebased all the duplicate addition out of my branch, it should be mergeable again. I added the |
For the latter I'm wondering wether we should switch to a |
0224952
to
110e22e
Compare
110e22e
to
ec2e1da
Compare
ec2e1da
to
72cefc5
Compare
Without propagation some builds of packages requiring ppx_cstruct would fail.
This update contains the switch from nocrypto to mirage-crypto
This update contains a switch to the dune build system as well as a switch from nocrypto to mirage-crypto.
Contains switch from nocrypto to mirage-crypto
72cefc5
to
0814411
Compare
@GrahamcOfBorg build jackline |
@GrahamcOfBorg eval |
Motivation for this change
Jackline and a few of the packages it depends on recently switched to mirage-crypto (a fork of nocrypto integrated with RNG code and using dune), so I figured I add the packages and update the packages that switched. This also lead to quite a long list of packages that had to be added.
For the mirage packages I mostly left out the subpackages that were intended for usage in Xen Unikernel, because we haven't yet packaged the related packages for
ocamlPackages
yet (i. e. mirage-runtime …). Let me know if I accidentally went something out erroneously.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)cc @vbgl