Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

bpo-34523: Fix C locale coercion on FreeBSD CURRENT#10672

Merged
vstinner merged 2 commits into
python:masterpython/cpython:masterfrom
vstinner:reset_asciiCopy head branch name to clipboard
Nov 23, 2018
Merged

bpo-34523: Fix C locale coercion on FreeBSD CURRENT#10672
vstinner merged 2 commits into
python:masterpython/cpython:masterfrom
vstinner:reset_asciiCopy head branch name to clipboard

Conversation

@vstinner

@vstinner vstinner commented Nov 23, 2018

Copy link
Copy Markdown
Member

bpo-34523, bpo-35290: C locale coercion now resets the Python
internal "force ASCII" mode. This change fix the filesystem encoding
on FreeBSD CURRENT which has a new "C.UTF-8" locale.

  • Add _Py_ResetForceASCII()
  • _coerce_default_locale_settings() now calls _Py_ResetForceASCII()

https://bugs.python.org/issue34523

bpo-34523, bpo-35290: C locale coercion now resets the Python
internal "force ASCII" mode. This change fix the filesystem encoding
on FreeBSD CURRENT which has a new "C.UTF-8" locale.

Add _Py_ResetForceASCII(). _Py_SetLocaleFromEnv() now calls
_Py_ResetForceASCII().
On Windows, the function does nothing.
@vstinner vstinner merged commit 353933e into python:master Nov 23, 2018
@vstinner vstinner deleted the reset_ascii branch November 23, 2018 12:08
vstinner added a commit that referenced this pull request Nov 23, 2018
)

bpo-34523, bpo-35290: C locale coercion now resets the Python
internal "force ASCII" mode. This change fix the filesystem encoding
on FreeBSD CURRENT, which has a new "C.UTF-8" locale, when
the UTF-8 mode is disabled.

Add _Py_ResetForceASCII(): _Py_SetLocaleFromEnv() now calls it.

(cherry picked from commit 353933e)
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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.