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

Update Travis settings for Rubinius. #79

Merged
merged 1 commit into from Feb 10, 2014
Merged

Update Travis settings for Rubinius. #79

merged 1 commit into from Feb 10, 2014

Conversation

@brixen
Copy link
Contributor

@brixen brixen commented Feb 10, 2014

No description provided.

@ghost
Copy link

@ghost ghost commented Feb 10, 2014

@brixen does rbx-2 make it unnecessary to include rubysl in your Gemfile?

@sferik
Copy link
Contributor

@sferik sferik commented Feb 10, 2014

@robgleeson Apparently so. The Rubinius tests for this branch are passing on Travis CI.

sferik added a commit that referenced this issue Feb 10, 2014
Update Travis settings for Rubinius.
@sferik sferik merged commit b9eb0bf into httprb:master Feb 10, 2014
1 check failed
@ghost
Copy link

@ghost ghost commented Feb 10, 2014

interesting. thanks!

@ghost
Copy link

@ghost ghost commented Feb 10, 2014

@sferik it looks like the http gem test helper doesn't initialize bundler through "bundler/setup" or use Bundler.require(…) to load Gemfile dependencies. maybe that's why it works? I saw a similar thing on a different repo a few days ago.
I looked @ https://github.com/tarcieri/http/blob/master/spec/spec_helper.rb

@brixen
Copy link
Contributor Author

@brixen brixen commented Feb 10, 2014

@robgleeson the 'rbx-2' is to limit testing on Rubinius 2.x since there will be other versions of Rubinius > 2.x soon. This part is unrelated to loading gems.

I've changed Rubinius code loading to avoid Bundler stripping pre-installed gems. I'll be making a patch to Bundler so it will specifically consider pre-installed gems but in any case, Bundler will no longer prevent eg rubysl-securerandom or whatever from being loaded. I'm writing a Migrating from MRI guide that will cover this.

bf4 added a commit to whitequark/parser that referenced this issue Feb 11, 2014
see http://www.benjaminfleischer.com/2013/12/05/testing-rubinius-on-travis-ci/ httprb/http#79 (comment)

The ‘rbx-2’ is to limit testing on Rubinius 2.x since there will be other versions of Rubinius > 2.x
@ghost
Copy link

@ghost ghost commented Feb 12, 2014

@brixen cool, thanks! look forward to the guide.

ms-ati pushed a commit to ms-ati/docile that referenced this issue Feb 21, 2014
Apparently, since rbx-2.2.5 you no longer need to require
gem rubysl or rubinius-coverage, and you must have travis
use the ruby named 'rbx-2'.

See
*  httprb/http#79
*  https://github.com/rubinius/rubinius/releases/tag/v2.2.5
wmoxam pushed a commit to wmoxam/phonie that referenced this issue Mar 2, 2014
arlimus added a commit to gittalk/git-issues that referenced this issue May 13, 2014
see: httprb/http#79
Signed-off-by: Dominik Richter <dominik.richter@gmail.com>
arlimus added a commit to gittalk/git-issues that referenced this issue May 13, 2014
see: httprb/http#79
Signed-off-by: Dominik Richter <dominik.richter@gmail.com>
arlimus added a commit to arlimus/excon that referenced this issue May 13, 2014
Recently TravisCI is experiencing issues with rbx-2.1.1 (bundler missing). Out of the possible fixes (including pre-installation of bundler) a recommended way is to specify rbx-2 in travis to set testing to Rubinius 2.x.

See also discussion here: httprb/http#79

Signed-off-by: Dominik Richter <dominik.richter@gmail.com>
arlimus added a commit to arlimus/excon that referenced this issue May 14, 2014
Recently TravisCI is experiencing issues with rbx-2.1.1 (bundler missing). Out of the possible fixes (including pre-installation of bundler) a recommended way is to specify rbx-2 in travis to set testing to Rubinius 2.x.

See also discussion here: httprb/http#79

Signed-off-by: Dominik Richter <dominik.richter@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants