From 4ee0f524d46e25b00b67615be58be44ea9e95abc Mon Sep 17 00:00:00 2001 From: DanS Date: Wed, 2 Sep 2026 01:12:08 -0500 Subject: [PATCH] build(release): strip the Windows app exe MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The Linux and macOS release paths already strip the main binary (and the Windows path even stripped the dragonx-wallet-rebuild helper), but the Windows app exe was shipping unstripped — ~5 MB of symbols on the full node, ~19 MB on the params-heavy lite build. Strip it right after the Windows build, best-effort with a warning fallback. Co-Authored-By: Claude Opus 4.8 (1M context) --- build.sh | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/build.sh b/build.sh index e54ff98..e76cc82 100755 --- a/build.sh +++ b/build.sh @@ -842,6 +842,11 @@ HDR cmake --build . -j "$JOBS" [[ -f "bin/${APP_BASENAME}.exe" ]] || { err "Windows build failed"; exit 1; } + # Strip the app exe — the Linux and macOS release paths already strip theirs, and even the Windows + # helper (dragonx-wallet-rebuild.exe) is stripped, but the main app exe was shipping unstripped + # (~5MB of symbols on the full node, ~19MB on the params-heavy lite build). Best-effort. + x86_64-w64-mingw32-strip --strip-all "bin/${APP_BASENAME}.exe" 2>/dev/null \ + || warn " strip unavailable — shipping unstripped ${APP_BASENAME}.exe" info "Binary: $(du -h "bin/${APP_BASENAME}.exe" | cut -f1)" # A full-node release MUST include the recovery helper — fail loudly, never ship without it.