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

Fix bundler installation shell command #41

Merged
merged 1 commit into from
Apr 27, 2015
Merged

Fix bundler installation shell command #41

merged 1 commit into from
Apr 27, 2015

Conversation

svyatov
Copy link
Contributor

@svyatov svyatov commented Apr 27, 2015

We should check gem list for specified ruby, not a default system's one.

nickjj added a commit that referenced this pull request Apr 27, 2015
Fix bundler installation shell command
@nickjj nickjj merged commit 26bf7f9 into rvm:master Apr 27, 2015
@nickjj
Copy link
Contributor

nickjj commented Apr 27, 2015

Good catch, it's been released into v1.3.7 which is now on the Galaxy.

@nickjj nickjj mentioned this pull request Apr 27, 2015
gildegoma added a commit to dispansible/rvm1-ansible-role that referenced this pull request Jun 30, 2015
Without this change, the Bundler installation introduced by rvm#40 and rvm#41
fails when the `rvm_rubies` list contains valid RVM aliases, like
following examples:

 - ruby-1.9.3 (→ ruby-1.9.3-p551)
 - ruby-2.0   (→ ruby-2.0.0-p643)
 - 2.1        (→ ruby-2.1.5)
 - ruby       (→ ruby-2.2.0)
 - jruby      (→ jruby-1.7.18)
 - etc.

Add more test coverage to `.travis.yml`.
gildegoma added a commit to dispansible/rvm1-ansible-role that referenced this pull request Jun 30, 2015
Without this change, the Bundler installation introduced by rvm#40 and rvm#41
fails when the `rvm_rubies` list contains valid RVM aliases, like
following examples:

 - ruby-1.9.3 (→ ruby-1.9.3-p551)
 - ruby-2.0   (→ ruby-2.0.0-p643)
 - 2.1        (→ ruby-2.1.5)
 - ruby       (→ ruby-2.2.0)
 - jruby      (→ jruby-1.7.18)
 - etc.

Add more test coverage to `.travis.yml`.
@jogaco jogaco mentioned this pull request Dec 6, 2016
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