Skip to content

gh-76785: Fix test_threading on Non-Subinterpreter Builds #113014

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.

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

Merged

Conversation

ericsnowcurrently
Copy link
Member

@ericsnowcurrently ericsnowcurrently commented Dec 12, 2023

gh-112982 broke test_threading on one of the s390 buildbots (Fedora Clang Installed). Apparently ImportError is raised (rather than ModuleNotFoundError) for the name part of "from" imports. This fixes that by catching ImportError in test_threading.py.

@bedevere-app bedevere-app bot added tests Tests in the Lib/test dir awaiting core review labels Dec 12, 2023
@ericsnowcurrently ericsnowcurrently enabled auto-merge (squash) December 12, 2023 16:01
@ericsnowcurrently ericsnowcurrently merged commit cde1417 into python:main Dec 12, 2023
@ericsnowcurrently ericsnowcurrently deleted the fix-test-threading-s390 branch December 12, 2023 17:03
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
…ongh-113014)

pythongh-112982 broke test_threading on one of the s390 buildbots (Fedora Clang Installed).  Apparently ImportError is raised (rather than ModuleNotFoundError) for the name part of "from" imports.  This fixes that by catching ImportError in test_threading.py.
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
…ongh-113014)

pythongh-112982 broke test_threading on one of the s390 buildbots (Fedora Clang Installed).  Apparently ImportError is raised (rather than ModuleNotFoundError) for the name part of "from" imports.  This fixes that by catching ImportError in test_threading.py.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant