-
Notifications
You must be signed in to change notification settings - Fork 302
Support building only tests using SwiftPM #1768
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
Conversation
1fdc196
to
f2fa6e4
Compare
@swift-ci Please test |
f2fa6e4
to
3ce1a32
Compare
@swift-ci Please test |
@swift-ci Please test Windows |
This allows us to build everything required for the executable using CMake and only the test targets using SwiftPM, enabling us to run tests on Windows from build.ps1 with minimal overhead.
3ce1a32
to
24488cc
Compare
@swift-ci Please test |
@swift-ci Please test Windows |
@@ -1,4 +1,4 @@ | |||
add_compile_options("$<$<COMPILE_LANGUAGE:Swift>:SHELL:-package-name SourceKitLSP>") | |||
add_compile_options("$<$<COMPILE_LANGUAGE:Swift>:SHELL:-package-name sourcekit_lsp>") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why the switch?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When building the tests using SwiftPM, this is the package name that SwiftPM will chose and it means that we can use declarations at the package level from the modules built using CMake.
This allows us to build everything required for the executable using CMake and only the test targets using SwiftPM, enabling us to run tests on Windows from build.ps1 with minimal overhead.
Analogous to swiftlang/swift-format#851