commit 90a3cf570a91ace5bd730a35a6f30a24e26267a0
Author: Scott Kostyshak <[email protected]>
Date:   Mon Jun 16 10:47:01 2025 +0200

    lyxbuild: CMake now uses '-Wno-nested-anon-types'
    
    As of dd4f9670, CMake handles the flag, which is better than doing
    it in lyxbuild.
---
 development/lyx-tester/lyxbuild | 18 +++++++++++++-----
 1 file changed, 13 insertions(+), 5 deletions(-)

diff --git a/development/lyx-tester/lyxbuild b/development/lyx-tester/lyxbuild
index 6d2035460f..1f310f8e69 100755
--- a/development/lyx-tester/lyxbuild
+++ b/development/lyx-tester/lyxbuild
@@ -522,16 +522,24 @@ else
 
 
   if [ "${WERROR}" = "1" ]; then
-    # should be a semi-colon separated list (if more than one option).
-    #
+    # CPPFLAGS should be a semi-colon separated list (if more than one option).
+
     # '-pedantic' warnings aren't too useful, but solving them
     # may make compilation smoother for compilers that LyX
     # developers are not testing.
     #
+    # If a warning from '-pedantic' comes up, we can ignore certain classes of
+    # warnings. e.g., see dd4f9670 for how we now append
+    # '-Wno-nested-anon-types' (which I think is the same as
+    # '-Wno-error=nested-anon-types')
+    #
+    # An alternative to dd4f9670 is handling it in 'lyxbuild' by adding the
+    # following *after* the initial CPPFLAGS definition below:
+    #if [ "${clang}" = "1" ]; then
+    #  CPPFLAGS+=";-Wno-error=nested-anon-types"
+    #fi
+
     CPPFLAGS="-Werror;-Wextra;-pedantic"
-    if [ "${clang}" = "1" ]; then
-      CPPFLAGS+=";-Wno-error=nested-anon-types"
-    fi
   else
     CPPFLAGS=""
   fi
-- 
lyx-cvs mailing list
[email protected]
https://lists.lyx.org/mailman/listinfo/lyx-cvs

Reply via email to