Skip to content

🍒 Use the Windows SDK arguments over the environment (llvm#144805) #10895

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Conversation

Steelskin
Copy link

If any of the Windows SDK (and MSVC)-related argument is passed in the command line, they should take priority over the environment variables like INCLUDE or LIB set by vcvarsall from the Visual Studio Developer Environment on Windows.

These changes ensure that all of the arguments related to VC Tools and the Windows SDK cause the driver to ignore the environment.

If any of the Windows SDK (and MSVC)-related argument is passed in the
command line, they should take priority over the environment variables
like `INCLUDE` or `LIB` set by vcvarsall from the Visual Studio
Developer Environment on Windows.

These changes ensure that all of the arguments related to VC Tools and
the Windows SDK cause the driver to ignore the environment.
@Steelskin Steelskin requested a review from a team as a code owner June 24, 2025 23:47
@Steelskin
Copy link
Author

@swift-ci please test windows platform

@Steelskin
Copy link
Author

@swift-ci please test

@compnerd compnerd merged commit 0a334dd into swiftlang:stable/20240723 Jun 26, 2025
3 checks passed
@Steelskin Steelskin deleted the fabrice/cp-no-include-for-win-sdk-version branch June 27, 2025 17:01
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants