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

Temporarily specify the Windows SDK version #913

Merged
merged 1 commit into from
Mar 24, 2025

Conversation

Steelskin
Copy link
Collaborator

This is necessary to work around an issue with the latest version (26100) of the Windows SDK where changes in headers result in circular module dependencies.

See #909 for details.

@Steelskin Steelskin force-pushed the fabrice/override-win-sdk branch from 67645cf to 11e5161 Compare March 4, 2025 21:37
@Steelskin Steelskin force-pushed the fabrice/override-win-sdk branch 8 times, most recently from 4cb52e9 to 0122156 Compare March 7, 2025 23:06
@Steelskin Steelskin force-pushed the fabrice/override-win-sdk branch 6 times, most recently from 0c47103 to 74b4b5d Compare March 17, 2025 21:19
@Steelskin Steelskin force-pushed the fabrice/override-win-sdk branch from 74b4b5d to 3857d48 Compare March 24, 2025 16:48
@Steelskin Steelskin requested a review from compnerd March 24, 2025 16:49
@Steelskin
Copy link
Collaborator Author

@compnerd Could I ask you to take another look at the updated patch? This finally passed:
https://github.com/thebrowsercompany/swift-build/actions/runs/14004017544/job/39218072058

I want to test our other bots before I land this.

This is necessary to work around an issue with the latest version
(26100) of the Windows SDK where changes in headers result in circular
module dependencies.
For the compilers build, this installs and uses MSVC 14.42 because the
current build does not work with MSVC 14.43. In addition, this removes
the 26100 Windows SDK to get the compilers to build until the bootstrap
toolchain supports the new SDK.

See #909 for details.
@Steelskin Steelskin force-pushed the fabrice/override-win-sdk branch from 3857d48 to d7a223d Compare March 24, 2025 17:21
Copy link
Owner

@compnerd compnerd left a comment

Choose a reason for hiding this comment

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

We will need to figure out how to remove this workaround ASAP

@Steelskin Steelskin merged commit 5e856b3 into main Mar 24, 2025
1 check passed
@Steelskin Steelskin deleted the fabrice/override-win-sdk branch March 24, 2025 20:26
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