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

[6.0] CMake: add option to link Swift content from Darwin toolchain #73283

Conversation

edymtt
Copy link
Contributor

@edymtt edymtt commented Apr 26, 2024

Explanation: introduce a CMake option that allow Swift binaries to link against content in Darwin toolchains -- this is needed in some Apple internal configurations
Radar: rdar://127014753
Scope: CMake build logic that configure the build for Swift binaries on Darwin platforms
Risk: low -- this behaviour is opt in
Testing: CI testing -- in particular ensured we can build a toolchain

Reviewed By: @compnerd and @al45tair in #73255

This is needed in specific Apple internal configurations -- as a result
of the limited applicability, this option is not exposed through
`build-script` on purpose.

Addresses rdar://127014753

(cherry picked from commit 5f545b4)
@edymtt edymtt added 🍒 release cherry pick Flag: Release branch cherry picks swift 6.0 labels Apr 26, 2024
@edymtt edymtt requested a review from a team as a code owner April 26, 2024 14:57
@edymtt
Copy link
Contributor Author

edymtt commented Apr 26, 2024

@swift-ci please test

@edymtt
Copy link
Contributor Author

edymtt commented Apr 26, 2024

@swift-ci please build toolchain

@edymtt
Copy link
Contributor Author

edymtt commented Apr 26, 2024

@swift-ci please test WebAssembly

@edymtt
Copy link
Contributor Author

edymtt commented Apr 26, 2024

This will need #73303 to be picked as well before merging.

Additional testing showed that the path I initially choose does not work
in all scenarios.

Expands on apple#73255
Still addresses rdar://127014753

(cherry picked from commit 291065d)
@edymtt
Copy link
Contributor Author

edymtt commented Apr 27, 2024

@swift-ci please test

@edymtt
Copy link
Contributor Author

edymtt commented Apr 27, 2024

@swift-ci please build toolchain

@edymtt
Copy link
Contributor Author

edymtt commented Apr 27, 2024

@swift-ci please test WebAssembly

@edymtt
Copy link
Contributor Author

edymtt commented Apr 28, 2024

@swift-ci please test Windows

@edymtt edymtt merged commit b42e726 into apple:release/6.0 Apr 29, 2024
9 checks passed
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 6.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants