Fix command-not-found handler for Suse and Fedora #1280

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
@Gonzih
Contributor

Gonzih commented Feb 2, 2014

fixes #1208

- if test -f /usr/lib/command-not-found
+ # First check if we are on OpenSUSE since SUSE's handler has no options
+ # and expects first argument to be a command and second database
+ if test -f /etc/SuSE-release

This comment has been minimized.

@zanchey

zanchey Feb 2, 2014

Member

I suggest if begin; test -f /etc/SuSE-release; and test -f /usr/bin/command-not-found; end (path may need checking).

@zanchey

zanchey Feb 2, 2014

Member

I suggest if begin; test -f /etc/SuSE-release; and test -f /usr/bin/command-not-found; end (path may need checking).

This comment has been minimized.

@Gonzih

Gonzih Feb 2, 2014

Contributor

Done. Not sure that this is necessary since suse have it always preinstalled, but ok lets keep this.

@Gonzih

Gonzih Feb 2, 2014

Contributor

Done. Not sure that this is necessary since suse have it always preinstalled, but ok lets keep this.

This comment has been minimized.

@zanchey

zanchey Feb 2, 2014

Member

Yikes, sorry, I should have made this type -p command-not-found. I'll do that in the merge.

@zanchey

zanchey Feb 2, 2014

Member

Yikes, sorry, I should have made this type -p command-not-found. I'll do that in the merge.

@zanchey

This comment has been minimized.

Show comment
Hide comment
@zanchey

zanchey Feb 2, 2014

Member

Merged with squash here:
To git@github.com:fish-shell/fish-shell.git
a1b43b7..27b6fe6 master -> master
Thanks!

Member

zanchey commented Feb 2, 2014

Merged with squash here:
To git@github.com:fish-shell/fish-shell.git
a1b43b7..27b6fe6 master -> master
Thanks!

@zanchey zanchey closed this Feb 2, 2014

@ghost ghost assigned zanchey Feb 2, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment