git: 50cdf3908631 - 2024Q3 - java/openjdk17: fix build with clang 19
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Wed, 07 Aug 2024 16:52:19 UTC
The branch 2024Q3 has been updated by dim: URL: https://cgit.FreeBSD.org/ports/commit/?id=50cdf3908631cd59cf470624850f5f4bb5bed5a2 commit 50cdf3908631cd59cf470624850f5f4bb5bed5a2 Author: Dimitry Andric <dim@FreeBSD.org> AuthorDate: 2024-08-01 16:05:22 +0000 Commit: Dimitry Andric <dim@FreeBSD.org> CommitDate: 2024-08-07 16:51:46 +0000 java/openjdk17: fix build with clang 19 Clang 19 has a number of new warnings, and because java/openjdk17 compiles with -Werror by default, it results in errors similar to: /wrkdirs/usr/ports/java/openjdk17/work/jdk17u-jdk-17.0.12-7-1/src/hotspot/os/posix/signals_posix.cpp:1681:20: error: cast from 'void (*)(int, siginfo_t *, ucontext_t *)' (aka 'void (*)(int, __siginfo *, __ucontext *)') to 'void (*)(int)' converts to incompatible function type [-Werror,-Wcast-function-type-mismatch] 1681 | act.sa_handler = (void (*)(int)) SR_handler; | ^~~~~~~~~~~~~~~~~~~~~~~~~~ I propose to disable warnings-as-errors for this port, similar to what is already done for openjdk8 and openjdk11. It does not make sense to have this on by default, unless you are an upstream maintainer. PR: 280561 Approved by: glewis (maintainer) MFH: 2024Q3 (cherry picked from commit 4c37f9f5f253a5701fa94ddfc00520889bca9ea4) --- java/openjdk17/Makefile | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/java/openjdk17/Makefile b/java/openjdk17/Makefile index e7039439f653..de71b4b482bb 100644 --- a/java/openjdk17/Makefile +++ b/java/openjdk17/Makefile @@ -139,11 +139,7 @@ CONFIGURE_ARGS+= --with-extra-ldflags="-Wl,-rpath=${LOCALBASE}/lib/gcc${GCC_DEFA --with-extra-cxxflags="-Wl,-rpath=${LOCALBASE}/lib/gcc${GCC_DEFAULT}" .else MAKE_ENV+= USE_CLANG=true -EXTRA_CFLAGS+= -Wno-unused-but-set-parameter -EXTRA_CFLAGS+= -Wno-bitwise-instead-of-logical -EXTRA_CFLAGS+= -Wno-deprecated-non-prototype -CONFIGURE_ARGS+= --with-extra-cflags="${EXTRA_CFLAGS}" -CONFIGURE_ARGS+= --with-extra-cxxflags="${EXTRA_CFLAGS}" +CONFIGURE_ARGS+= --disable-warnings-as-errors .endif .if ${ARCH} == aarch64 || ${ARCH:Mpowerpc64*}