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
navi: init at 2.7.1 #94469
navi: init at 2.7.1 #94469
Conversation
|
||
nativeBuildInputs = [ makeWrapper ]; | ||
|
||
postInstall = '' |
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.
Upstream also provides shell completion files:
https://github.com/denisidoro/navi/tree/master/shell
You install install them using installShellCompletion
after putting installShellFiles
in nativeBuildInputs
.
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.
They are not really completions, they are adding key binding for navi widget, and upstream recommends opting into it via invoking subcommand in your rc file — would we want to implicitly set it up for users?
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.
Ah, sorry for the noise. No, I don't think we should set them up implicitly.
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 worries, thank you for review!
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.
LGTM
Result of nixpkgs-review pr 94469
1
1 package built:
- navi
The program seems to work as expected. Thanks a lot for contributing this!
Motivation for this change
https://github.com/denisidoro/navi
Closes #82476 (previous attempt at packaging it) since it seems to have stagnated.
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)