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

Add instructions for archlinux #211

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add instructions for archlinux #211

wants to merge 2 commits into from

Conversation

2m
Copy link

@2m 2m commented Jun 2, 2018

Archlinux now has a jabba package which installs only the jabba binary.

That needs to be coupled with a plugin for a corresponding shell for jabba to work. For example, a plugin for oh-my-zsh: ohmyzsh/ohmyzsh#6886

@2m 2m changed the title Add instruction for archlinux Add instructions for archlinux Jun 2, 2018
@shyiko
Copy link
Owner

shyiko commented Jun 7, 2018

Just so that it would be clear: I'm not ignoring this PR. I'll merge it in as soon as ohmyzsh/ohmyzsh#6886 in closed.

@2m
Copy link
Author

2m commented Jun 8, 2018

Of course. But oh-my-zsh repo has >800 open PRs. I should probably put jabba plugin somewhere else.

@2m
Copy link
Author

2m commented Jun 17, 2018

It seems that PRs are not being merged in the oh-my-zsh repository. I have packaged up zsh jabba plugin to a separate repository (https://github.com/2m/zsh-jabba) and fixed up README in this PR.

@2m
Copy link
Author

2m commented Oct 6, 2018

I also packaged up the fish integration script and a bit of completion support here: https://github.com/2m/fish-jabba

Also updated this PR to mention fish plugin.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@shyiko Can we get this merged as soon as Marty addresses my comments in the PR?

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@2m
Copy link
Author

2m commented Jan 16, 2021

Addressed the feedback and rebased on top of the latest master.

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

Successfully merging this pull request may close these issues.

2 participants