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
gripgen: init at 0.8 #72893
gripgen: init at 0.8 #72893
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.
Cool! Works, tried to "grip" through nixpkgs.
pkgs/tools/text/gripgen/default.nix
Outdated
owner = "sc0ty"; | ||
repo = "grip"; | ||
rev = "v${version}"; | ||
sha256 = "sha256:0bkqarylgzhis6fpj48qbifcd6a26cgnq8784hgnm707rq9kb0rx"; |
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.
sha256 = "sha256:0bkqarylgzhis6fpj48qbifcd6a26cgnq8784hgnm707rq9kb0rx"; | |
sha256 = "0bkqarylgzhis6fpj48qbifcd6a26cgnq8784hgnm707rq9kb0rx"; |
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.
Will do in a wee bit
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.
done in 125615b
{ boost, catch2, cmake, fetchFromGitHub, stdenv }: | ||
|
||
stdenv.mkDerivation rec { | ||
pname = "gripgen"; |
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 = "gripgen"; | |
pname = "grip"; |
gripgen is only the tool to generate the database. (Also change commit and 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.
Grip is already taken,
Its some gui app, that's why I used gripgen
Ideally, we should have all our pkgs namespaced,
`tools.text.grip
but that's another battle
@tex has the thing sorted, so I can close this one |
Motivation for this change
grip (gripgen) was missing in nixpkgs,
http://sc0ty.pl/2016/12/grip-indexed-grep/
grip allows me to grep through a 120GB source codebase in less than 10 secs
Things done
sandbox
innix.conf
on non-NixOS linux)Tested compilation of all pkgs that depend on this change usingnix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)