Skip to content

Fixes android symbol loader not having is_loaded (build error)#1407

Closed
lattice0 wants to merge 3 commits intorust-windowing:masterfrom
lattice0:master
Closed

Fixes android symbol loader not having is_loaded (build error)#1407
lattice0 wants to merge 3 commits intorust-windowing:masterfrom
lattice0:master

Conversation

@lattice0
Copy link

Fixes #1307

  • Tested on all platforms changed
  • Compilation warnings were addressed
  • cargo fmt has been run on this branch
  • cargo doc builds successfully
  • Added an entry to CHANGELOG.md if knowledge of this change could be valuable to users
  • Updated documentation to reflect any user-facing changes, including notes of platform-specific behavior
  • Created or updated an example program if it would help users understand this functionality

@lattice0
Copy link
Author

Hello, before I fix the CI/CD stuff and the missing checkboxes, can I know if something's missing? I think these changes fix the problem. Android seems to not have dynamic loading of dynamically loadable libs so there's no .is_loaded() for it.

@lattice0
Copy link
Author

PS: all testes failed because I pointed to a local winit for my stuff. Gonna fix it soon but currently maybe leave like this so my local project works, is there a problem? just want to keep things simple

@lattice0 lattice0 changed the title Fixes android build error Fixes android symbol loader not having is_loaded (build error) May 21, 2022
@MarijnS95
Copy link
Member

FYI this is going to get closed when #1398 is merged, which turns out to be a better solution to this problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Error compiling to Android with cargo apk

2 participants