Add Android ICU4C cross-compile packaging and CI - #26
Open
hahn-kev-bot wants to merge 10 commits into
Open
Conversation
Enables building SIL-modified ICU for Android ABIs and verifies the APK-ready output contract in FW Branch CI. Co-authored-by: Cursor <cursoragent@cursor.com>
Bash unsets every name in a multi-assignment local before applying values, so referencing major in the same statement failed after the host build. Co-authored-by: Cursor <cursoragent@cursor.com>
Only packaging and ICU source are needed for the cross-compile, so skip full history and unrelated trees. Co-authored-by: Cursor <cursoragent@cursor.com>
Packs Android ICU natives and data for MAUI consumers, uploads the nupkg on CI, and publishes to nuget.org on fw pushes. Co-authored-by: Cursor <cursoragent@cursor.com>
GitHub rejects upload-artifact@v2 and cache@v2; also refresh checkout, setup-msbuild, and GITHUB_OUTPUT. Co-authored-by: Cursor <cursoragent@cursor.com>
Resolve debian-packaging conflict by keeping noble packaging Install prerequisites from fw alongside Android CI updates. Co-authored-by: Cursor <cursoragent@cursor.com>
Ship archive-packaged stub libicudata with icudt*.dat, link unversioned SONAMEs for APK loading, and correct the PowerShell multi-ABI wrapper plus host/CI cache behavior. Co-authored-by: Cursor <cursoragent@cursor.com>
Without %(RecursiveDir) MSBuild flattens ABI folders into build/, so consumer Exists() checks skip all natives and data; verify the nupkg contents after pack. Co-authored-by: Cursor <cursoragent@cursor.com>
CI showed build/%(RecursiveDir) still omitted build/assets; emit concrete PackagePath entries for props, assets, and each ABI instead. Co-authored-by: Cursor <cursoragent@cursor.com>
CI and packaging now include 32-bit ARM alongside x86_64 and arm64-v8a so apps can run on older ARM devices. Co-authored-by: Cursor <cursoragent@cursor.com>
hahn-kev
marked this pull request as ready for review
August 11, 2026 10:01
|
@ermshiperete I think this is ready to go. In order to test it I downloaded the nupkg and used it in my tests in sillsdev/icu-dotnet#230, so I'm fairly confident that this is working correctly. As you can see AI wrote all the code for the build scripts, I've glanced over it but I don't really write shell scripts myself. If you have any feedback I'm happy to pass that back to an AI to fix it. Let me know what you think. |
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
2 participants
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.
Building ICU for Android so we can use it in icu-dotnet. Supports x86_64, arm64-v8a, and armeabi-v7a.
AI summary
Summary
build-android.sh/build-android.ps1to cross-compile this checkout's SIL-modified ICU4C for Android (x86_64,arm64-v8a) with NDK tooling and APK-ready.so/.datlayout.icu4c/packaging/README.android.md(linked from the top-level README).android-buildjob to FW Branch CI that installs NDK 27, builds both ABIs, and verifies the expected output files.Test plan
android-buildis green on this PRbash icu4c/packaging/build-android.sh --arch=x86_64locally with a Linux NDKMade with Cursor
This change is