Skip to content

Conversation

@al45tair
Copy link
Contributor

@al45tair al45tair commented Jun 9, 2023

Older glibc doesn't include these. Nor does Musl, it seems. The previous fix put the gettid() declaration too far down the file.

rdar://110417355

Older glibc doesn't include these.  Nor does Musl, it seems.  The
previous fix put the gettid() declaration too far down the file.

rdar://110417355
@al45tair al45tair requested a review from a team as a code owner June 9, 2023 08:09
@al45tair
Copy link
Contributor Author

al45tair commented Jun 9, 2023

Explanation: The previous fix was right but in the wrong place in the file so we still had the problem. Also, @MaxDesiatov pointed out a problem with Musl which will probably also affect older glibc, so fix that at the same time.
Risk: Low. The code change only affects Linux.
Original PR: #66486
Reviewed by: @MaxDesiatov
Resolves: rdar://110417355
Tests: This fixes a Linux nightly build failure on platforms with older glibc.

@al45tair
Copy link
Contributor Author

al45tair commented Jun 9, 2023

@swift-ci Please test

@al45tair al45tair added 🍒 release cherry pick Flag: Release branch cherry picks swift 5.9 labels Jun 9, 2023
@MaxDesiatov
Copy link
Contributor

@swift-ci test windows

@al45tair
Copy link
Contributor Author

al45tair commented Jun 9, 2023

@swift-ci Please test Windows platform

@tshortli tshortli merged commit 66a38a1 into swiftlang:release/5.9 Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🍒 release cherry pick Flag: Release branch cherry picks swift 5.9

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants