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 Travis mime-types dependency #40

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gerrywastaken
Copy link

@gerrywastaken gerrywastaken commented May 6, 2016

Travis throws a gem install error about mime-types requiring Ruby
2.0, unless we pre install a version that still supports Ruby 1.9.


Hullo

By chance I was fixing this on another project and noticed you guys had pretty much the same issue. I attempted doing a pre-install using a .travis.yml, as other have suggested. However that didn't work. I tested running bundle install against Ruby 1.9.3p551 on my machine and did not encounter this error, so this work around is only in place for Travis.

Report by another user that the issue is still around despite being closed:
travis-ci/travis-ci#5145 (comment)

As a test I rebased #39 (which is waiting on this CI fix) and all came up green: https://travis-ci.org/gerrywastaken/wbench/builds/128276308

Travis throws a gem install error about mime-types requiring Ruby
2.0, unless we pre install a version that still supports Ruby 1.9.
@gerrywastaken
Copy link
Author

gerrywastaken commented May 6, 2016

Ahhh, geez, that didn't fail when I ran it on my fork: https://travis-ci.org/gerrywastaken/wbench/builds/128272785

However I also fixed this issue recently, so I'll add another commit to this PR to (hopefully) address this failure too.

Before this fix Travis reports the following error during the gem
install stage.

    NoMethodError: undefined method `spec' for nil:NilClass

Fix found here:
travis-ci/travis-ci#5239 (comment)
@gerrywastaken
Copy link
Author

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.

None yet

1 participant