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

[FreeBSD] Fix FreeBSD build #5146

Merged
merged 3 commits into from
Dec 17, 2024
Merged

Conversation

michael-yuji
Copy link
Contributor

Various fixes to enable building on FreeBSD.

Fixed an issue that warning messages from libthr can be emitted to stderr when thread specific data is used. This is due to CoreFoundation setting the TSD to CF_TSD_BAD_PTR even after PTHERAD_DESTRUCTOR_ITERATIONS iterators of dtor calls. This causes libthr to emit a warning to stderr about left-over non-null TSD after max destructor iterations.

Unverified

The signing certificate or its chain could not be verified.
Various fixes to enable building on FreeBSD.

Fixed an issue that warning messages from libthr can be emitted to stderr when
thread specific data is used. This is due to CoreFoundation setting the TSD to
`CF_TSD_BAD_PTR` even after `PTHERAD_DESTRUCTOR_ITERATIONS` iterators of dtor
calls. This causes libthr to emit a warning to stderr about left-over non-null
TSD after max destructor iterations.
@compnerd
Copy link
Member

compnerd commented Dec 9, 2024

@swift-ci please test

Unverified

The signing certificate or its chain could not be verified.
@michael-yuji
Copy link
Contributor Author

@swift-ci please test

1 similar comment
@compnerd
Copy link
Member

compnerd commented Dec 9, 2024

@swift-ci please test

@michael-yuji
Copy link
Contributor Author

@swift-ci please test Windows platform

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@michael-yuji
Copy link
Contributor Author

@swift-ci please test

@parkera
Copy link
Contributor

parkera commented Dec 16, 2024

Are you ready to merge this?

@michael-yuji
Copy link
Contributor Author

Are you ready to merge this?

Yes, this pull request is ready to be merged. However, I lack write access to the repository.

@finagolfin finagolfin merged commit 238130a into swiftlang:main Dec 17, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants