Skip to content

[3.7] bpo-34589: Add -X coerce_c_locale option; C locale coercion off by default #9379

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Sep 18, 2018
Merged

[3.7] bpo-34589: Add -X coerce_c_locale option; C locale coercion off by default #9379

merged 3 commits into from
Sep 18, 2018

Conversation

vstinner
Copy link
Member

@vstinner vstinner commented Sep 18, 2018 •
edited by bedevere-bot
Loading

@vstinner
bpo-34589: Make _PyCoreConfig.coerce_c_locale private (GH-9371)
f59e6b0 
_PyCoreConfig:

* Rename coerce_c_locale to _coerce_c_locale
* Rename coerce_c_locale_warn to _coerce_c_locale_warn

These fields are now private (name prefixed by "_").

(cherry picked from commit 188ebfa)
@vstinner
bpo-34589: C locale coercion off by default (GH-9073)
71f3810 
Py_Initialize() and Py_Main() cannot enable the C locale coercion
(PEP 538) anymore: it is always disabled. It can now only be enabled
by the Python program ("python3).

test_embed: get_filesystem_encoding() doesn't have to set PYTHONUTF8
nor PYTHONCOERCECLOCALE, these variables are already set in the
parent.

(cherry picked from commit 7a0791b)
@vstinner
bpo-34589: Add -X coerce_c_locale command line option (GH-9378)
1593547 
Add a new -X coerce_c_locale command line option to control C locale
coercion (PEP 538).

(cherry picked from commit dbdee00)
@vstinner vstinner merged commit 144f1e2 into python:3.7 Sep 18, 2018
@vstinner vstinner deleted the pep538_37 branch September 18, 2018 01:01
vstinner added a commit that referenced this pull request Sep 19, 2018
@vstinner
Revert "[3.7] bpo-34589: Add -X coerce_c_locale option; C locale coer…
95cc3ee 
…cion off by default (GH-9379)" (GH-9416)

This reverts commit 144f1e2.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vstinner @the-knights-who-say-ni @bedevere-bot