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 PyPy support #401

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

Add PyPy support #401

wants to merge 8 commits into from

Conversation

James-E-A
Copy link

The Windows CMD still uses Codepage 437 for regular I/O by default.

However, PyPy currently tries to treat it as UTF-8.

This pull request should fix that, making the just_fix_windows_console function even better.

Is this something you'd be interested in?

- This would PREVENT conflicts that would otherwise arise if PyPy
  copies PEP 528 per se.

- This would also NOT conflict if PyPy copies "our" solution
  instead of PEP 528.

- This COULD cause conflicts if PyPy comes up with its own,
  completely independent fix, if that fix keeps console I/O as a
  subclass of `io.FileIO`.

- In any case, a guard like

      and not (platform.python_implementation() == 'PyPy' and sys.pypy_version_info >= (8,0))

  should be appended to the existing CPython version check when the
  time comes.
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.

1 participant