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 support for Ruby 1.9.3 #344

Merged
merged 3 commits into from Jan 9, 2016
Merged

Drop support for Ruby 1.9.3 #344

merged 3 commits into from Jan 9, 2016

Conversation

tfausak
Copy link
Collaborator

@tfausak tfausak commented Jan 8, 2016

Fixes #343.

@tfausak tfausak self-assigned this Jan 8, 2016
@tfausak tfausak added this to the v3.0.0 milestone Jan 8, 2016
@tfausak
Copy link
Collaborator Author

tfausak commented Jan 8, 2016

We never officially supported JRuby or Rubinius. Should we remove them from the Travis build matrix?

@AaronLasseigne
Copy link
Owner

I'd like to continue supporting JRuby. It's a reasonable and used alternative to MRI. Rubinius I could live without.

@tfausak
Copy link
Collaborator Author

tfausak commented Jan 8, 2016

Can you figure out how to get JRuby running in Ruby 2.0 mode on Travis then?

@AaronLasseigne
Copy link
Owner

Sure, I'll look into it.

@tfausak
Copy link
Collaborator Author

tfausak commented Jan 8, 2016

Might be as easy as `JRUBY_OPTS='--2.0'.

@tfausak
Copy link
Collaborator Author

tfausak commented Jan 8, 2016

Well that was unexpected:

ERROR:  While executing gem ... (ArgumentError)
    marshal data too short

https://travis-ci.org/orgsync/active_interaction/jobs/101112443

@tfausak tfausak mentioned this pull request Jan 8, 2016
@tfausak tfausak mentioned this pull request Jan 8, 2016
send("#{name}=", value)
end unless method_defined?(:[]=)
end

Copy link
Owner

Choose a reason for hiding this comment

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

I thought the plan was to avoid backward incompatible changes for now.

Copy link
Owner

Choose a reason for hiding this comment

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

As a note I'm fine with dropping 1.9.3 like a bad habit.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I planned on leaving this in. When I changed the gem spec to require Ruby >= 2, I realized that this served absolutely no purpose. If we wanted to continue to unofficially support Ruby 1.9.3, we'd need to keep Ruby >= 1.9.3 in the gem spec. And that basically means officially supporting Ruby 1.9.3, which defeats the whole point.

All that to say, I've changed my mind. I'm fine with dropping support for 1.9.3. If someone wants to use ActiveInteraction on 1.9.3, they need to use some version < 3.

Copy link
Owner

Choose a reason for hiding this comment

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

ok

@AaronLasseigne
Copy link
Owner

:shipit:

tfausak added a commit that referenced this pull request Jan 9, 2016
Drop support for Ruby 1.9.3
@tfausak tfausak merged commit fc6f402 into master Jan 9, 2016
@tfausak tfausak deleted the gh-343-ruby branch January 9, 2016 22:14
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.

None yet

2 participants