feat: publish io.parity:truapi-host as an AAR - #337
Draft
valentinfernandez1 wants to merge 2 commits into
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #328.
Related https://github.com/paritytech/polkadot-app-android-v2/pull/953
What
truapi now publishes the Android host as
io.parity:truapi-hoston GitHub Packages. The AAR bundles the per-ABIlibtruapi_server.so(arm64-v8a, armeabi-v7a, x86, x86_64), the UniFFI Kotlin bindings, and the Kotlin shell, all built from the same source. Consumers only need Gradle. JNA is the only runtime dependency.What changed
release-androidworkflow: builds and publishes the AAR on atruapi-host-android@<version>tag or a manual run.ci-androidworkflow: compiles the shell against fresh bindings on PRs touchingandroid/or the native crates.make android-jninow covers all four ABIs and removes stray dependency.sofiles, so the AAR carries exactly fourlibtruapi_server.so.io.parity:truapi-host, version comes from-PtruapiHostVersion.review-fixturesbin intruapi-platformprints canonical SCALE hex perUserConfirmationReviewvariant, with a golden test that fails on any encoding drift.Draft: verifying the publish from this branch
workflow_dispatchonly works once the workflow file is on the default branch, so this PR carries a temporary push trigger on this branch. Every push publishes0.1.0-rc.<run number>to GitHub Packages, which lets us prove the whole pipeline before merging. Before marking ready for review: remove the temporary trigger and delete the rc packages from the registry.Once the pipeline is proven, a follow-up will fold the Android publish into the normal
release:commit flow so it is not a separate step people can forget.