Skip to content

Commit

Permalink
pythongh-116869: Enable -Werror in test_cext for Free Threading (pyth…
Browse files Browse the repository at this point in the history
…on#117106)

Check for warnings, but don't enable the compiler flag
-Werror=declaration-after-statement.
  • Loading branch information
vstinner authored Mar 20, 2024
1 parent 104602a commit f4cc77d
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions Lib/test/test_cext/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -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 = []
Expand Down

0 comments on commit f4cc77d

Please sign in to comment.