Skip to content

Conversation

@vstinner
Copy link
Member

@vstinner vstinner commented Sep 3, 2018

bpo-34544: If _Py_CoerceLegacyLocale() fails to coerce the C locale,
restore the LC_CTYPE locale to the its previous value.

https://bugs.python.org/issue34544

bpo-34544: If _Py_CoerceLegacyLocale() fails to coerce the C locale,
restore the LC_CTYPE locale to the its previous value.
@vstinner vstinner requested a review from ncoghlan September 3, 2018 14:51
@vstinner vstinner changed the title _Py_CoerceLegacyLocale() restores LC_CTYPE on fail bpo-34544: _Py_CoerceLegacyLocale() restores LC_CTYPE on fail Sep 3, 2018
@vstinner vstinner merged commit 8ea0911 into python:master Sep 3, 2018
@miss-islington
Copy link
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

@vstinner vstinner deleted the fix_coerce_locale branch September 3, 2018 15:05
@miss-islington
Copy link
Contributor

Sorry, @vstinner, I could not cleanly backport this to 3.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 8ea09110d413829f71d979d8c7073008cb87fb03 3.7

@bedevere-bot
Copy link

GH-9046 is a backport of this pull request to the 3.7 branch.

vstinner added a commit that referenced this pull request Sep 3, 2018
bpo-34544: If _Py_CoerceLegacyLocale() fails to coerce the C locale,
restore the LC_CTYPE locale to the its previous value.

(cherry picked from commit 8ea0911)
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.

4 participants