andsens / homeshick

git dotfiles synchronizer written in bash
MIT License
2.1k stars 147 forks source link

Silence homeshick binary finding in fish shell #208

Closed languitar closed 2 years ago

languitar commented 2 years ago

With a recent system update, type -P in fish shell started printing an error message in case no homeshick binary exists on PATH. This is an expected condition in the homeshick fish wrapper and therefore the error message should probably not reach the user.

andsens commented 2 years ago

It seems like there's a switch for that specific case: https://fishshell.com/docs/current/cmds/type.html

-q or --query Suppresses all output; this is useful when testing the exit status. For compatibility with old fish versions this is also --quiet.

Could you check if type -q also works and change the PR if that's the case?

languitar commented 2 years ago

Unfortunately that doesn't work in combination with -P:

$ type -q -P homeshick
type: invalid option combination
andsens commented 2 years ago

Roger that, merged.