bpo-41700: Skip test if the locale is not supported by corona10 · Pull Request #22081 · python/cpython · GitHub
Skip to content

bpo-41700: Skip test if the locale is not supported - #22081

Merged
corona10 merged 1 commit into
python:masterfrom
corona10:bpo-41700
Sep 4, 2020
Merged

corona10 merged 1 commit into
python:masterfrom
corona10:bpo-41700

Conversation

@corona10

@corona10 corona10 commented Sep 3, 2020

Copy link
Copy Markdown
Member

@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.

Your PR doesn't fix the issue, see https://bugs.python.org/issue41700#msg376320 to reproduce the issue.

Comment thread Lib/test/test_c_locale_coercion.py Outdated

@corona10 corona10 left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@vstinner
Thanks, Victor
I fixed the wrong point
I 've updated the PR and it works well on the reproducing scenario

@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. I tested manually that the PR fix https://bugs.python.org/issue41700.

It should be backport to 3.9, and maybe also to 3.8, no?

@corona10

corona10 commented Sep 4, 2020

Copy link
Copy Markdown
Member Author

@corona10
corona10 merged commit 54a66ad into python:master Sep 4, 2020
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @corona10 for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8, 3.9.
🐍🍒⛏🤖

@bedevere-bot

Copy link
Copy Markdown

GH-22085 is a backport of this pull request to the 3.9 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 4, 2020
(cherry picked from commit 54a66ad)

Co-authored-by: Dong-hee Na <donghee.na92@gmail.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 4, 2020
(cherry picked from commit 54a66ad)

Co-authored-by: Dong-hee Na <donghee.na92@gmail.com>
@bedevere-bot

Copy link
Copy Markdown

GH-22086 is a backport of this pull request to the 3.8 branch.

corona10 added a commit that referenced this pull request Sep 4, 2020
)

(cherry picked from commit 54a66ad)

Co-authored-by: Dong-hee Na <donghee.na92@gmail.com>

Co-authored-by: Dong-hee Na <donghee.na92@gmail.com>
corona10 added a commit that referenced this pull request Sep 4, 2020
)

(cherry picked from commit 54a66ad)

Co-authored-by: Dong-hee Na <donghee.na92@gmail.com>

Co-authored-by: Dong-hee Na <donghee.na92@gmail.com>
@bedevere-bot

Copy link
Copy Markdown

shihai1991 added a commit to shihai1991/cpython that referenced this pull request Sep 5, 2020
…lots1

* origin/master: (63 commits)
  bpo-41627: Distinguish 32 and 64-bit user site packages on Windows (pythonGH-22098)
  bpo-38585: Remove references to defusedexpat (pythonGH-22095)
  bpo-41721: Add xlc options (pythonGH-22096)
  bpo-40486: Specify what happens if directory content change diring iteration (pythonGH-22025)
  bpo-41638: Improve ProgrammingError message for absent parameter. (pythonGH-21999)
  bpo-41713: _signal doesn't use multi-phase init (pythonGH-22087)
  bpo-41700: Skip test if the locale is not supported (pythonGH-22081)
  [doc] Update documentation on logging optimization. (pythonGH-22075)
  Fix 'gather' rules in the python parser generator (pythonGH-22021)
  bpo-41697: Correctly handle KeywordOrStarred when parsing arguments in the parser (pythonGH-22077)
  [doc] Fix a typo in the graphlib docs (python#22030)
  bpo-1635741: Port _signal module to multi-phase init (PEP 489) (pythonGH-22049)
  bpo-39883: Use BSD0 license for code in docs (pythonGH-17635)
  bpo-39010: Improve test shutdown (python#22066)
  bpo-41696: Fix handling of debug mode in asyncio.run (python#22069)
  bpo-41690: Use a loop to collect args in the parser instead of recursion (pythonGH-22053)
  closes bpo-41689: Preserve text signature from tp_doc in C heap type creation. (pythonGH-22058)
  Fix invalid escape sequences in the peg_highlight Sphinx extension (pythonGH-22047)
  bpo-41675: Modernize siginterrupt calls (pythonGH-22028)
  bpo-41685: Don't pin setuptools version anymore in Doc/Makefile (pythonGH-22062)
  ...
xzy3 pushed a commit to xzy3/cpython that referenced this pull request Oct 18, 2020
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.

5 participants