nix-community / nix-zsh-completions

ZSH Completions for Nix
BSD 3-Clause "New" or "Revised" License
228 stars 31 forks source link

replace sha256sum with nix-hash #14

Closed LnL7 closed 6 years ago

LnL7 commented 6 years ago

sha256sum isn't necessarily available on all platforms (eg. darwin) and nix-hash is already used for the base32 conversion.

hedning commented 6 years ago

This looks good. Do you mind if I apply your fix to nix-bash-completions too?

LnL7 commented 6 years ago

Sure, I don't think there's anything zsh specific in there.

hedning commented 6 years ago

Yeah, pasting it to the bash version worked fine.

Thanks :)