Skip to content
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 compatibility #50

Open
douglascamata opened this issue May 31, 2016 · 11 comments
Open

fish-shell compatibility #50

douglascamata opened this issue May 31, 2016 · 11 comments

Comments

@douglascamata
Copy link

fish-shell uses, like me, are very forgotten when it comes to libraries that needs to be loaded into the shell. It would be nice to see some integration here.

@taylor
Copy link
Owner

taylor commented May 31, 2016

This was added in #33. Is there a problem with fish support?

@douglascamata
Copy link
Author

douglascamata commented May 31, 2016

I didn't see any documentation about this. The "tested shells" section in the README didn't mention fish-shell and there are no setup instructions for fish-shell specifically... so I assumed there's no support. Maybe documentation about it should be added?

@taylor
Copy link
Owner

taylor commented May 31, 2016

@douglascamata pull request welcome :)

@douglascamata
Copy link
Author

I'll see what I can do 👍

@jfis
Copy link

jfis commented Jun 22, 2016

using

test -s "$HOME/.kiex/scripts/kiex" && source "$HOME/.kiex/scripts/kiex"

i got

Unsupported use of '&&'. In fish, please use 'COMMAND; and COMMAND'.
fish: test -s "$HOME/.kiex/scripts/kiex" && source "$HOME/.kiex/scripts/kiex"

then changing to

test -s "$HOME/.kiex/scripts/kiex.fish" and; source "$HOME/.kiex/scripts/kiex.fish"

i got

Unsupported use of '&&'. In fish, please use 'COMMAND; and COMMAND'.
~/.kiex/scripts/kiex (line 3):   && source $HOME/.kiex/scripts/kiex.csh \
                                  ^
from sourcing file ~/.kiex/scripts/kiex
called on standard input

i was able to get fish working by skipping scripts/kiex and pointing to scripts/kiex.fish directly

test -s "$HOME/.kiex/scripts/kiex.fish" and; source "$HOME/.kiex/scripts/kiex.fish"

everything but sub-shell worked for me after that

thanks for kiex!

@zzztuzzz
Copy link

THX!

@taylor
Copy link
Owner

taylor commented Oct 19, 2016

The instructions shown during install will have to be updated for fish -- https://github.com/taylor/kiex/blob/master/kiex#L379. It is already done for some parts of the program such as https://github.com/taylor/kiex/blob/master/kiex#L379

I'll get fish setup in my test environment.

@howdoicomputer
Copy link

kiex use also doesn't work because kiex generates a bash compliant env file.

For my elixir version, I manually changed it to this:

$ cat elixir-1.3.4.env 
set -xg ELIXIR_VERSION 1.3.4
set -xg PATH $HOME/.kiex/elixirs/elixir-1.3.4/bin $PATH
set -xg MIX_ARCHIVES $HOME/.kiex/mix/archives/elixir-1.3.4

@optikfluffel
Copy link

Any updates on this?

@carina-akaia
Copy link

carina-akaia commented Dec 3, 2017

@jfis, I have an error in kiex.fish (at least for current fish version)
I am corrected it for a little bit:
test -s "$HOME/.kiex/scripts/kiex.fish"; and source "$HOME/.kiex/scripts/kiex.fish"
Now everything is works.
Just add above string to the ~/.config/fish/config.fish file
Enjoy!

@carina-akaia
Copy link

maybe issue must be closed now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants