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

Add Unicode support in Windows and Python 2.7 #200

Merged
merged 1 commit into from
Apr 20, 2021

Conversation

sevein
Copy link
Contributor

@sevein sevein commented Apr 12, 2021

This pull request enables Unicode support when running Python 2.7 from the Windows console. It relies on the win-unicode-console package which monkeypatches the standard library when platform.system() == 'Windows' and six.PY2. It can be reverted once fido abandons Python 2.7 (EOL reached) since 3.6+ is perfectly capable.

Fixes #100.

image

Use win-unicode-console to install various fixes needed in Python 2 on
Windows: display of Unicode characters in Windows console, use of unicode
in sys.argv, etc...

See https://pypi.org/project/win_unicode_console/ for more details.
@sevein sevein force-pushed the dev/issue-100-unicode-windows branch from 6a83a69 to e06592f Compare April 12, 2021 15:22
Copy link
Contributor

@mistydemeo mistydemeo left a comment

Choose a reason for hiding this comment

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

Nice. I didn't realize it was possible to express platform_system-specific deps in setup.py.

@sevein
Copy link
Contributor Author

sevein commented Apr 13, 2021

@mistydemeo, thanks for looking at this. That marker is also new to me! This seems to be the full list: https://www.python.org/dev/peps/pep-0508/#environment-markers.
I wasn't sure about win-unicode-console, but it seems widely used in github.com/search, e.g. https://github.com/jonathanj/eliottree, which installs the dependency using the same marker.

@sevein sevein merged commit 9b9370e into openpreserve:rc/1.6 Apr 20, 2021
@sevein sevein deleted the dev/issue-100-unicode-windows branch April 20, 2021 14:14
@carlwilson carlwilson mentioned this pull request Jun 15, 2022
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