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
pythonPackages.powerline: Added fish binding #86874
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.
please take a look at https://nixos.org/nixpkgs/manual/ under the installShellfiles
section
install -dm755 "$out/share/fish/vendor_functions.d" | ||
install -m644 "powerline/bindings/fish/powerline-setup.fish" "$out/share/fish/vendor_functions.d/powerline-setup.fish" |
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.
install -dm755 "$out/share/fish/vendor_functions.d" | |
install -m644 "powerline/bindings/fish/powerline-setup.fish" "$out/share/fish/vendor_functions.d/powerline-setup.fish" | |
installShellCompletion powerline/bindings/fish/powerline-setup.fish |
... else where
nativeBuildInputs = [ installShellFiles ];
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.
I had seen that, however installShellCompletion
can't be used in this case. The integration has to be installed to vendor_functions.d and not vendor_completions.d, as Powerline provides a function not a completion. I don't see any helper functions for installing functions.
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, I see. okay then
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.
diff LGTM
@justin-gerhardt congrats on your first PR to nixpkgs! :) |
Motivation for this change
Adding the fish shell integration
Should allow use by invoking the powerline-setup function from fish
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)