Skip to content

[swift][AutoLinkExtract] Don't add autolink hint for frameworks. (#65… #65184

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
Apr 14, 2023

Conversation

ributzka
Copy link
Contributor

…051)

Default system linkers on non-Darwin platforms do not support the -framework argument for framework linking. This change updates autolinking to not emit -framework into the .o _swift1_autolink_entries metadata when there is no native linker support.

This is related to rdar://106578342.

(cherry picked from commit 17c5010)

…ftlang#65051)

Default system linkers on non-Darwin platforms do not support the `-framework`
argument for framework linking. This change updates autolinking to not emit
`-framework` into the .o _swift1_autolink_entries metadata when there is no
native linker support.

This is related to rdar://106578342.

(cherry picked from commit 17c5010)
@ributzka ributzka requested a review from a team as a code owner April 14, 2023 16:23
@ributzka
Copy link
Contributor Author

@swift-ci please test

@ributzka ributzka requested a review from artemcm April 14, 2023 18:59
@ributzka ributzka merged commit 6776feb into swiftlang:release/5.9 Apr 14, 2023
@ributzka ributzka deleted the eng/PR-106578342-5.9 branch April 14, 2023 22:10
@AnthonyLatsis AnthonyLatsis added the 🍒 release cherry pick Flag: Release branch cherry picks label May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants