Make -Werror optional but enable it for CI. (#6032)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index fd6b65998bf6b94fc709c4da22eadf4fd16fb221..abd47ec97eb2ed2a7ee5f486d05196533704f0e3 100644 (file)
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -155,7 +155,8 @@ jobs:
       run: |
         ${{ matrix.env }} ./configure.sh ${{ matrix.config }} \
           --prefix=$(pwd)/build/install \
-          --unit-testing
+          --unit-testing \
+          --werror
 
     - name: Build
       run: make -j2

diff --git a/CMakeLists.txt b/CMakeLists.txt
index c32c10f4d6d468c36a3094857e4284198f1174b6..b48974cc3321fd33aaee0e75b1c707b33c11fdcb 100644 (file)
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -234,7 +234,6 @@ include(Config${CMAKE_BUILD_TYPE})
 
 add_check_c_cxx_flag("-O${OPTIMIZATION_LEVEL}")
 add_check_c_cxx_flag("-Wall")
-add_check_c_cxx_flag("-Werror")
 add_check_c_cxx_flag("-Wno-unused-private-field")
 add_check_c_flag("-fexceptions")
 add_check_cxx_flag("-Wsuggest-override")

diff --git a/configure.sh b/configure.sh
index 419d66ea8e3b67061233519c47e8fb15ee47f613..1e036e5488b2f973deb451270a028756bd9b8f9b 100755 (executable)
--- a/configure.sh
+++ b/configure.sh
@@ -51,6 +51,7 @@ The following flags enable optional features (disable with --no-<option name>).
   --asan                   build with ASan instrumentation
   --ubsan                  build with UBSan instrumentation
   --tsan                   build with TSan instrumentation
+  --werror                 build with -Werror
 
 Optional Packages:
 The following flags enable optional packages (disable with --no-<option name>).
@@ -150,6 +151,7 @@ ubsan=default
 unit_testing=default
 valgrind=default
 win64=default
+werror=default
 
 abc_dir=default
 antlr_dir=default
@@ -186,6 +188,8 @@ do
     --tsan) tsan=ON;;
     --no-tsan) tsan=OFF;;
 
+    --werror) werror=ON;;
+
     --assertions) assertions=ON;;
     --no-assertions) assertions=OFF;;
 
@@ -355,6 +359,11 @@ done
 
 #--------------------------------------------------------------------------#
 
+if [ $werror != default ]; then
+  export CFLAGS=-Werror
+  export CXXFLAGS=-Werror
+fi
+
 cmake_opts=""
 
 [ $buildtype != default ] \