Fish shell compatibility

[cesards]

Please tick one:

• this is a new Feature Request, a conversation was started in the Gitter CLI Development Room.

Extract script archive...
Install scripts...
Set version to 5.7.3+337 ...
Attempt update of login bash profile on OSX...
Added sdkman init snippet to /Users/xxx/.bash_profile
Attempt update of zsh profile...
Updated existing /Users/xxx/.zshrc

It adds sdkman init snippet to bash and zsh, but I'm currently using Fish (I moved from ZSH a while ago) and it doesn't seem to support it.

I might try to create a PR with the right script for Fish shell if you agree on that. And it's a good way to start my Hacktoberfest!

[reitzig]

If you'll forgive the plug, may I recommend sdkman-for-fish?

A draft for showing it while installing SDKMAN! is here: sdkman/sdkman-hooks/pull/9

Given that a complete rewrite in Go is underway (which will make special support for different shells obsolete, I assume), I don't think we should expect much "official" support here. I'm sure @marc0der will check out pull requests for documentation?

[phatblat]

There's also omf-sdk for Oh My Fish users

[davinkevin]

A default integration should be better 👍. Like @cesards proposes, I can propose.a contribution to support that if ok for you.

With sdkman-for-fish, I have to install fisher to install it… which is another plugin to manage a sdkman.

[hg1112]

Any update on this issue ?

[jonasheschl]

I would also be interested in using sdkman with the fish shell. Is there a way of using sdkman with fish as of now?

The sdkman-for-fish plugin sadly does not seem to work anymore. @reitzig mentioned that the rewrite in Go "will make special support for different shells obsolete". Though from what i can see sdkman does not use Go. Was the rewrite cancelled, has there been another rewrite? Is special support for different shell obsolete, in other words, does sdkman work with every shell out-of-the box some way, as implied by the reitzigs comment?

[reitzig]

The sdkman-for-fish plugin sadly does not seem to work anymore

While I should definitely push an update at some point, it still works for me. If you could leave your issue over in my repo, I'd appreciate it.

Was the rewrite cancelled, has there been another rewrite?

There hasn't been, and to my knowledge, there isn't one planned. As of now, sdkman is a Bash application manipulating environment variables, which means you'll need some form of wrapper to make it work in other shells.

[jonasheschl]

Thank you for the info.

I just looked into setting up sdkman-for-fish again, the project does indeed work. Sorry for that, my bad. I assumed the installation was broken when I saw the last commit to the repo being from 2020 and sdkman-for-fish prompting that sdkman could not be found, even though I had previously installed it.

sdkman-for-fish did successfully locate sdkman when I answered the prompt "sdkman could not be found, do you want to install it" with yes.

[reitzig]

Thanks, glad it worked out!

Did you run afoul of reitzig/sdkman-for-fish#34, by any chance? If not, I'd be interested in the specifics of your setup. sdkman-for-fish is definitely supposed to find sdkman (at the default location, as of now)!

[jonasheschl]

It was like this for me:

• I installed sdkman in the default path
• I tried to install sdkman-for-fish
• While installing, sdkman-for-fish prompted me that sdkman could not be found, and if i wanted to install it [y/N]
• Since I had previously installed sdkman, I assumed that sdkman-for-fish was broken and aborted the install

A few days later, when you mentioned that sdkman-for-fish was still working for you, I attempted the installation again. This time, when sdkman-for-fish prompted me that sdkman could not be found and whether I wanted to install sdkman, I entered y.

After I entered y and hit return, sdkman-for-fish successfully found sdkman and worked without a flaw.

So not a real issue here, just a confusion on my part.

Oh, and thank you for your quick response.

[ibqn]

Hi guys, I am missing official support for the fish shell

[marc0der]

Hi @ibqn, since the whole of SDKMAN is currently written in bash, it is not possible to officially support fish shell.

[dmikusa]

In case it's helpful to anyone, I added this function to my fish config (~/.config/fish/config.fish). It wraps the sdk command so you can run it from your fish shell.

function sdk
    bash -c "source '~/.sdkman/bin/sdkman-init.sh'; sdk $argv[1..]"
end

and this which adds your candidates to $PATH.

fish_add_path (find ~/.sdkman/candidates/*/current/bin -maxdepth 0)

Doing this, most of the sdk commands work. I can install, remove, and set defaults, as well as use the tools installed. The only thing that definitely doesn't work is sdk use to temporarily change a version because of the way I'm wrapping the call.

[gengjiawen]

@dmikusa-pivotal Thx for sharing, works like a charm.

I changed to $HOME for compatibilty

function sdk
    bash -c "source '$HOME/.sdkman/bin/sdkman-init.sh'; sdk $argv[1..]"
end

[vonbarnekowa]

Has anyone found a solution to make sdk use work?

[reitzig]

If you'll forgive the plug, may I recommend sdkman-for-fish?
☝️

Long due an update, but sdk use has always worked for me.
Addendum: Patched to be current with sdk as of June this year.