Skip to content

Propagate '-application-extension' to module interface loader sub-invocations. #61545

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

Merged
merged 1 commit into from
Oct 13, 2022

Conversation

artemcm
Copy link
Contributor

@artemcm artemcm commented Oct 11, 2022

This flag restricts availability of certain symbols to ensure the code cannot use declarations that are explicitly unavalable to extensions. This restriction should be passed down to dependency modules also.

@artemcm artemcm requested a review from nkcsgexi October 11, 2022 18:23
…ocations.

This flag restricts availability of certain symbols to ensure the code cannot use declarations that are explicitly unavalable to extensions. This restriction should be passed down to dependency modules also.
@artemcm artemcm force-pushed the PropagateApplicationExtensionStatus branch from 3601399 to 0c470f2 Compare October 11, 2022 18:24
@artemcm
Copy link
Contributor Author

artemcm commented Oct 11, 2022

@swift-ci test

Copy link
Contributor

@nkcsgexi nkcsgexi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch!

@DougGregor
Copy link
Member

swiftlang/swift-format#424

@swift-ci please smoke test

@artemcm
Copy link
Contributor Author

artemcm commented Oct 12, 2022

@swift-ci please smoke test macOS platform

@artemcm
Copy link
Contributor Author

artemcm commented Oct 12, 2022

@swift-ci please test macOS platform

@artemcm artemcm merged commit 625048a into swiftlang:main Oct 13, 2022
@artemcm artemcm deleted the PropagateApplicationExtensionStatus branch October 13, 2022 15:55
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.

3 participants