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
jackline: 2020-04-24 → 2020-09-03 #97122
Conversation
@GrahamcOfBorg build jackline |
About What case is more likely? Trying out with a custom version of, say, |
For the moment 4.08 is the only version So I suppose as soon as we have |
Regarding camlp4, it might be more valuable to rewrite the parser for SASL without the stream-parser notation of camlp4… Regarding jackline in nixpkgs, I prefer without the last change. Indeed, it would allow to write: But you are the maintainer. |
Port package to dune
5ff9e08
to
4f3310d
Compare
Alright, I'm keeping it as it was. Regarding |
IMHO, it is better not to package In the particular case of |
Motivation for this change
I also now use the required packages as inputs instead of the whole
ocamlPackages
set.@vbgl How do you feel about using
ocamlPackages.callPackage
outside ofocamlPackages
? If it's a problem, I can easily revert the change.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)