Skip to content

fix: verify android elf build ids#131

Open
ovitrif wants to merge 1 commit into
masterfrom
ovi/fix-core-android-build-ids
Open

fix: verify android elf build ids#131
ovitrif wants to merge 1 commit into
masterfrom
ovi/fix-core-android-build-ids

Conversation

@ovitrif

@ovitrif ovitrif commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

Fixes #130

Implements the bitkit-core producer portion of synonymdev/bitkit-android#1105. Companion PRs: synonymdev/ldk-node#98, synonymdev/vss-rust-client-ffi#19, pubky/paykit-rs#124, and synonymdev/bitkit-android#1106.

Description

  • links every published Android ABI with a SHA-1 GNU build ID while preserving the existing 16 KB page-size linker flags
  • requires build-ID presence before stripping, verifies strip survival, and compares each final AAR library with its unstripped symbol counterpart
  • requires full DWARF metadata in the native debug-symbol archive and makes publication depend on artifact validation
  • bumps the cross-platform package to 0.4.4 and regenerates Android, iOS, Python, and Swift artifacts

Preview

N/A — build and publishing changes only.

QA Notes

Manual Tests

  • Published 0.4.4 to the task-local Maven repository and consumed its AAR and native-debug-symbols classifier in the Bitkit Android release build.
  • Independently inspected all packaged and symbol libraries with NDK llvm-readelf -n.

Automated Checks

  • cargo fmt --check
  • cargo clippy --all-targets --all-features (passes with existing warnings)
  • cargo test --all-features -- --skip modules::blocktank — 399 passed, 25 ignored, 48 filtered
  • ./gradlew build
  • ./gradlew validatePublishedNativeArtifacts publishToMavenLocal
  • shellcheck -e SC2115 build_android.sh
  • GNU build-ID presence, strip survival, full DWARF, and packaged-to-symbol parity passed for armeabi-v7a, arm64-v8a, x86, and x86_64

chatgpt-codex-connector[bot]

This comment was marked as resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: add Android ELF build IDs

1 participant