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

bpo-28180: Standard stream & FS encoding differ on Mac OS X #2208

Merged
merged 2 commits into from
Jun 15, 2017

Conversation

ncoghlan
Copy link
Contributor

In the C locale on Mac OS X, the default filesystem encoding
used for operating system interfaces is UTF-8, but the
default encoding used on the standard streams is still ASCII.

In the C locale on Mac OS X, the default filesystem encoding
used for operating system interfaces is UTF-8, but the
default encoding used on the standard streams is still ASCII.
@ncoghlan
Copy link
Contributor Author

ncoghlan commented Jun 15, 2017

Custom build on the Mac OS X buildbot: http://buildbot.python.org/all/builders/x86%20Tiger%20custom/builds/6/

@ncoghlan ncoghlan changed the title bpo-28180: Standard stream & FS encoding differ on Mac OS X WIP: bpo-28180: Standard stream & FS encoding differ on Mac OS X Jun 15, 2017
@ncoghlan
Copy link
Contributor Author

Initial version still had some failures related to the POSIX locale (where Mac OS X isn't behaving the same way Linux systems do), so I amended the PR to skip those cases. Next custom test run: http://buildbot.python.org/all/builders/x86%20Tiger%20custom/builds/7

@ncoghlan ncoghlan changed the title WIP: bpo-28180: Standard stream & FS encoding differ on Mac OS X bpo-28180: Standard stream & FS encoding differ on Mac OS X Jun 15, 2017
@ncoghlan
Copy link
Contributor Author

Latest custom build was green, so going ahead & merging this :)

@ncoghlan ncoghlan merged commit 7926516 into python:master Jun 15, 2017
@ncoghlan ncoghlan deleted the bpo-28180-fix-mac-os-x-test-errors branch March 30, 2018 07:48
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