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

Ported PR #4856 to 3-2-stable. #7299

Closed
wants to merge 2 commits into from
Closed

Conversation

iHiD
Copy link
Contributor

@iHiD iHiD commented Aug 9, 2012

Hi guys,
#4856 was committed to master, but never backported to 3-2-stable. I only realised when I upgraded Rails to the latest 3-2 version and it broke my app. I'd been using a patched version locally. My fail.

The files that were updated on master haven't changed on 3-2-stable, so it should be very straightforward.

Thanks,
iHiD

@rafaelfranca
Copy link
Member

The backport is not easy. You will need to backport the Oracle fixes that @yahonda did.

I think that all the fixes are linked at #4856.

@iHiD
Copy link
Contributor Author

iHiD commented Aug 9, 2012

Thanks for the feedback, @rafaelfranca.

As far as I can tell, there's just one small change in #7173. I've backported #7173. Can you confirm this please and let me know if there's another commit that's not referenced that I've missed? Thanks.

rafaelfranca added a commit that referenced this pull request Aug 9, 2012
@rafaelfranca
Copy link
Member

Merged. Thanks

@iHiD
Copy link
Contributor Author

iHiD commented Aug 9, 2012

Thanks, @rafaelfranca.

@yahonda
Copy link
Member

yahonda commented Aug 9, 2012

This fix also works with Oracle. Thanks for taking care of it.

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

3 participants