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 0.10 stable build #1847

Merged

Conversation

mehlah
Copy link
Member

@mehlah mehlah commented Jan 27, 2016

This PR should fix the build for 0.10-stable branch.
It's a first step I hope towards releasing an intermediate version with some backports, before the 1.0 release ✌️.
@thomasfedb I'd be happy to discuss and share some thoughts on the release plan you guys wants to go with.

Fix #1822

Mehdi Lahmam added 5 commits January 27, 2016 04:13
@thomasfedb
Copy link
Contributor

Appears to be some issue installing i18n with 3.2 on CI.

@mehlah
Copy link
Member Author

mehlah commented Jan 27, 2016

@thomasfedb the build fails for older Rubies (1.8 and 1.9.2) due to the gems dependencie https://travis-ci.org/carrierwaveuploader/carrierwave/jobs/105075508#L189
I guess we could fix that with something like gem 'i18n', '~> 0.6.0', :platforms => [:mri_18, :mri_19] but I'm not sure if we want to support Ruby < 1.9.3.
Let me know what do you think.

@thomasfedb
Copy link
Contributor

Given that Rails 3.2 supports Ruby 1.8.7, and also that we're fixing this retrospectively, I feel that we should continue to test 0.10.x against 1.8.7, although we could drop support in 0.11.x if that is released.

@thomasfedb
Copy link
Contributor

Given that activesupport should have a dependency on an appropriate version of i18n I'm curious how this issue arises...

For example, in 3.2.22.1, the version of i18n depended on is >= 0.6.4, ~> 0.6.

@thomasfedb thomasfedb mentioned this pull request Jan 28, 2016
s.add_development_dependency "fog", ">= 1.3.1"
s.add_development_dependency "fog", "~> 1.20.0"
s.add_development_dependency "unf"
s.add_development_dependency "net-ssh", "~> 2.9.0"
Copy link

Choose a reason for hiding this comment

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

To fix test for ruby 1.8.7, we should specify some gem versions:

s.add_development_dependency "mini_magick", "< 4"
s.add_development_dependency "i18n", "< 0.7"
s.add_development_dependency "mime-types", "< 2"
s.add_development_dependency "sprockets", "< 3"
s.add_development_dependency "net-ssh", "< 2.10"
s.add_development_dependency "rack-cache", "< 1.3"

Mehdi Lahmam added 5 commits February 1, 2016 00:16
Specs fails for Rails 4.1.0 but not Rails 4.1.0.rc1
It seems to be a Rails issue fixed at
rails/rails@595f7b0

This commit fixes it by requiring Rails when necessary to ensure it's
loaded and `Rails.env` is defined.
With Container based CI, public projects can cache the bundle.
This make the build 🚤
https://docs.travis-ci.com/user/caching/#Enabling-Bundler-caching
@mehlah
Copy link
Member Author

mehlah commented Jan 31, 2016

Annd it's 💚 ! Thanks @mtsmfm for your hints.

@mtsmfm
Copy link

mtsmfm commented Jan 31, 2016

🎉

@thomasfedb
Copy link
Contributor

Looks great!

thomasfedb added a commit that referenced this pull request Feb 1, 2016
Fix dependency and config issues in CI build
@thomasfedb thomasfedb merged commit d73990b into carrierwaveuploader:0.10-stable Feb 1, 2016
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.

3 participants