gh-116869: Enable -Werror in test_cext for Free Threading (#117106)

vstinner · web-flow · commit f4cc77d494ee · 2024-03-21T00:06:24.000+01:00
Check for warnings, but don't enable the compiler flag
-Werror=declaration-after-statement.
diff --git a/Lib/test/test_cext/setup.py b/Lib/test/test_cext/setup.py
@@ -11,17 +11,19 @@
 
 
 SOURCE = 'extension.c'
-if not support.MS_WINDOWS and not support.Py_GIL_DISABLED:
+if not support.MS_WINDOWS:
     # C compiler flags for GCC and clang
     CFLAGS = [
         # The purpose of test_cext extension is to check that building a C
         # extension using the Python C API does not emit C compiler warnings.
         '-Werror',
-
-        # gh-116869: The Python C API must be compatible with building
-        # with the -Werror=declaration-after-statement compiler flag.
-        '-Werror=declaration-after-statement',
     ]
+    if not support.Py_GIL_DISABLED:
+        CFLAGS.append(
+            # gh-116869: The Python C API must be compatible with building
+            # with the -Werror=declaration-after-statement compiler flag.
+            '-Werror=declaration-after-statement',
+        )
 else:
     # Don't pass any compiler flag to MSVC
     CFLAGS = []
