-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Generate entrypoints for test bundles on macOS #9018
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
base: main
Are you sure you want to change the base?
Conversation
@swift-ci test |
@@ -695,6 +695,11 @@ public final class SwiftBuildSystem: SPMBuildCore.BuildSystem { | |||
settings["ARCHS"] = architectures.joined(separator: " ") | |||
} | |||
|
|||
// When building with the CLI for macOS, test bundles should generate entrypoints for compatibility with swiftpm-testing-helper. | |||
if buildParameters.triple.isMacOSX { |
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.
macOS, or all Darwin?
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.
Well we don't support command-line execution of tests on other Apple platforms anyway, so...
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.
Execution doesn't matter, this is about the build. The build system does not assume that the machine/tool building the tests is the one running them.
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.
Okay: we support neither building nor executing tests on iOS using SwiftPM.
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.
Looks good, though it does appears we have regressed in some areas.
627f0ba
to
b19a7f1
Compare
@swift-ci test |
@swift-ci test Windows |
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.
I understand the PR is in draft, but I have a comment, which may or may not be applicable :)
b19a7f1
to
86ad587
Compare
@swift-ci test |
@swift-ci test Windows |
@swift-ci test self hosted |
86ad587
to
dc198e1
Compare
@swift-ci test self hosted |
dc198e1
to
2a9d838
Compare
@swift-ci test |
1 similar comment
@swift-ci test |
One remaining text failure should be fixed by #9115 |
@swift-ci test self hosted |
Closes #8973
Ensure test bundles get a main symbol on macOS so swiftpm-testing-helper can kick off tests. We may want to rename this entrypoint to solve some issues with testable executables though.