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

rinari repository not up to date with rails #198

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tonini
Copy link
Collaborator

@tonini tonini commented Aug 28, 2013

Hi,

rinari isn't actually working well with rails 3/4 and one of the reasons might be that there're no updates for the last 5 years.

the rinari fork https://github.com/eschulte/rinari is well maintained by @purcell, maybe we consider to use that source instead?

@senny whats your opinion on that?

@senny
Copy link
Owner

senny commented Aug 28, 2013

We should either drop it, or update it. However there is an important caveat: Rinari bundles some tools used in the ruby-bundle. I'm not sure what the current situation is and if we are overwriting these libraries or are actually using them.

  • ruby-mode
  • ruby-compilation
  • ...

@purcell
Copy link

purcell commented Aug 28, 2013

@senny Rinari doesn't bundle ruby-mode or inf-ruby any more: ruby-compilation.el and jump.el are in the same github repo, but they (and rinari) are made available as 3 separate packages.

@senny
Copy link
Owner

senny commented Aug 28, 2013

oh, that is great news! I've given up on Rinari some time ago but I think it's a good time to revisit this decision.

@tonini are you interested in working on a PR with an updated version of Rinari? I'm willing to work with it for some time to see how it fits into cabbage.

@purcell do you happen to know if Rinari works with Enhanced-Ruby-Mode?

@tonini
Copy link
Collaborator Author

tonini commented Aug 28, 2013

@senny Great! sure I will make a PR with the updated version.

@purcell
Copy link

purcell commented Aug 28, 2013

@senny Yes, I think so. We have at least one chunk of code (syntax-highlighting of Rails keywords) which explicitly supports both ruby-mode and enhanced-ruby-mode.

@senny
Copy link
Owner

senny commented Aug 28, 2013

@purcell great. I'm gonna try those in combination. Thank you for jumping in and sharing your insight 💛 .

the rinari package created by @technomancy isn't maintained anymore
and behind rails version 3/4. The rinari fork under github.com/eschulte/rinari
is up to date with the current state of rails.

* https://github.com/eschulte/rinari.git is the new source of the rinari
  version which cabbage use.
* cabbage need to provide the `findr` package because it isn't included in
  the `jump` package repository anymore.
@tonini
Copy link
Collaborator Author

tonini commented Aug 28, 2013

@senny the branch is ready for testing.

You have to git submodule sync before doing a scripts/update.sh, but I think you know that already. ;)

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.

3 participants