Mirrors/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf.git synced 2026-10-02 11:10:54 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix(build): set -Wunused-but-set-variable=1 for GNU compiler to maintain backward compatibility

Browse Source
This commit is contained in:
Alexey Lapshin
2026-06-17 17:24:25 +07:00
parent 7d8c01b879
commit 71618c88e3
3 changed files with 22 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
CMakeLists.txt
+6
View File
@@ -104,6 +104,12 @@ if(CMAKE_C_COMPILER_ID MATCHES "GNU")
list(APPEND c_compile_options "-Wno-old-style-declaration") list(APPEND c_compile_options "-Wno-old-style-declaration")
endif() endif()
# TODO IDF-15784: remove in IDF v7.0 ?
check_c_compiler_flag("-Wunused-but-set-variable=1" compiler_supports_wunused_but_set_variable_eq_1)
if(compiler_supports_wunused_but_set_variable_eq_1)
list(APPEND compile_options "-Wunused-but-set-variable=1")
endif()
# Clang finds some warnings in IDF code which GCC doesn't. # Clang finds some warnings in IDF code which GCC doesn't.
# All these warnings should be fixed before Clang is presented # All these warnings should be fixed before Clang is presented
# as a toolchain choice for users. # as a toolchain choice for users.
tools/ci/configure_ci_environment.sh
+10 -1
View File
@@ -17,8 +17,17 @@ DEBUG_SHELL=${DEBUG_SHELL:-"0"}
# (Depends on default options '-Wno-error=XXX' used in the IDF build system) # (Depends on default options '-Wno-error=XXX' used in the IDF build system)
if [ "$IDF_TOOLCHAIN" != "clang" ]; then if [ "$IDF_TOOLCHAIN" != "clang" ]; then
PEDANTIC_FLAGS="-Werror -Werror=deprecated-declarations -Werror=unused-variable -Werror=unused-but-set-variable -Werror=unused-function" PEDANTIC_FLAGS="-Werror -Werror=deprecated-declarations -Werror=unused-variable -Werror=unused-function"
export PEDANTIC_CFLAGS="${PEDANTIC_FLAGS} -Wstrict-prototypes" export PEDANTIC_CFLAGS="${PEDANTIC_FLAGS} -Wstrict-prototypes"
# TODO IDF-15784
case "$CI_JOB_NAME" in
test_pytest_linux|test_sockets_on_host|test_transport_on_host|test_pytest_macos)
export PEDANTIC_CFLAGS="${PEDANTIC_CFLAGS} -Werror=unused-but-set-variable"
;;
*)
export PEDANTIC_CFLAGS="${PEDANTIC_CFLAGS} -Werror=unused-but-set-variable=1"
;;
esac
else else
export PEDANTIC_CFLAGS="-Werror" export PEDANTIC_CFLAGS="-Werror"
fi fi
tools/cmakev2/project.cmake
+6
View File
@@ -236,6 +236,12 @@ function(__init_project_configuration)
list(APPEND c_compile_options "-Wno-old-style-declaration") list(APPEND c_compile_options "-Wno-old-style-declaration")
endif() endif()
# TODO IDF-15784: remove in IDF v7.0 ?
check_c_compiler_flag("-Wunused-but-set-variable=1" compiler_supports_wunused_but_set_variable_eq_1)
if(compiler_supports_wunused_but_set_variable_eq_1)
list(APPEND compile_options "-Wunused-but-set-variable=1")
endif()
# Clang finds some warnings in IDF code which GCC doesn't. # Clang finds some warnings in IDF code which GCC doesn't.
# All these warnings should be fixed before Clang is presented # All these warnings should be fixed before Clang is presented
# as a toolchain choice for users. # as a toolchain choice for users.