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
mercurial: install zsh completions #51712
Conversation
@@ -50,7 +50,8 @@ in python2Packages.buildPythonApplication { | |||
chmod u+x $out/share/cgi-bin/hgweb.cgi | |||
|
|||
# install bash completion | |||
install -D -v contrib/bash_completion $out/share/bash-completion/completions/mercurial | |||
install -v -m644 -D contrib/bash_completion $out/share/bash-completion/completions/mercurial | |||
install -v -m644 -D contrib/zsh_completion $out/share/zsh/site-functions/_hg |
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 is going to the nix store anyway, so, maybe, -m444
?
@@ -50,7 +50,8 @@ in python2Packages.buildPythonApplication { | |||
chmod u+x $out/share/cgi-bin/hgweb.cgi | |||
|
|||
# install bash completion | |||
install -D -v contrib/bash_completion $out/share/bash-completion/completions/mercurial | |||
install -v -m644 -D contrib/bash_completion $out/share/bash-completion/completions/mercurial |
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.
Completion doesn't work in bash it seems, we need to copy to hg
, _hg
or hg.bash
for bash-completion
to lazy load the file. Should probably fix it while we're here :)
@@ -50,7 +50,8 @@ in python2Packages.buildPythonApplication { | |||
chmod u+x $out/share/cgi-bin/hgweb.cgi | |||
|
|||
# install bash completion |
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.
And refresh that comment to install shell completions
or install bash/zsh completions
Sorry for delay, updated as suggested!
Tested bash completion before/after, confirmed didn't work and now does
^_^.
…On Mon, 10 Dec 2018 13:50:28 -0800, Renaud ***@***.***> wrote:
c0bw3b commented on this pull request.
> @@ -50,7 +50,8 @@ in python2Packages.buildPythonApplication {
chmod u+x $out/share/cgi-bin/hgweb.cgi
# install bash completion
And refresh that comment to `install shell completions` or `install bash/zsh completions`
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
#51712 (review) part: text/html
|
Thanks :) |
Motivation for this change
They're useful, you know :).
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)