[GL] Testing with the Android Emulator on CI #3604
Conversation
![monocodus[bot]](https://avatars.githubusercontent.com/in/28526?s=60&v=4){kind=small}
examples/quad_android/src/lib.rs
Outdated
| @@ -0,0 +1,935 @@ | |||
| #[cfg(not(any( | |||
There was a problem hiding this comment.
does this have to be a separate example? could we make it a part of the existing quad instead?
There was a problem hiding this comment.
We should have a separate example because we need to compile crate-type = ["cdylib"] for Android. This should be partially fixed, because I removed most of code duplication.
Gordon-F
force-pushed
the
android_emulator_ci
branch
from
aff3acb
to
b85e539
Compare
| @@ -0,0 +1,37 @@ | |||
| [package] | |||
There was a problem hiding this comment.
another option, not necessarily a better one, is to have examples/src/lib.rs, so that the project is a cdylib library with binaries. The library would only be doing stuff on Android at this moment, but maybe we can find more use for this later?
There was a problem hiding this comment.
cargo apk allows run apk file only from cargo workspace root. But cargo doesn't allow pass features in cargo workspace root.
error: --features is not allowed in the root of a virtual workspaceThere was a problem hiding this comment.
This is error already fixed in cargo-apk. I will try to remove more code duplication with examples/src/lib.rs.
|
I'm going to close this PR, since |
Just a proof-of-concept of how we can run a
quad exampleon CI with the Android Emulator.All other jobs temporary removed (for speed up).
TODO:
thread '<unnamed>' panicked at 'called `Option::unwrap()` on a `None` value', /Users/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/ndk-glue-0.2.1/src/lib.rs:255:34Sometimes running apps panicked on the Android Emulator rust-mobile/ndk#116