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

Support for Swift wrapper #110

Closed
wants to merge 2 commits into from
Closed

Conversation

conanoc
Copy link
Contributor

@conanoc conanoc commented Mar 6, 2023

This fixes #99, which is necessary for Swift.

The xcframework built for reactive-native can be shared with the Swift wrapper. But, it requires a different archive format. So I added a workflow step to archive it using zip with a different folder name AskarFramework.
The Swift wrapper in different repo could download the zipped xcframework for its dependencies.

@conanoc conanoc mentioned this pull request Mar 6, 2023
@andrewwhitehead
Copy link
Member

Maybe we should just publish the xcframework as a zip and the Android libraries as a separate .tar.gz?

@conanoc
Copy link
Contributor Author

conanoc commented Mar 7, 2023

Maybe we should just publish the xcframework as a zip and the Android libraries as a separate .tar.gz?

Sounds good. @blu3beri What do you think?

@conanoc
Copy link
Contributor Author

conanoc commented Mar 7, 2023

I think I need to test uniffi first because uniffi will generate a different header file. I'll close this now and re-open it later if uniffi fails.

@conanoc conanoc closed this Mar 7, 2023
@TimoGlastra
Copy link
Member

Maybe we should just publish the xcframework as a zip and the Android libraries as a separate .tar.gz?

We need the ios and android build to be published in one tar/zip so they can be dynamically downloaded when the react native package is installed from npm. We can't download multiple binaries, or we'd have to implement that logic ourselves

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.

cbindgen does not generate some types
3 participants