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
zsh-autosuggestions: init at 0.3.3 #21792
Conversation
a546237
to
7ff69a2
Compare
|
||
installPhase = '' | ||
install -d $out/share/zsh-autosuggestions | ||
install zsh-autosuggestions.zsh \ |
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.
you can skip the first install by adding -D to the second one.
|
||
buildInputs = [ zsh ]; | ||
|
||
buildPhase = ""; |
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.
buildPhases = [ "unpackPhase" "installPhase" ]; instead of empty buildPhase.
stdenv.mkDerivation rec { | ||
name = "${pkgName}-${version}"; | ||
|
||
src = fetchurl { |
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.
fetchFromGitHub can be used here.
7ff69a2
to
dca0d65
Compare
Thanks @Mic92 for the suggestions of improvement. The commit has been changed accordingly. Weirdly enough, the hash didn't stay the same when I switched to fetchFromGitHub. |
Added a related `programs.zsh.enableAutosuggestions` option
dca0d65
to
7c62e12
Compare
That's to be expected because fetchFromGitHub normalizes the timestamps in the archive in case Github chooses to rebuild the tarball. |
Motivation for this change
PR #19179 added zsh-syntax-highlighting. I needed zsh-autosuggestions, which is a plugin often paired with it as they both provide a fish-like experience.
Things done
Added the zsh-autosuggestions package.
Added also a related
programs.zsh.enableAutosuggestions
option, as advised in #19179@loskutov, I left you as a maintainer given your work on packaging zsh-syntax-highlighting.
Let me know if it's ok for you or if you want me to change that.
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)