frida / frida-rust

Frida Rust bindings
Other
188 stars 53 forks source link

Fix iOS build #104

Closed domenukk closed 1 year ago

domenukk commented 1 year ago

iOS was broken both at compile time (i8 != ui pointers) and link time (missing dependencies). This PR fixes that and also adds CI.

domenukk commented 1 year ago

Giving up to add Android to CI, it doesn't find the time.h header (same error as #34):

/Users/runner/work/frida-rust/frida-rust/target/aarch64-linux-android/debug/build/frida-sys-15ad3d6ad5fda1d0/out/frida-core.h:8763:10: fatal error: 'time.h' file not found
saruman9 commented 1 year ago

@domenukk,

I have determined that some versions of NDK and Rust are incompatible. For example, such versions are non-working:

Try these versions: rust 1.67, NDK 22, clang30. This worked for me when I was building LibAFL.

domenukk commented 1 year ago

Didn't mean to close :) PR should be ready. I'll not do Android CI for now, it's too much trial and error

domenukk commented 1 year ago

@meme would appreciate a merge and release whenever it's convenient, then we can update LibAFL too, thx :)

domenukk commented 1 year ago

Thanks! :)

domenukk commented 1 year ago

Could you make a release plz? :)

meme commented 1 year ago

Done!