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 plugin support #85

Open
calebmeyer opened this issue Feb 27, 2015 · 10 comments
Open

Fish plugin support #85

calebmeyer opened this issue Feb 27, 2015 · 10 comments

Comments

@calebmeyer
Copy link

I have had great success with jenv so far (all the version and add commands are spot on), so I thought I'd try out some plugins. I'm using fish and oh-my-fish. The jenv plugin is definitely loaded, as it's referenced in the error:

jenv enable-plugin maven                                                                                                                  
fish: Expected a command name, got token of type 'Run job in background'. Did you mean 'COMMAND; and COMMAND'? See the help section for the 'and' builtin command by typing 'help and'.
/Users/cm022291/.oh-my-fish/functions/jenv.fish (line 4): begin; echo maven plugin activated && jenv rehash
                                                                                              ^
in . (source) call of file '-',
    called on line 37 of file '/Users/cm022291/.oh-my-fish/functions/jenv.fish',

in function 'jenv',
    called on standard input,
    with parameter list 'enable-plugin maven'

source: Error while reading file '-'

I looked at the jenv.fish file, and I wasn't able to find && anywhere in it.

@kptdobe
Copy link

kptdobe commented Mar 12, 2015

Plugins are not "fish" ready, only implemented for bash. If I find the time, I'll write the maven one that I need too.

@cassiosp
Copy link

Any movement around this?

@Mazorius
Copy link

I had the same issue.
Did someone already fix that?

@Mazorius
Copy link

I fixed the maven problem by instead adding to config.fish set -e JAVA_HOME adding set JAVA_HOME ( jenv javahome )

@dotCipher
Copy link

Thanks, I had the same issue with using the export plugin in fish shell. The solution @Mazorius provided above works as a workaround.

@dbrimley
Copy link

I'm falling foul of using the maven plugin with fish. However I get a different error. I've tried @Mazorius solution but that didn't help...

screenshot 2018-05-12 19 32 51

@Mazorius
Copy link

Mazorius commented May 22, 2018

I do not install the plugin, because of this issue.

I only add the line set -e JAVA_HOME ( jenv javahome ) to my fish.config.

Maybe I will also look into the scripts if I find some free time slot.

@yanns
Copy link

yanns commented Dec 3, 2018

Using set -e JAVA_HOME ( jenv javahome ) does not work for me:

set: Values cannot be specfied with erase
~/.config/fish/config.fish (line 3):
set -e JAVA_HOME ( jenv javahome )
^
from sourcing file ~/.config/fish/config.fish
	called during startup

I had to use set -gx JAVA_HOME ( jenv javahome )

@peterwillcn
Copy link

@vorburger
Copy link

wget https://raw.githubusercontent.com/gcuisinier/jenv/master/fish/jenv.fish

This is now available in this repo (here); I've diff'd it, it's the same, at 8c66619 (file) and 2329e5c (HEAD).

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

9 participants