Skip to content

Conversation

owenv
Copy link
Contributor

@owenv owenv commented Sep 22, 2025

Explanation: SwiftPM's native build system does not have the concept of embedding the compatibility libraries for distribution, so add rpaths to the span back deploy library in the toolchain when targeting macOS < 26.


Original:
SwiftPM's native build system does not have the concept of embedding the compatibility libraries for distribution, so add rpaths to the span back deploy library in the toolchain when targeting macOS < 26. Ensure we always insert /usr/lib/swift ahead of the added rpaths to prefer the OS content when available. this mirrors the handling of the concurrency compat lib

Closes #9163

…ftlang#9164)

SwiftPM's native build system does not have the concept of embedding the
compatibility libraries for distribution, so add rpaths to the span back
deploy library in the toolchain when targeting macOS < 26. Ensure we
always insert /usr/lib/swift ahead of the added rpaths to prefer the OS
content when available. this mirrors the handling of the concurrency
compat lib

Closes swiftlang#9163
@owenv owenv requested a review from a team as a code owner September 22, 2025 16:15
@owenv
Copy link
Contributor Author

owenv commented Sep 22, 2025

@swift-ci test

@owenv
Copy link
Contributor Author

owenv commented Sep 22, 2025

@swift-ci test self hosted

Copy link
Member

@DougGregor DougGregor left a comment

Choose a reason for hiding this comment

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

Looks good, thank you!

@owenv
Copy link
Contributor Author

owenv commented Sep 22, 2025

@swift-ci test Windows

@owenv
Copy link
Contributor Author

owenv commented Sep 23, 2025

swiftlang/swift#84445
@swift-ci test Windows

1 similar comment
@owenv
Copy link
Contributor Author

owenv commented Sep 23, 2025

swiftlang/swift#84445
@swift-ci test Windows

@owenv
Copy link
Contributor Author

owenv commented Sep 23, 2025

blocked by unexpected passes in lldb tests

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