MichaelAquilina / zsh-you-should-use

📎 ZSH plugin that reminds you to use existing aliases for commands you just typed
GNU General Public License v3.0
1.42k stars 44 forks source link

Use tput #63

Closed MichaelAquilina closed 5 years ago

ypconstante commented 5 years ago

I'm getting an error while starting zsh


/home/ypconstante/.local/share/zsh/zgen/MichaelAquilina/zsh-you-should-use-master/you-should-use.plugin.zsh:5: command not found: !tput
MichaelAquilina commented 5 years ago

Hi there @ypconstante. Thanks for reporting. Will take a look immediately

MichaelAquilina commented 5 years ago

@ypconstante what OS are you running on? Quite odd you dont have tput installed. From what I can tell despite that error message - you should use should still be working as normal correct? (I just need to suppress the warning)

MichaelAquilina commented 5 years ago

Uploaded 0.7.1 if you could check it out

ypconstante commented 5 years ago

It's a mint 19, and tput is available. Adding a space between ! and tput solves the error.

which tput: /usr/bin/tput
/home/ypconstante/.local/share/zsh/zgen/MichaelAquilina/zsh-you-should-use-master/you-should-use.plugin.zsh:5: command not found: !tput
ypconstante commented 5 years ago

The final solution for me was if ! type "tput" > /dev/null; then, just adding a space between between ! and tput was showing the version in the terminal.

MichaelAquilina commented 5 years ago

Interesting that this does not fail for me locally (opensuse) and on CircleCI (Ubuntu).

What version of zsh are you on?

MichaelAquilina commented 5 years ago

Interestingly, adding a space on my system between ! and tput causes it not to work correctly anymore.

MichaelAquilina commented 5 years ago

Ok updated. Could you test out the latest version?

ypconstante commented 5 years ago

It's the latest version available to mintzsh 5.4.2 (x86_64-ubuntu-linux-gnu). With the latest version everything is working as expected, thanks!