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 RVM step to build instructions #944

Closed
wants to merge 1 commit into from
Closed

Add RVM step to build instructions #944

wants to merge 1 commit into from

Conversation

@Sinjo
Copy link
Contributor

@Sinjo Sinjo commented Aug 3, 2013

Adds a reminder for RVM users to switch to system Ruby to the instructions in BUILDING.md

It was something silly that caught me out, so I figured it might be worth mentioning it (given how commonly used RVM is).

@mkristian
Copy link
Member

@mkristian mkristian commented Aug 3, 2013

may I ask what the effect was with rvm not using system ? command build/jruby should just work independent of the actual environment

@Sinjo
Copy link
Contributor Author

@Sinjo Sinjo commented Aug 3, 2013

When I ran (with an RVM managed Ruby in use):

bin/jruby -S rake test:mri19

I would get an error like:

rake aborted!
Could not find 'rspec' (>= 0) among 18 total gem(s)

I believe this was to blame:

sinjo@widdershins:~/projects/jruby$ bin/jruby -S gem environment
...
  - EXECUTABLE DIRECTORY: /home/sinjo/.rvm/gems/ruby-1.9.3-p429/bin

Same command with system Ruby selected:

  - EXECUTABLE DIRECTORY: /home/sinjo/projects/jruby/bin
@BanzaiMan
Copy link
Member

@BanzaiMan BanzaiMan commented Aug 4, 2013

Do we know if rbenv and/or chruby are affected? I am open to adding a more general warning than just rvm.

enebo added a commit that referenced this pull request Sep 17, 2013
@enebo
Copy link
Member

@enebo enebo commented Sep 17, 2013

I could not merge this for some reason and made some textual changes. Fixed in b633137.

@enebo enebo closed this Sep 17, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants