Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Install script doesn't handle ruby being present but gem being missing #843

Closed
tornewuff opened this Issue Jan 16, 2013 · 8 comments

Comments

Projects
None yet
2 participants

On Debian/Ubuntu, gem is packaged separately from ruby. If you have ruby installed but not gem, the advice ends up saying:

✔ Ruby check.
✖ RubyGems [not installed]
* You'll acquire RubyGems with your ruby installation.

with no suggestion on how to resolve this situation. It should suggest "sudo apt-get install rubygems".

Member

hemanth commented Jan 17, 2013

@tornewuff Could you please tell about how did you install?

"sudo apt-get install rubygems" was all I needed to be able to install.

Member

hemanth commented Jan 17, 2013

Yes, that is clear, but what I asked was, how did you install yeoman? If you read install.sh#L279 it's clear that rubygems must be bundled with ruby.

I installed yeoman itself with "sudo npm install -g yeoman", as the script says. Other than installing rubygems manually I just followed install.sh's instructions (ruby itself was already installed on this machine).

I'm not sure I follow what you mean. rubygems doesn't have to be bundled with ruby and I don't see anything in install.sh's actual logic to suggest it does (only in the messages it prints).

Member

hemanth commented Jan 17, 2013

@tornewuff Which version of ubuntu/debian? 1.9.3 must be bundled with gems.

I'm using Ubuntu 12.04 LTS (precise). The default version of Ruby is still 1.8 on precise, not 1.9, and the yeoman install script specifies Ruby 1.8.7 as a minimum, which is what precise provides.

Sorry if I wasn't clear; I wasn't aware that Ruby 1.9 now bundles gems as I haven't used Ruby myself in a while and all my machines still have 1.8.7.

Member

hemanth commented Jan 18, 2013

Ah ha, okies. Thanks for the update, will send a P.R soon on this.

@hemanth hemanth closed this Jan 18, 2013

Member

hemanth commented Jan 20, 2013

@alrra Sweet :)

@sindresorhus sindresorhus added a commit that referenced this issue Jan 20, 2013

@sindresorhus sindresorhus Merge pull request #854 from alrra/fix#843
Audit script: improve messages for `RubyGems` (fix #843)
6bcc19a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment