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

Drop 1.9.3 support #913

Merged
merged 1 commit into from
Nov 30, 2017
Merged

Drop 1.9.3 support #913

merged 1 commit into from
Nov 30, 2017

Conversation

tamird
Copy link
Contributor

@tamird tamird commented Nov 29, 2017

@sodabrew I might've missed something, but doesn't look like there's a huge upshot to doing this (at least not until we're ready to drop 2.0.0).

@@ -1,6 +1,8 @@
#ifndef MYSQL2_CLIENT_H
#define MYSQL2_CLIENT_H

// rb_thread_call_without_gvl was added in 2.0.0. See:
// https://github.com/ruby/ruby/commit/c51a826.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could be removed? Or is it needed for RBX?

@sodabrew
Copy link
Collaborator

Can remove if Thread.respond_to?(:handle_interrupt) blocks in client.rb and statement.rb, too.

@tamird
Copy link
Contributor Author

tamird commented Nov 29, 2017 via email

@sodabrew
Copy link
Collaborator

sodabrew commented Nov 29, 2017

This matches up with all the changes I was testing locally, thank you!

Even RHSCL Ruby 1.9.3 ended over a year ago: https://access.redhat.com/support/policy/updates/rhscl/ (search for ruby)

Unfortunately Ubuntu Trusty with its ridiculous Ruby 1.9.3 default is supported as a distribution until 2019, oh well.

@tamird
Copy link
Contributor Author

tamird commented Nov 29, 2017

Yikes :(

@sodabrew
Copy link
Collaborator

I noticed a few more if... handle interrupts? in the specs.

@sodabrew sodabrew added this to the 0.5.0 milestone Nov 29, 2017
@tamird
Copy link
Contributor Author

tamird commented Nov 29, 2017

Just saw one, done.

@sodabrew sodabrew merged commit 5fa7700 into brianmario:master Nov 30, 2017
@tamird tamird deleted the remove-1.9-support branch November 30, 2017 01:34
@tamird
Copy link
Contributor Author

tamird commented Nov 30, 2017

Interesting that RHSCL also dropped support for 2.0 at the same time they dropped 1.9. Shall we keep this train rolling?

@sodabrew
Copy link
Collaborator

Just a quick inventory suggests that only Ubuntu Trusty and Debian Wheezy are holding out at 1.9.3/2.0.0. Wheezy EOL is May 2018, and Trusty EOL is April 2019 (ugh, as mentioned, but also by that time will be two full LTS releases behind, so I'm less worried).

Older Mac OS X is at Ruby 2.0, Debian Jessie is on Ruby 2.1, and meanwhile everyone else has congregated at Ruby 2.3 now: Debian Squeeze and Buster, Ubuntu Xenial and (probably) Bionic, OS X High Sierra, and RHSCL 7.

jeremy added a commit to jeremy/mysql2 that referenced this pull request Dec 11, 2017
* master:
  GitHub is HTTPS by default (brianmario#922)
  Fix wrong value of type YEAR on big endian environment. (brianmario#921)
  Avoid 0 byte allocation call when statement takes no parameters
  Small spec improvement for RSpec style
  Travis CI for Mac OS X builds use the same steps again
  More specific exception classes (brianmario#911)
  Travis CI matrix add Ruby 2.5 and switch Mac OS X to Ruby 2.3 (ala High Sierra)
  README text for query options on Statement#execute
  Accept query options on Statement#execute (brianmario#912)
  Drop 1.9.3 support (brianmario#913)
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.

2 participants