Skip to content

Commit

Permalink
build: remove C flags to disable warnings on JDK17 (#3900)
Browse files Browse the repository at this point in the history
  • Loading branch information
gdams authored Aug 2, 2024
1 parent 7dc73f3 commit fa9e245
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions build-farm/platform-specific-configurations/mac.sh
Original file line number Diff line number Diff line change
Expand Up @@ -52,10 +52,10 @@ else
# JDK17 requires metal (included in full xcode) as does JDK11 on aarch64
# JDK11 on x64 is matched for consistency
XCODE_SWITCH_PATH="/Applications/Xcode.app"
# JDK11 and 17 on Mac (x86 and aarch) has excessive warnings.
# JDK11 (x86 and aarch) has excessive warnings.
# This is due to a harfbuzz fix which is pending backport.
# Suppressing the warnings for now to aid triage.
if [[ "$JAVA_FEATURE_VERSION" -le 17 ]]; then
if [[ "$JAVA_FEATURE_VERSION" -le 11 ]]; then
export cxx_flags_bucket="${cxx_flags_bucket} -Wno-deprecated-builtins -Wno-deprecated-declarations -Wno-deprecated-non-prototype"
export c_flags_bucket="${c_flags_bucket} -Wno-deprecated-builtins -Wno-deprecated-declarations -Wno-deprecated-non-prototype"
fi
Expand Down

0 comments on commit fa9e245

Please sign in to comment.