Skip to content

Windows: Use pre-installed CMake and Ninja #80918

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
merged 1 commit into from
Apr 19, 2025

Conversation

etcwilde
Copy link
Member

Teach build.ps1 to search for a pre-installed CMake and Ninja from the PATH instead of only using one installed in Visual Studio. If one is not installed or on the path, the script will attempt to use one installed in Visual Studio. Preference is given to the executable found in the path environment variable.

@etcwilde etcwilde requested a review from compnerd April 18, 2025 19:48
@etcwilde
Copy link
Member Author

@swift-ci please smoke test

@etcwilde etcwilde force-pushed the ewilde/use-installed-cmake-ninja branch 2 times, most recently from a6b5a3d to 54289fa Compare April 19, 2025 00:01
@etcwilde
Copy link
Member Author

@swift-ci please smoke test

@etcwilde etcwilde requested a review from compnerd April 19, 2025 00:02
@etcwilde etcwilde force-pushed the ewilde/use-installed-cmake-ninja branch from 54289fa to 25b2df6 Compare April 19, 2025 00:08
@etcwilde
Copy link
Member Author

@swift-ci please smoke test

@etcwilde etcwilde force-pushed the ewilde/use-installed-cmake-ninja branch from 25b2df6 to 2ae9ca3 Compare April 19, 2025 00:09
@etcwilde
Copy link
Member Author

@swift-ci please smoke test

Teach build.ps1 to search for a pre-installed CMake and Ninja from the
PATH instead of only using one installed in Visual Studio. If one is not
installed or on the path, the script will attempt to use one installed
in Visual Studio. Preference is given to the executable found in the
path environment variable.
@etcwilde etcwilde force-pushed the ewilde/use-installed-cmake-ninja branch from 2ae9ca3 to b15303d Compare April 19, 2025 00:51
@etcwilde
Copy link
Member Author

@swift-ci please smoke test

@etcwilde etcwilde merged commit 3d43e1a into swiftlang:main Apr 19, 2025
3 checks passed
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