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
kenlm: init at unstable-2020-11-14 #103858
base: master
Are you sure you want to change the base?
Conversation
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.
A few suggestions :)
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.
Thank you for the detailed review! (If you can't tell, I don't write a lot of Nix and cargo-culted much of this commit).
I've resolved all of your comments; in general, I did exactly as requested, except that I removed the "doCheck" entirely (there's no obvious test target in kenlm
).
pname = "kenlm-unstable"; | ||
version = "2020-11-03"; |
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.
pname = "kenlm-unstable"; | |
version = "2020-11-03"; | |
pname = "kenlm"; | |
version = "unstable-2020-11-03"; |
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.
@IvarWithoutBones suggested the opposite of this change and referred to #68518
I marked this as stale due to inactivity. → More info |
nativeBuildInputs = [ cmake ]; | ||
buildInputs = [ boost eigen xz zlib bzip2 ]; | ||
|
||
meta = with stdenv.lib; { |
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.
meta = with stdenv.lib; { | |
meta = with lib; { |
Motivation for this change
kenlm
is a natural language processing tool for dealing with language models: estimating, filtering, and querying.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)