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

UniFFI: supports bindings generated from multiple crates #2208

Merged
merged 6 commits into from
Sep 4, 2024

Conversation

MatMaul
Copy link
Contributor

@MatMaul MatMaul commented Sep 2, 2024

This changes allow to build a functional wheel of Matrix Rust SDK FFI bindings.

Copy link

netlify bot commented Sep 2, 2024

Deploy Preview for maturin-guide ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit d9eaa0e
🔍 Latest deploy log https://app.netlify.com/sites/maturin-guide/deploys/66d6f4a9f17448000890ff4b
😎 Deploy Preview https://deploy-preview-2208--maturin-guide.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@MatMaul MatMaul changed the title Add support for bindings generated from multiple crates UniFFI: supports bindings generated from multiple crates Sep 2, 2024
@messense
Copy link
Member

messense commented Sep 3, 2024

Thanks for the PR, could you please add a test crate in test-crates/ and write a test case? (See https://github.com/PyO3/maturin/blob/main/tests/run.rs)

src/module_writer.rs Outdated Show resolved Hide resolved
src/module_writer.rs Outdated Show resolved Hide resolved
@messense messense added the bindings/uniffi uniffi bindings label Sep 3, 2024
@MatMaul
Copy link
Contributor Author

MatMaul commented Sep 3, 2024

Good idea, minimal test case added.

Copy link
Member

@messense messense left a comment

Choose a reason for hiding this comment

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

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bindings/uniffi uniffi bindings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants