Skip to content

build: Ensure static headers are always symlinked #82125

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

Conversation

Steelskin
Copy link
Contributor

When using the Swift static build on Windows, projects like Foundation use the clang headers from the toolchain build lib directory. No target in the toolchain build depends on the static clang headers, so they do not get installed. As a workaround, this makes the static clang headers symlink target a dependency of the regular dynamic version of the headers, ensuring they are installed as part of the toolchain build.

When using the Swift static build on Windows, projects like Foundation
use the clang headers from the toolchain build `lib` directory. No
target in the toolchain build depends on the static clang headers, so
they do not get installed. As a workaround, this makes the static clang
headers symlink target a dependency of the regular dynamic version of
the headers, ensuring they are installed as part of the toolchainb
build.
@Steelskin Steelskin requested a review from a team as a code owner June 9, 2025 22:31
@compnerd compnerd requested a review from etcwilde June 9, 2025 22:33
@compnerd
Copy link
Member

compnerd commented Jun 9, 2025

@swift-ci please smoke test

@compnerd
Copy link
Member

@swift-ci please smoke test Linux platform

@compnerd compnerd enabled auto-merge June 10, 2025 19:17
@compnerd compnerd merged commit 9f5c090 into swiftlang:main Jun 10, 2025
3 checks passed
@Steelskin Steelskin deleted the fabrice/symlink-clang-headers-static-dep branch June 10, 2025 20:59
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.

2 participants