Skip to content

[3.7] bpo-34537: Fix test_gdb:test_strings with LC_ALL=C (GH-9483)#9485

Merged
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-7279b51-3.7
Sep 22, 2018
Merged

[3.7] bpo-34537: Fix test_gdb:test_strings with LC_ALL=C (GH-9483)#9485
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-7279b51-3.7

Conversation

@miss-islington

@miss-islington miss-islington commented Sep 22, 2018

Copy link
Copy Markdown
Contributor

We cannot simply call locale.getpreferredencoding() here,
as GDB might have been linked against a different version
of Python with a different encoding and coercion policy
with respect to PEP 538 and PEP 540.

Thanks to Victor Stinner for a hint on how to fix this.
(cherry picked from commit 7279b51)

Co-authored-by: Elvis Pranskevichus elvis@magic.io

https://bugs.python.org/issue34537

We cannot simply call locale.getpreferredencoding() here,
as GDB might have been linked against a different version
of Python with a different encoding and coercion policy
with respect to PEP 538 and PEP 540.

Thanks to Victor Stinner for a hint on how to fix this.
(cherry picked from commit 7279b51)

Co-authored-by: Elvis Pranskevichus <elvis@magic.io>

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Good bot.

@miss-islington

Copy link
Copy Markdown
Contributor Author

@elprans and @vstinner: Status check is done, and it's a success ✅ .

@miss-islington
miss-islington merged commit e5fde1f into python:3.7 Sep 22, 2018
@miss-islington
miss-islington deleted the backport-7279b51-3.7 branch September 22, 2018 01:29
@miss-islington

Copy link
Copy Markdown
Contributor Author

@elprans and @vstinner: Status check is done, and it's a success ✅ .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants