This is an automated email from the ASF dual-hosted git repository.
jimjag pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/openoffice.git
The following commit(s) were added to refs/heads/trunk by this push:
new 6ca087ffd6 nss: disable -Werror and gtests for the macOS build
6ca087ffd6 is described below
commit 6ca087ffd66b43b0098357ff9ea4c4aa083a0c98
Author: Jim Jagielski <[email protected]>
AuthorDate: Thu Jun 11 16:29:36 2026 -0400
nss: disable -Werror and gtests for the macOS build
NSS 3.39's coreconf/werror.mk enables -Werror by default, and clang 13+
adds -Wunused-but-set-variable, which 2018-era NSS code trips:
shvfy.c:336:9: error: variable 'count' set but not used
[-Werror,-Wunused-but-set-variable]
The Linux BUILD_ACTION already passes NSS_ENABLE_WERROR=0; give macOS
the same treatment, plus NSS_DISABLE_GTESTS=1 as on Linux/FreeBSD so
the bundled gtest suite is not built either.
Co-Authored-By: Claude Fable 5 <[email protected]>
---
main/nss/makefile.mk | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/main/nss/makefile.mk b/main/nss/makefile.mk
index afd4ed74b8..106143e7fe 100644
--- a/main/nss/makefile.mk
+++ b/main/nss/makefile.mk
@@ -75,6 +75,11 @@ PATCH_FILES+=nss_linux.patch
BUILD_ACTION+=NSS_DISABLE_GTESTS=1
.ENDIF
+.IF "$(OS)"=="MACOSX"
+# clang 13+ warns on unused-but-set variables, fatal under NSS's default
-Werror
+BUILD_ACTION+=NSS_DISABLE_GTESTS=1 NSS_ENABLE_WERROR=0
+.ENDIF
+
.IF "$(GUI)"=="WNT"
