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

Add debug symbols for iOS and macOS builds #49

Merged
merged 4 commits into from
Jan 15, 2025

Conversation

mugikhan
Copy link
Contributor

@mugikhan mugikhan commented Dec 18, 2024

There is now an error message in Xcode 16 for a missing dSYM folder and DWARF file.

The archive did not include a dSYM for the powersync-sqlite-core.framework with the 
UUIDs [7E9FE23E-F16F-35CB-830B-54AD4C79F057]. Ensure that the archive's dSYM 
folder includes a DWARF file for powersync-sqlite-core.framework with the expected UUIDs.

@stevensJourney
Copy link
Contributor

I tested this locally.

Initially I could reproduce the warning by using our latest powersync-sqlite-core pod and submitting an app build to TestFlight. The submission stated the above warning.

I then built the XCFramework locally from this repo, removed the powersync-sqlite-core pod dependency and linked the local XCFramework instead to my project. The submission to TestFlight succeeded without any warnings.

I'm happy to merge this from my side.

@stevensJourney stevensJourney marked this pull request as ready for review January 15, 2025 12:09
Copy link
Contributor

@rkistner rkistner left a comment

Choose a reason for hiding this comment

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

There is already a v0.3.8 - need to merge main and bump again

@stevensJourney
Copy link
Contributor

There is already a v0.3.8 - need to merge main and bump again

👍 Bumped again.

@stevensJourney stevensJourney merged commit 024fda8 into main Jan 15, 2025
32 checks passed
@stevensJourney stevensJourney deleted the fix/add-dysm-for-frameworks branch January 15, 2025 15: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.

3 participants