Skip to content

Updated MRI and JRuby Travis build targets #218

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

Merged
merged 2 commits into from
Jan 1, 2015

Conversation

jdantonio
Copy link
Member

With the release of MRI 2.2.0 and the impending release of JRuby 9000 it is time to update our Travis targets:

  • Added MRI 2.2.0
  • Updated MRI 2.1.2 to 2.1.4
  • Removed jruby-19mode (TravisCI known issue)
  • Added jruby-1.7.17 in place of the broken jruby-19mode target

@jdantonio jdantonio changed the title Updated MRI and JRuby versions. Updated MRI and JRuby Travis build targets Jan 1, 2015
@jdantonio jdantonio mentioned this pull request Jan 1, 2015
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 74bc9d4 on updating-rubies-on-travis into 8ef62bb on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+7.16%) when pulling bd100f8 on updating-rubies-on-travis into 8ef62bb on master.

jdantonio added a commit that referenced this pull request Jan 1, 2015
@jdantonio jdantonio merged commit 6e8cf5f into master Jan 1, 2015
@jdantonio jdantonio deleted the updating-rubies-on-travis branch January 1, 2015 18:40
- 2.0.0
- 1.9.3
- ruby-head
- jruby-19mode
- jruby-1.7.17
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why not 1.7.18 ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because 1.7.18 is broken on Travis, too. It uses the fuzzy parameter and results in grabbing JRuby 9000. Specifying 1.7.17 is a recommended workaround.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah :/ thanks for explaining

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now you know why there are two commits! :-)

My first commit set 1.7.18. I reasoning was the same as yours. Once that test run failed again I did some more Googling. That's when I found the 1.7.17 workaround. It's a very unintuitive issue.

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