Fix the test that broke after 3.12 changes to locale.normalize
#452
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- On Linux, C.UTF-8 locale is now part of the standard. macOS and Windows use en_US.UTF-8 instead. - Up to 3.11, `locale.normalize` was converting `C.UTF-8` into `en_US.UTF-8`. If you normalized the locale string, it was impossible to set `C.UTF-8`. - In 3.12, `C.UTF-8` maps to itself. - python/cpython#14925 - https://peps.python.org/pep-0538/ - `C.UTF-8` doesn't seem to be used anywhere else in the codebase, except for this one test. - Other fix would be to inject a previous value to `locale.locale_alias` dictionary for certain platforms. However, this is part of the private implementation and could change without prior notice. - More changes to the locale are scheduled for 3.15.
kkalinowski-reef
added a commit
to reef-technologies/b2-sdk-python
that referenced
this pull request
Dec 6, 2023
* master: Fix the test that broke after 3.12 changes to `locale.normalize` (Backblaze#452) fix changelog items prefix use proper exception class fix: provide error message for mismatching upload size options
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
locale.normalizewas convertingC.UTF-8intoen_US.UTF-8. If you normalized the locale string, it was impossible to setC.UTF-8;C.UTF-8maps to itself;localeworks on duct tape for some time now) ;C.UTF-8doesn't seem to be used anywhere else in the codebase, except for this one test;locale.locale_aliasdictionary for certain platforms. However, this is part of the private implementation and could change without prior notice;