Skip to content

Conversation

@artemcm
Copy link
Contributor

@artemcm artemcm commented Feb 17, 2023

The logic assumed we don't have the .private. in the interface name and looked for an adjacent binary module with .private.swiftmodule, which we do not produce. So also try stripping off the .private. and do a lookup.

@artemcm artemcm requested review from nkcsgexi and xymus February 17, 2023 00:11
@artemcm
Copy link
Contributor Author

artemcm commented Feb 17, 2023

@swift-ci test

@artemcm artemcm force-pushed the PrivatePeerCandidateDiscovery branch from db62258 to 6b13bf0 Compare February 20, 2023 23:36
@artemcm
Copy link
Contributor Author

artemcm commented Feb 20, 2023

@swift-ci test

@artemcm artemcm force-pushed the PrivatePeerCandidateDiscovery branch from 6b13bf0 to 6d59868 Compare February 21, 2023 17:58
@artemcm
Copy link
Contributor Author

artemcm commented Feb 21, 2023

1 similar comment
@artemcm
Copy link
Contributor Author

artemcm commented Feb 21, 2023

@artemcm
Copy link
Contributor Author

artemcm commented Feb 21, 2023

swiftlang/swift-driver#1282
@swift-ci test macOS Platform

@artemcm artemcm merged commit 1e49c3b into swiftlang:main Feb 22, 2023
@artemcm artemcm deleted the PrivatePeerCandidateDiscovery branch February 22, 2023 16:09
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