ReviewOS

also looking at this

craft-native/craft

perf(android): ship libcraft.so without its DWARF, and keep the DWARF for symbols

#217
Merged glennmichael123 wants to merge fix/android-strip-libcraft into main
6 files +235 -14
.github/workflows/README.mdmodified+4-1
Changes to .github/workflows/README.md
@@ -55,7 +55,10 @@ into the APK and once without:
5555
5656The Zig Android library links bionic, so building it needs the NDK —
5757`sdkmanager --install "ndk;26.1.10909125"`. Without it `build-android` refuses
58rather than producing a library that installs and then fails to load.
58rather than producing a library that installs and then fails to load. The NDK's
59`llvm-objcopy` also strips each release library's DWARF into
60`zig-out/android-symbols/`, and the suite refuses to start on a `libcraft.so`
61that still carries any.
5962
6063```bash
6164cd packages/zig && zig build build-android-all -Doptimize=ReleaseSafe -Dandroid-ndk="$ANDROID_NDK_HOME" && cd -