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

Fix unicode encoding by reflecting changes in mysqlclient-python #5

Closed
wants to merge 1 commit into from

Conversation

bastei
Copy link

@bastei bastei commented Jan 23, 2019

unicode_literal is not directly accessible in mysqlclient-python since 1.3.11. This PR fixes unicode_literal for Python2.7.

@dataflake dataflake self-requested a review January 23, 2019 15:58
Copy link
Member

@dataflake dataflake left a comment

Choose a reason for hiding this comment

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

This patch breaks Python 3 compatibility, where unicode is not available.

@bastei
Copy link
Author

bastei commented Jan 23, 2019

Sorry, I was a little bit too fast. It patches ZMySQLDA for Python2.7 but breaks Python3. I will repair that.

@bastei bastei force-pushed the unicode_literal_fix branch from 3af5b86 to d76abb9 Compare January 23, 2019 16:50
@bastei bastei force-pushed the unicode_literal_fix branch from d76abb9 to 014af95 Compare January 23, 2019 16:54
@dataflake
Copy link
Member

The master branch now contains a fix and tests for it. Could you test master and let me know if it fixes your issue? I'll create a new beta release if it works for you.

@bastei
Copy link
Author

bastei commented Jan 24, 2019

Yes, the fix in master is working for me. So we can consider this PR as obsolete. Thanks a lot!

@bastei bastei closed this Jan 24, 2019
@dataflake
Copy link
Member

Version 4.0b4 is now available on PyPI

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