From 27ea8fc2faa4e4753a9cff0a7f48d1dba68013d6 Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Sun, 8 Mar 2020 11:08:13 +0100 Subject: [PATCH] configure: convert -I to -isystem as a last step As all the -I uses in CFLAGS at that point are for system headers and third party libraries this helps us remove/ignore warnings on those! Closes #5060 --- configure.ac | 2 ++ m4/curl-compilers.m4 | 9 ++++----- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/configure.ac b/configure.ac index 4f4aeb91b..973394bce 100755 --- a/configure.ac +++ b/configure.ac @@ -4259,6 +4259,8 @@ if test "$want_thres" = "yes" && test "x$USE_THREADS_POSIX" != "x1"; then fi fi +CURL_CONVERT_INCLUDE_TO_ISYSTEM + dnl ************************************************************ dnl disable verbose text strings dnl diff --git a/m4/curl-compilers.m4 b/m4/curl-compilers.m4 index 79460b6d2..196b285be 100644 --- a/m4/curl-compilers.m4 +++ b/m4/curl-compilers.m4 @@ -452,8 +452,10 @@ dnl GNUC versions these warnings are not silenced. AC_DEFUN([CURL_CONVERT_INCLUDE_TO_ISYSTEM], [ AC_REQUIRE([CURL_SHFUNC_SQUEEZE])dnl AC_REQUIRE([CURL_CHECK_COMPILER])dnl + AC_MSG_CHECKING([convert -I options to -isystem]) if test "$compiler_id" = "GNU_C" || test "$compiler_id" = "CLANG"; then + AC_MSG_RESULT([yes]) tmp_has_include="no" tmp_chg_FLAGS="$CFLAGS" for word1 in $tmp_chg_FLAGS; do @@ -484,6 +486,8 @@ AC_DEFUN([CURL_CONVERT_INCLUDE_TO_ISYSTEM], [ CPPFLAGS="$tmp_chg_FLAGS" squeeze CPPFLAGS fi + else + AC_MSG_RESULT([no]) fi ]) @@ -569,11 +573,6 @@ AC_DEFUN([CURL_SET_COMPILER_BASIC_OPTS], [ AC_REQUIRE([CURL_SHFUNC_SQUEEZE])dnl # if test "$compiler_id" != "unknown"; then - # - if test "$compiler_id" = "GNU_C" || - test "$compiler_id" = "CLANG"; then - CURL_CONVERT_INCLUDE_TO_ISYSTEM - fi # tmp_save_CPPFLAGS="$CPPFLAGS" tmp_save_CFLAGS="$CFLAGS"