Skip to content
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

[Distributed] Accessor must be available cross module in resilient mode #80523

Conversation

ktoso
Copy link
Contributor

@ktoso ktoso commented Apr 4, 2025

This is an important fix for libraries using @Resolvable in resilient libraries. Without the fix we're missing an accessor and this will fail some remote calls which make use of remote calls on resolvable protocols. This would manifest as missing accessor error thrown by the executeDistributedTarget function.

resolves rdar://148224780

@ktoso ktoso requested a review from rjmccall as a code owner April 4, 2025 04:28
@ktoso
Copy link
Contributor Author

ktoso commented Apr 4, 2025

@swift-ci please test

@ktoso ktoso requested a review from xedin April 4, 2025 04:28
@ktoso ktoso force-pushed the wip-distributed-cross-module-resolvable-accessible-fun branch from 3c0e644 to 2b1290d Compare April 4, 2025 04:29
@ktoso
Copy link
Contributor Author

ktoso commented Apr 4, 2025

@swift-ci please test

@ktoso ktoso force-pushed the wip-distributed-cross-module-resolvable-accessible-fun branch from 2b1290d to cadd458 Compare April 4, 2025 05:39
@ktoso
Copy link
Contributor Author

ktoso commented Apr 4, 2025

@swift-ci please smoke test

@ktoso
Copy link
Contributor Author

ktoso commented Apr 4, 2025

@swift-ci please test macOS

This is an important fix for libraries using @resolvable in resilient
libraries. Without the fix we're missing an accessor and this will fail
some remote calls which make use of remote calls on resolvable
protocols. This would manifest as missing accessor error thrown by the
executeDistributedTarget function.

resolves rdar://148224780
@ktoso ktoso force-pushed the wip-distributed-cross-module-resolvable-accessible-fun branch from cadd458 to d7cd90d Compare April 7, 2025 04:14
@ktoso
Copy link
Contributor Author

ktoso commented Apr 7, 2025

@swift-ci please smoke test

@ktoso ktoso force-pushed the wip-distributed-cross-module-resolvable-accessible-fun branch from d7cd90d to a3644a2 Compare April 7, 2025 04:20
@ktoso
Copy link
Contributor Author

ktoso commented Apr 7, 2025

@swift-ci please smoke test

@ktoso ktoso enabled auto-merge (squash) April 7, 2025 04:22
@ktoso ktoso merged commit f21d029 into swiftlang:main Apr 7, 2025
3 checks passed
@ktoso ktoso deleted the wip-distributed-cross-module-resolvable-accessible-fun branch April 7, 2025 09: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