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

Change required activesupport version to >=4.0.2 #266

Merged
merged 1 commit into from
Sep 23, 2015
Merged

Change required activesupport version to >=4.0.2 #266

merged 1 commit into from
Sep 23, 2015

Conversation

kattrali
Copy link
Contributor

When a user only has activesupport 3.2.15 installed, invoking plugin in a Podfile calls Hash.deep_stringify_keys causing a crash. deep_stringify_keys was first added to activesupport in version 4.0.2, but the base version required by core is 3.2.15. As noted in the comments on CocoaPods/CocoaPods#2307, the base activesupport version was held back to continue support for versions of Ruby before 2.0.0.

As of 76845c8, Cocoapods no longer supports Ruby versions less than 2.0, invalidating the reasoning for holding back the base version number.

Invoking `plugin` in a Podfile calls `Hash.deep_stringify_keys`
(http://apidock.com/rails/v4.0.2/Hash/deep_stringify_keys), which was
first added to activesupport in version 4.0.2. As noted in 
Cocoapods/Core#3076, the base activesupport version was held back to
continue support for versions of Ruby before 2.0.0.

As of 76845c8, Cocoapods no longer supports Ruby versions less than 2.0,
invalidating the reasoning for holding back the base version number.
kylef added a commit that referenced this pull request Sep 23, 2015
…pendency

Change required activesupport version to >=4.0.2
@kylef kylef merged commit f06004c into CocoaPods:master Sep 23, 2015
@kylef
Copy link
Contributor

kylef commented Sep 23, 2015

Thanks @kattrali 🎈.

@kattrali kattrali deleted the update-activesupport-runtime-dependency branch September 23, 2015 23:50
Ashton-W pushed a commit to Ashton-W/Core that referenced this pull request Nov 2, 2015
…untime-dependency

Change required activesupport version to >=4.0.2
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

2 participants