rust-mobile / ndk

Rust bindings to the Android NDK
Apache License 2.0
1.11k stars 110 forks source link

CI: Drop unneeded `cargo-ndk` tool from build-test #476

Closed MarijnS95 closed 3 months ago

MarijnS95 commented 3 months ago

A recent borked cargo-ndk release ^1 lead me to investigate why we are using a nonstandard tool in our CI in the first place. Compiling / build-testing the ndk crate does not require any external tooling whatsoever nor an SDK setup: this crate is a pure wrapper over pre-generated C bindings in the ndk-sys crate, which only require NDK libraries (and an Android-capable linker) when linking actual runnable binaries (typically shared libraries or executables). And the Rust compiler natively supports compiling Rust code for Android targets.