Skip to content

feat: replace Google NDK with monero-ndk v1.0.1 - #3

Open
Justxd22 wants to merge 2 commits into
MrCyjaneK:cyjan-wip-experimentsfrom
Justxd22:pr/monero-ndk
Open

feat: replace Google NDK with monero-ndk v1.0.1#3
Justxd22 wants to merge 2 commits into
MrCyjaneK:cyjan-wip-experimentsfrom
Justxd22:pr/monero-ndk

Conversation

@Justxd22

@Justxd22 Justxd22 commented Mar 9, 2026

Copy link
Copy Markdown
  • Replace Google NDK blob (android-ndk-r28c, ~1.7GB) with from-source monero-ndk v1.0.1 (~150MB) built from LLVM/clang
  • Drop 7zip and python@3.11 dependencies (no longer needed tarball extracts directly)
  • Simplify build steps using SYSROOT_TRIPLE env var instead of per-arch copy commands
  • Fix OpenSSL x86_64-linux-android config: split x86_64-linux* wildcard so Android doesn't match the desktop linux-x86_64 target

@Justxd22

Copy link
Copy Markdown
Author

Update now it checkout monero-ndk and build from source, a test build is on the way

https://github.com/Justxd22/simplybs/actions/runs/22910629494

@Justxd22

Justxd22 commented Mar 13, 2026

Copy link
Copy Markdown
Author

"version": "r28c",
"version": "monero-ndk-1.0.1",
"type": "native",
"download": [

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.download block should contain all dependencies, so make fetch should not be executed later - this is required for the builds to work offline

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed! now sources are fetched from simplybs!

@MrCyjaneK

MrCyjaneK commented Mar 16, 2026

Copy link
Copy Markdown
Owner

Doesn't compile on macOS

@MrCyjaneK

Copy link
Copy Markdown
Owner

Also doesn't compile on latest Ubuntu

@Justxd22

Justxd22 commented Jun 29, 2026

Copy link
Copy Markdown
Author

Also doesn't compile on latest Ubuntu

investigating this ^
weird thing it it works fine on git actions

@MrCyjaneK

Copy link
Copy Markdown
Owner

Please try rebasing on latest branch and let me know how it goes, I've removed all dependencies (other than libc)

@Justxd22

Copy link
Copy Markdown
Author

rebased now building on ubuntu, i will report back here in a while!

@MrCyjaneK
MrCyjaneK force-pushed the cyjan-wip-experiments branch 6 times, most recently from 6304418 to 6f491f1 Compare July 30, 2026 08:58
@MrCyjaneK
MrCyjaneK force-pushed the cyjan-wip-experiments branch 2 times, most recently from d6605f6 to c82675f Compare August 15, 2026 08:53
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.

2 participants