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

Prevent static XCFramework bundles from being embedded on iOS #92519

Merged
merged 1 commit into from
May 30, 2024

Conversation

mihe
Copy link
Contributor

@mihe mihe commented May 29, 2024

Currently when archiving/distributing iOS apps that utilize extensions that have been built with the XCFramework approach, as seen in godot-cpp's test project here, you will currently run into errors when you try to validate said app, due to the .a files of the two XCFramework bundles (godot-cpp and the extension) being bundled within the .app.

This PR fixes this by checking to see whether the XCFramework only contains static libraries before respecting IOSExportAsset::should_embed, same as what regular frameworks already do as of #86288, as seen a few lines above this change.

CC @bruvzg

@mihe mihe requested a review from a team as a code owner May 29, 2024 16:03
@AThousandShips AThousandShips added this to the 4.3 milestone May 29, 2024
@akien-mga akien-mga merged commit 8ec272f into godotengine:master May 30, 2024
16 checks passed
@akien-mga
Copy link
Member

Thanks!

@mihe mihe deleted the stop-xcframework-embed branch May 30, 2024 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants