zkmopro/mopro

Move binary scripts to `build.rs`

Open

#357 opened on Mar 6, 2025

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Rust (110 forks)auto 404
DevExgood first issue

Repository metrics

Stars
 (238 stars)
PR merge metrics
 (PR metrics pending)

Description

Problem

Now we use CLI to generate bindings for iOS and Android

but if it could be moved to build.rs instead of running cargo run --bin ios It would make the building easier

Details

Ideally we just move the script to mopro-ffi/build.rs and when users activate

[build-dependencies]
mopro-ffi = {version = "0.1", features = ["ios"]}

the current app_config/ios.rs scripts executed and the bindings are generated

Question The current uniffi needs to build first and execute binary after building e.g.

cargo build --release
cargo run --bin uniffi-bindgen generate --library target/release/libmath.so --language kotlin --out-dir out

not sure if it can be executed in the same build script

Acceptance criteria

Run the scripts

Contributor guide

Research direction
Investigate how to move the binary script logic into build.rs, considering the uniffi build requirement. Check if the build script can run after cargo build? Possibly use cargo build script to generate bindings or a cust custom build step.
Tech stack
rust
Domain
mobile devbuild system
Issue type
Refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Mostly clear
Prerequisites
RustCargo
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
50