Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

gh-117398: Use the correct module loader for iOS in datetime CAPI test. #120477

Merged
merged 1 commit into from
Jun 15, 2024

Conversation

freakboy3742
Copy link
Contributor

@freakboy3742 freakboy3742 commented Jun 14, 2024

#119604 added a CAPI test for subinterpreters in the datetime tests; however, the test uses the ExtensionFileLoader to load _testcapi. iOS requires the use of the AppleFrameworkLoader.

The buildbot validation will fail because of #120476; however, with this PR, test_datetime no longer fails.

@freakboy3742
Copy link
Contributor Author

!buildbot iOS

@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @freakboy3742 for commit 03293ac 🤖

The command will test the builders whose names match following regular expression: iOS

The builders matched are:

  • iOS ARM64 Simulator PR

Copy link
Member

@ericsnowcurrently ericsnowcurrently left a comment

Choose a reason for hiding this comment

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

LGTM

@ericsnowcurrently
Copy link
Member

Thanks for fixing this. Sorry I didn't catch it yesterday! FWIW, I like your solution taking advantage of the f-string. 😄

@freakboy3742 freakboy3742 merged commit 5c58e72 into python:main Jun 15, 2024
37 of 38 checks passed
@freakboy3742 freakboy3742 deleted the datetime-capitest branch June 15, 2024 00:05
mrahtz pushed a commit to mrahtz/cpython that referenced this pull request Jun 30, 2024
@bedevere-app
Copy link

bedevere-app bot commented Jul 10, 2024

GH-121561 is a backport of this pull request to the 3.13 branch.

freakboy3742 pushed a commit that referenced this pull request Jul 10, 2024
…API test (GH-120477) (#121561)

Use the correct binary module loader for iOS.
noahbkim pushed a commit to hudson-trading/cpython that referenced this pull request Jul 11, 2024
estyxx pushed a commit to estyxx/cpython that referenced this pull request Jul 17, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
OS-ios skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants