Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add workaround for building Android bindings with supported NDK versions (r23+) #1782

Merged

Conversation

jmartinesp
Copy link
Contributor

Adds a temporary workaround to bindings/matrix-sdk-ffi/build.rs where we replace a linked library with a version that works fine in Android devices with x86_64 arch. This should let us release working versions of the Rust SDK for Android even while this bug still exists.

  • Public API changes documented in changelogs (optional)

@jmartinesp jmartinesp requested a review from a team April 17, 2023 14:31
@jmartinesp jmartinesp marked this pull request as ready for review April 17, 2023 14:39
@codecov
Copy link

codecov bot commented Apr 17, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (cd865d2) 75.95% compared to head (fc324dd) 75.95%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1782   +/-   ##
=======================================
  Coverage   75.95%   75.95%           
=======================================
  Files         139      139           
  Lines       15465    15465           
=======================================
  Hits        11747    11747           
  Misses       3718     3718           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Collaborator

@jplatte jplatte left a comment

Choose a reason for hiding this comment

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

A few codestyle nitpicks, otherwise looks good 👍🏼

@jplatte jplatte enabled auto-merge (squash) April 17, 2023 15:40
@jplatte jplatte merged commit c5919e3 into main Apr 17, 2023
@jplatte jplatte deleted the fix/jme/android-x86_64-issues-with-supported-ndk-versions branch April 17, 2023 16:06
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants