Skip to content

Use cargokit for builds and generate Rust->C FFI bindings #60

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.

Sign up for GitHub

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

Jump to bottom
Draft
sneurlax wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

Use cargokit for builds and generate Rust->C FFI bindings #60

sneurlax wants to merge 16 commits into from

Conversation

@sneurlax
Copy link
Member

@sneurlax sneurlax commented Mar 5, 2025 •
edited
Loading

Supercedes #56

@sneurlax
build!: remove lib link in linux example app cmakelists
fd5e2d9 
functionality to be re-added via irondash/cargokit
@sneurlax
Squashed 'cargokit/' content from commit 918f280
0e80393 
git-subtree-dir: cargokit
git-subtree-split: 918f28083f9ef24ce08b9e90d2c46e6a52812d5d
@sneurlax
Merge commit '0e8039303c9c7f3b0a9161bd3b4af378ac7cecfd' as 'cargokit'
4f3942a
@sneurlax
docs: document cargokit updates
f592db4 
added in previous merge commit as in `git subtree add --prefix cargokit https://github.com/irondash/cargokit.git main --squash` IAW https://matejknopp.com/post/flutter_plugin_in_rust_with_no_prebuilt_binaries/
@sneurlax
build: WIP macos cargokit integration
cd45c83
@sneurlax
build: WIP ios cargokit integration
70ebb12
@sneurlax
build: WIP linux cargokit integration
98e6c6c
@sneurlax
build: WIP windows cargokit integration
a94d893
@sneurlax
build: WIP android cargokit integration
0f499f2
@sneurlax
fix: standardize crate and lib names
a1d6e7f 
on underscores.

TODO rename to epic_wallet_ffi anyways.
@sneurlax sneurlax changed the title Use cargokit for FFI bindings generation Add tests, use cargokit for FFI bindings generation Mar 5, 2025
@sneurlax
Copy link
Member Author

sneurlax commented Mar 12, 2025

Closes #57

@sneurlax sneurlax changed the title Add tests, use cargokit for FFI bindings generation (3) Use cargokit for FFI bindings generation Mar 12, 2025
@sneurlax sneurlax changed the title (3) Use cargokit for FFI bindings generation Reorganize codebase, add tests and fix example app, and use cargokit for FFI bindings generation Mar 12, 2025
@sneurlax sneurlax mentioned this pull request Mar 12, 2025
Draft
47 tasks
@sneurlax sneurlax changed the title Reorganize codebase, add tests and fix example app, and use cargokit for FFI bindings generation Reorganize codebase, add tests, fix example app, and use cargokit for FFI bindings generation Mar 12, 2025
@sneurlax sneurlax closed this Mar 12, 2025
@sneurlax sneurlax reopened this Mar 17, 2025
@sneurlax sneurlax marked this pull request as ready for review March 17, 2025 19:16
@sneurlax sneurlax changed the title Reorganize codebase, add tests, fix example app, and use cargokit for FFI bindings generation Reorganize codebase, add tests, fix example app, use cargokit for builds, and generate Rust->C FFI bindings Mar 17, 2025
sneurlax added a commit to cypherstack/stack_wallet that referenced this pull request Mar 17, 2025
@sneurlax
feat: update flutter_libepiccash to cargokit branch
39f3c68 
TODO: merge cypherstack/flutter_libepiccash#60 "Reorganize codebase, add tests, fix example app, use cargokit for builds, and generate Rust->C FFI bindings" to main
@sneurlax sneurlax marked this pull request as draft March 17, 2025 21:56
@sneurlax
Copy link
Member Author

sneurlax commented Mar 17, 2025

Reverted to draft while Android issues are resolved (eg. for the example app)

@sneurlax sneurlax changed the title Reorganize codebase, add tests, fix example app, use cargokit for builds, and generate Rust->C FFI bindings Use cargokit for builds and generate Rust->C FFI bindings Mar 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sneurlax