Tried out Claude on some simple things... used Fable 5!

> On Jun 11, 2026, at 4:30 PM, [email protected] wrote:
> 
> 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"
> 
> 

--
Jim

    "The responsibility is all yours;
     No one can stop you from being honest or straightforward."
                  - Marcus Aurelius

Reply via email to