Skip to content

[5.3] Fix module triples for “ios-like” Mac Catalyst modules #31576

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
May 7, 2020

Conversation

beccadax
Copy link
Contributor

@beccadax beccadax commented May 6, 2020

Cherry-picks #31394 to 5.3 branch:

The “ios-like” build flavor is used to build modules that do not exist on macOS for Mac Catalyst. Even though they are built for the “OSX” SDK, they need to have a “MACCATALYST”-style module triple; unfortunately, the transition to naming swiftmodules by module triple in #31170 did not handle this edge case correctly.

This commit handles that by piggybacking on a similar special case used to change the lib/swift subdirectory.

Fixes rdar://62902611.

The “ios-like” build flavor is used to build modules that do not exist on macOS for Mac Catalyst. Even though they are built for the “OSX” SDK, they need to have a “MACCATALYST”-style module triple; unfortunately, the transition to naming swiftmodules by module triple in swiftlang#31170 did not handle this edge case correctly.

This commit handles that by piggybacking on a similar special case used to change the lib/swift subdirectory.
@beccadax beccadax added the r5.3 label May 6, 2020
@beccadax beccadax requested a review from a team as a code owner May 6, 2020 04:27
@beccadax
Copy link
Contributor Author

beccadax commented May 6, 2020

@swift-ci test

@beccadax beccadax requested a review from DougGregor May 6, 2020 17:42
@beccadax beccadax merged commit dc9de24 into swiftlang:release/5.3 May 7, 2020
@AnthonyLatsis AnthonyLatsis added swift 5.3 🍒 release cherry pick Flag: Release branch cherry picks labels Jan 8, 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 swift 5.3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants