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
alacritty: install shell completions #43143
Conversation
Success on aarch64-linux (full log) Attempted: alacritty Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: alacritty Partial log (click to expand)
|
@@ -92,6 +92,10 @@ in buildRustPackage rec { | |||
patchelf --set-rpath "${stdenv.lib.makeLibraryPath rpathLibs}" $out/bin/alacritty | |||
'') + '' | |||
|
|||
install -D alacritty-completions.zsh "$out/share/zsh/site-functions/_alacritty" | |||
install -Dt "$out/etc/bash_completion.d" alacritty-completions.bash | |||
install -D alacritty-completions.fish "$out/share/fish/vendor_completions.d/alacritty.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.
It really bothers me how the second line has the target as the first parameter while the others do not!
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 could change that, but then it would bother me that alacritty-completions.bash
turns up twice in that line :D
3289662
to
86d8507
Compare
Success on aarch64-linux (full log) Attempted: alacritty Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: alacritty Partial log (click to expand)
|
Awesome :) |
Motivation for this change
https://github.com/jwilm/alacritty#shell-completions
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)cc @Mic92