Skip to content
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

Jump to bottom

[3.8] GH-102306 Avoid GHA CI macOS test_posix failure by using the appropriate macOS SDK #102307

Merged
merged 1 commit into from Mar 4, 2023
Merged

[3.8] GH-102306 Avoid GHA CI macOS test_posix failure by using the appropriate macOS SDK #102307

merged 1 commit into from Mar 4, 2023

Conversation

ned-deily
Copy link
Member

@ned-deily ned-deily commented Feb 27, 2023 •
edited by bedevere-bot

@ned-deily ned-deily self-assigned this Feb 27, 2023
@ned-deily ned-deily changed the title [3.8] Avoid GHA CI macOS test_posix failure by using the appropriate macOS SDK [3.8] GH-102306 Avoid GHA CI macOS test_posix failure by using the appropriate macOS SDK Feb 28, 2023
@bedevere-bot bedevere-bot mentioned this pull request Feb 28, 2023
Open
@ned-deily
Copy link
Member Author

As a proof-of-concept, just forcing use of the macOS 12 SDK does solve the problem. This is one possible solution, likely the simplest, as noted in the issue.

@ned-deily ned-deily assigned ambv and unassigned ned-deily Feb 28, 2023
corona10
corona10 approved these changes Mar 4, 2023
View changes
Copy link
Member

@corona10 corona10 left a comment

Choose a reason for hiding this comment

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

lgtm

@ned-deily ned-deily merged commit 4812813 into python:3.8 Mar 4, 2023
15 checks passed
@ned-deily ned-deily deleted the gh-102306-38 branch March 4, 2023 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@corona10 corona10 corona10 approved these changes

Assignees

@ambv ambv

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@ned-deily @corona10 @ambv @bedevere-bot