Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mobile support #5

Open
Tracked by #88
richard-ramos opened this issue Oct 4, 2022 · 0 comments
Open
Tracked by #88

Mobile support #5

richard-ramos opened this issue Oct 4, 2022 · 0 comments
Assignees
Labels
E:RLN non-native SDKs See https://github.com/waku-org/pm/issues/88 for details

Comments

@richard-ramos
Copy link
Member

richard-ramos commented Oct 4, 2022

Currently go-zerokit-rln can only be used in desktop devices. Support for mobile devices must be added:
According to the NDK docs

ABI Triple
armeabi-v7a armv7-linux-androideabi
arm64-v8a aarch64-linux-android
x86 i686-linux-android
x86-64 x86_64-linux-android

Do notice that there's a note that says:

Note: For 32-bit ARM, the compiler is prefixed with armv7a-linux-androideabi, but the binutils tools are prefixed with arm-linux-androideabi. For other architectures, the prefixes are the same for all tools.

So maybe it's necessary to support arm-linux-androideabi which might be problematic due to the following error:

error[E0432]: unresolved imports `std::sync::atomic::AtomicI64`, `std::sync::atomic::AtomicU64`
 --> /cargo/registry/src/github.com-1ecc6299db9ec823/loupe-0.1.3/src/memory_usage/sync.rs:7:43
  |
7 |         AtomicBool, AtomicI16, AtomicI32, AtomicI64, AtomicI8, AtomicIsize, AtomicU16, AtomicU32,
  |                                           ^^^^^^^^^ no `AtomicI64` in `sync::atomic`
8 |         AtomicU64, AtomicU8, AtomicUsize,
  |         ^^^^^^^^^ no `AtomicU64` in `sync::atomic`

Although I expect that armv7-linux-androideabi is the one to be used and should work fine. Zerokit compiles fine for all the target triples from the first table. For ios, I was able to cross compile zerokit to aarch64-apple-ios and x86_64-apple-ios succesfully.

For implementing this task, we need to modify go-zerokit-rln-x86_64, go-zerokit-rln-arm and go-zerokit-rln-apple to add build constraints that take into account when android is being used.

@richard-ramos richard-ramos self-assigned this Sep 12, 2023
@fryorcraken fryorcraken added the E:RLN non-native SDKs See https://github.com/waku-org/pm/issues/88 for details label Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E:RLN non-native SDKs See https://github.com/waku-org/pm/issues/88 for details
Projects
None yet
Development

No branches or pull requests

2 participants