Skip to content

[Swift 5.6 compat] use compat threading library #61007

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
merged 1 commit into from
Sep 9, 2022

Conversation

etcwilde
Copy link
Member

@etcwilde etcwilde commented Sep 8, 2022

Use the compatibility threading library instead of linking against the normal threading library. The normal threading library target doesn't exist yet.

rdar://99646957

@etcwilde
Copy link
Member Author

etcwilde commented Sep 8, 2022

@swift-ci please test

Copy link
Contributor

@al45tair al45tair left a comment

Choose a reason for hiding this comment

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

Not sure why it wasn’t already using the compatibility version.

@etcwilde
Copy link
Member Author

etcwilde commented Sep 9, 2022

Not sure why it wasn’t already using the compatibility version.

I think it's because you landed those changes between when I wrote that part of the 5.6 compat libs and when landed the PR.

@etcwilde
Copy link
Member Author

etcwilde commented Sep 9, 2022

The windows failure is due to the missing link dependency between swiftAST and swiftBasic added in #61001.
Merging.

@etcwilde etcwilde merged commit 6b85bdf into swiftlang:main Sep 9, 2022
@etcwilde etcwilde deleted the ewilde/fix-threadinglib branch September 12, 2022 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants