cargo cinstall fails on Android and iOS because there's no shared libs

[nirbheek]

Error: CliError { error: Some(failed to copy `/home/nirbheek/projects/repos/cerbero.git/rust/build/sources/android_universal/armv7/gst-plugins-rs-0.8.4.9999/_builddir/armv7-linux-androideabi/release/libgstrsjson.so` to `/home/nirbheek/projects/repos/cerbero.git/rust/build/dist/android_universal/armv7/lib/gstreamer-1.0/libgstrsjson.so`

Caused by:
    No such file or directory (os error 2)), exit_code: 101 }

Error: CliError { error: Some(failed to copy `/Users/nirbheek/projects/repos/cerbero.git/rust/build/sources/ios_universal/arm64/gst-plugins-rs-0.8.4.9999/_builddir/aarch64-apple-ios/release/libgstrsaudiofx.dylib` to `/Users/nirbheek/projects/repos/cerbero.git/rust/build/dist/ios_universal/arm64/lib/gstreamer-1.0/libgstrsaudiofx.dylib`

Caused by:
    No such file or directory (os error 2)), exit_code: 101 }

Cross-compiling to macOS arm64 works fine, and explicitly passing --library-type cdylib --library-type staticlib doesn't change anything.

[lu-zero]

Looks like upstream decided that making cdylibs is not supported somehow.

[nirbheek]

Did you find some issues / PRs that should be linked from here?

[lu-zero]

Sadly not, if you run cargo build --target {*-ios} or such you'll see the warning.

[nirbheek]

The main issue for iOS seems to be rust-lang/rust#21727, the major blocker is rust-lang/rust#95847 (comment). One potential fix would be rust-lang/cargo#4881.

Cargo doesn't seem to emit any warnings for Android though, and I can't find any issues about that either.

[lu-zero]

the lib in android is somehow missing in target or the naming logic got a quirk? Thank you a lot for testing :)

[lu-zero]

I tested locally and the ndk21 works fine and it installs for android...

❯ cargo cinstall --manifest-path=example-project/Cargo.toml --target aarch64-linux-android --destdir=/tmp/android21
   Compiling proc-macro2 v1.0.43
   Compiling quote v1.0.21
   Compiling unicode-ident v1.0.3
   Compiling syn v1.0.99
   Compiling serde_derive v1.0.144
   Compiling serde v1.0.144
   Compiling semver v1.0.13
   Compiling serde_json v1.0.85
   Compiling camino v1.1.1
   Compiling ryu v1.0.11
   Compiling itoa v1.0.3
   Compiling libc v0.2.132
   Compiling cargo-platform v0.1.2
   Compiling cargo_metadata v0.14.2
   Compiling example-project v0.1.0 (/Users/lu_zero/Sources/rust/cargo-c/example-project)
    Finished release [optimized] target(s) in 15.82s
    Building pkg-config files
    Building header file using cbindgen
  Populating uninstalled header directory
  Installing pkg-config file
  Installing header file
  Installing static library
  Installing shared library

❯ tree /tmp/android21/
/tmp/android21/
└── usr
    └── local
        ├── include
        │   ├── example-project-0.1
        │   │   └── example_project
        │   │       ├── example_project.h
        │   │       ├── file.h
        │   │       ├── generated.h
        │   │       └── subdir
        │   │           └── in_subdir.h
        │   └── otherplace
        │       └── other_file.h
        └── lib
            ├── libexample-project.a
            ├── libexample-project.so -> libexample-project.so.0.1.0
            ├── libexample-project.so.0 -> libexample-project.so.0.1.0
            ├── libexample-project.so.0.1.0
            └── pkgconfig
                └── example_project.pc

[lu-zero]

Also cargo-c already sets the crate-type so all you need is to revert the revert (and point whoever is developing cargo-mobile now here)

[lu-zero]

@nirbheek is this still a problem?

[lu-zero]

@nirbheek is there anything left to be done about this?

[nirbheek]

Seems to be fixed! Thanks for checking in.