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

Bump the minimum version of mysql2 gem to 0.5.0 #33876

Closed
wants to merge 1 commit into from

Conversation

yahonda
Copy link
Member

@yahonda yahonda commented Sep 13, 2018

Summary

Rails 6 drops MySQL 5.1 support and requires MySQL 5.5.8 by #33853
mysql2 gem 0.5.0 also drops MySQL 5.1 support.

https://github.com/brianmario/mysql2/releases/tag/0.5.0

MySQL 5.5 or higher required. MySQL 5.0 and 5.1 are not supported.

Related to #31636 #32310 #33853

cc @sodabrew

Rails 6 drops MySQL 5.1 support and requires MySQL 5.5.8 by rails#33853
mysql2 gem 0.5.0 also drops MySQL 5.1 support.

https://github.com/brianmario/mysql2/releases/tag/0.5.0
> MySQL 5.5 or higher required. MySQL 5.0 and 5.1 are not supported.

Related to rails#31636 rails#32310 rails#33853
@rails-bot
Copy link

r? @georgeclaghorn

(@rails-bot has picked a reviewer for you, use r? to override)

@kamipo
Copy link
Member

kamipo commented Sep 13, 2018

Required MySQL 5.5.8 is for using utf8mb4 by default.
mysql2 0.4.4 still works well with utf8mb4.

@kamipo kamipo closed this Sep 13, 2018
@sodabrew
Copy link
Contributor

Sounds good thanks for the heads up!

@jeremy
Copy link
Member

jeremy commented Sep 13, 2018

I would like to drop the pessimistic < 0.6.0 constraint, in any case. These have only burned us and made upgrades needlessly difficult for app devs. I'd be happy to invert that problem 😅

yahonda added a commit to yahonda/rails that referenced this pull request Sep 14, 2018
@yahonda yahonda deleted the bump_mysql2_gem_requirement branch September 23, 2018 05:05
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

6 participants