This is an automated email from the ASF dual-hosted git repository.
jvanderzee pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/master by this push:
new 82fad6c616 Set -Werror=no-stringop-overflow for boringssl (#11602)
82fad6c616 is described below
commit 82fad6c6165fa2c7de9f8eb9af432fc9dc2c35f4
Author: JosiahWI <[email protected]>
AuthorDate: Thu Jul 25 06:11:41 2024 -0500
Set -Werror=no-stringop-overflow for boringssl (#11602)
* Set -Werror=no-stringop-overflow for boringssl
Fixes #11592.
---
tools/build_boringssl_h3_tools.sh | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/tools/build_boringssl_h3_tools.sh
b/tools/build_boringssl_h3_tools.sh
index 8c5979619f..809cdd3167 100755
--- a/tools/build_boringssl_h3_tools.sh
+++ b/tools/build_boringssl_h3_tools.sh
@@ -141,6 +141,9 @@ set -e
# Note: -UBORINGSSL_HAVE_LIBUNWIND
# Disable related libunwind test builds, there are some version number issues
# with this pkg in Ubuntu 20.04, so disable this to make sure it builds.
+# Note: -Wno-error=stringop-overflow
+# GCC 13 has a stringop-overflow false positive. It only affects the build
+# of the static lib.
cmake \
-B build-shared \
-DGO_EXECUTABLE=${GO_BINARY_PATH} \
@@ -155,7 +158,7 @@ cmake \
-DCMAKE_INSTALL_PREFIX=${BASE}/boringssl \
-DCMAKE_BUILD_TYPE=Release \
-DCMAKE_CXX_FLAGS='-Wno-error=ignored-attributes -UBORINGSSL_HAVE_LIBUNWIND'
\
- -DCMAKE_C_FLAGS=${BSSL_C_FLAGS} \
+ -DCMAKE_C_FLAGS="-Wno-error=stringop-overflow ${BSSL_C_FLAGS}" \
-DBUILD_SHARED_LIBS=0
cmake --build build-shared -j ${num_threads}
cmake --build build-static -j ${num_threads}
