-
Notifications
You must be signed in to change notification settings - Fork 110
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
Fish shell integration is not working #55
Comments
Got it working with manually loading the init.fish file if test -z $TMUXIFIER
set -gx TMUXIFIER "$HOME/.tmuxifier"
end
# Add `bin` directroy to `$PATH`.
if not contains "$TMUXIFIER/bin" $PATH
set -gx PATH "$TMUXIFIER/bin" $PATH
end
# If `tmuxifier` is available, and `$TMUXIFIER_NO_COMPLETE` is not set, then
# load Tmuxifier shell completion.
if test -n (which tmuxifier); and test -z $TMUXIFIER_NO_COMPLETE
. "$TMUXIFIER/completion/tmuxifier.fish"
end |
Hey, it seems the shell detection code isn't detecting your shell correctly. You can force the init script to assume a specific shell by doing this:
But I'm curious as to why the detection fails. Would you mind running |
The output is:
So I think everything is right. |
I encounter this problem as well because my default shell is bash and I subsequently load fish (I haven't made it my default shell). Therefore I see this: mark@mark-mint17 /m/r/d/u/w/w/g/j/tmuxifier> echo $SHELL mark@mark-mint17 /m/r/d/u/mark> fish --version mark@mark-mint17 /m/r/d/u/w/w/g/j/tmuxifier> set | grep fish Perhaps detecting the existence of |
The root cause appears to be that Fish 2.0.0, released May 17, 2013, is part of the Ubuntu 14.04LTS/Linux Mint 17 OS distro. My brief search of the fish shell repo shows that the "source" command wasn't documented until August 14, 2013, see https://github.com/fish-shell/fish-shell/commits/5c25be51ea9ab623ae55728eee1987d0f16df255/doc_src/source.txt I am guessing "source" was released with Fish 2.1.0 and that explains the See my pull request for patches. According to the fish shell documentation, the "." alias to the source command will be deprecated in the future. So my fix is not future proof without detecting fish --version < 2.1.0 |
My 2 cents: since fish is a pretty new shell and with a niche market. I think fish users should and are able to keep their fish shell updated. So there is no need to support old version. Of course this will change after fish matures and have stable releases. |
When I'm doing the part of Setup with
eval (tmuxifier init -)
I get the following error:It seems like the
init.fish
is not loaded. Do I have to put it in the~/.config/fish
folder? I'm a fish newbie so I can't really help you much with this issue.The text was updated successfully, but these errors were encountered: