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 README.md to match current implementation #34

Merged
merged 2 commits into from
Jun 25, 2016

Conversation

envp
Copy link
Member

@envp envp commented Feb 1, 2016

Partially fulfills #21

Updated table in README to match current implementation.

@envp envp mentioned this pull request Feb 1, 2016
2 tasks
@agarie
Copy link
Member

agarie commented Feb 1, 2016

Huh, apparently there is a problem with installing distribution in some test builds:

https://travis-ci.org/SciRuby/distribution/jobs/106278870

It worked in Ruby 2.0:

https://travis-ci.org/SciRuby/distribution/jobs/106278868

The same thing happened for PRs #33 (output here) and #34 (output here). Can you take a look?

@envp
Copy link
Member Author

envp commented Feb 2, 2016

The key differences between pass / fail I see are:

  1. Job #39.1 used gem version 2.4.5. Right now, 2.5.1 is the configured one for ruby193.
  2. Only ruby200 on Travis uses gem 2.4.6 for Job #54.2 (last one that passed) and the others use 2.5.1. Job 39.2 uses gem 2.4.5

I think that recent gem version changes is causing some trouble with rspec. I can't comment on my c9.io workspace yet but will check the gem version there too.

This looks like a travis configuration issue.

Maybe relevant?. The same error message appears in distribution's failed build logs too.

Update:
@agarie #37 fixes all of the above.

@envp envp self-assigned this Jun 24, 2016
@envp envp merged commit 8b994ce into SciRuby:master Jun 25, 2016
@envp envp deleted the update_readme_for_rng_21 branch June 25, 2016 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants