Skip to content

Conversation

@grynspan
Copy link
Contributor

This PR renames the entry point function used by SwiftPM to call into swift-testing such that it does not need to be SPI. By marking it SPI, we unintentionally made it very difficult to use with a toolchain that does not include private module information.

swift-testing will continue to export the old symbol name as used by SwiftPM until this change has firmly landed in toolchains, after which we will remove the old function.

See: swiftlang/swift-testing#141

This PR renames the entry point function used by SwiftPM to call into swift-testing such that it does not need to be SPI. By marking it SPI, we unintentionally made it very difficult to use with a toolchain that does not include private module information.

swift-testing will continue to export the old symbol name as used by SwiftPM until this change has firmly landed in toolchains, after which we will remove the old function.
@grynspan
Copy link
Contributor Author

@swift-ci please test

@grynspan grynspan enabled auto-merge (squash) November 29, 2023 19:12
grynspan added a commit to swiftlang/swift-testing that referenced this pull request Nov 29, 2023
This PR removes the `@_spi(SwiftPackageManagerSupport)`-annotated `swiftPMEntryPoint()` function now that SwiftPM has adopted the newer synonym, `__swiftPMEntryPoint()`.

Blocked by swiftlang/swift-package-manager#7137.
@grynspan grynspan merged commit 7d03e52 into main Nov 29, 2023
@grynspan grynspan deleted the jgrynspan/rename-swift-testing-entry-point branch November 29, 2023 20:59
grynspan added a commit to swiftlang/swift-testing that referenced this pull request Dec 4, 2023
This PR removes the `@_spi(SwiftPackageManagerSupport)`-annotated `swiftPMEntryPoint()` function now that SwiftPM has adopted the newer synonym, `__swiftPMEntryPoint()`.

Blocked by swiftlang/swift-package-manager#7137.
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.

5 participants