Merge pull request #3195 from dago/gccflags
authorPavel Rochnyak <pavel2000@ngs.ru>
Fri, 28 Jun 2019 10:57:34 +0000 (17:57 +0700)
committerGitHub <noreply@github.com>
Fri, 28 Jun 2019 10:57:34 +0000 (17:57 +0700)
Use GCC-specific flags only when compiling with GCC

1  2 
configure.ac

diff --combined configure.ac
@@@ -635,6 -635,16 +635,6 @@@ f
  
  # }}}
  
 -# For the dns plugin
 -AC_CHECK_HEADERS([arpa/nameser.h])
 -AC_CHECK_HEADERS([arpa/nameser_compat.h], [], [],
 -  [[
 -    #if HAVE_ARPA_NAMESER_H
 -    # include <arpa/nameser.h>
 -    #endif
 -  ]]
 -)
 -
  AC_CHECK_HEADERS([net/if_arp.h], [], [],
    [[
      #if HAVE_SYS_SOCKET_H
@@@ -767,8 -777,10 +767,10 @@@ AC_CHECK_FUNCS_ONCE([ 
  
  AC_FUNC_STRERROR_R
  
- SAVE_CFLAGS="$CFLAGS"
- CFLAGS="-Wall -Werror"
+ if test "x$GCC" = "xyes"; then
+   SAVE_CFLAGS="$CFLAGS"
+   CFLAGS="$CFLAGS -Wall -Werror"
+ fi
  SAVE_LDFLAGS="$LDFLAGS"
  LDFLAGS=""
  if test "x$ac_system" = "xWindows"; then
@@@ -3858,7 -3870,10 +3860,10 @@@ if test "x$with_libnetsnmp" = "xyes"; t
    SAVE_CPPFLAGS="$CPPFLAGS"
    SAVE_LDFLAGS="$LDFLAGS"
    SAVE_LIBS="$LIBS"
-   CPPFLAGS="$CPPFLAGS $with_libnetsnmp_cppflags -Wall -Werror"
+   CPPFLAGS="$CPPFLAGS $with_libnetsnmp_cppflags"
+   if test "x$GCC" = "xyes"; then
+     CPPFLAGS="$CPPFLAGS -Wall -Werror"
+   fi
    LDFLAGS="$LDFLAGS $with_libnetsnmp_ldflags"
    LIBS="$LIBS -lnetsnmp"
  
@@@ -4321,7 -4336,10 +4326,10 @@@ if test "x$with_libperl" = "xyes"; the
    # (see issues #41 and #42)
    SAVE_CFLAGS="$CFLAGS"
    SAVE_LIBS="$LIBS"
-   CFLAGS="$CFLAGS $PERL_CFLAGS -Wall -Werror"
+   CFLAGS="$CFLAGS $PERL_CFLAGS"
+   if test "x$GCC" = "xyes"; then
+     CFLAGS="$CFLAGS -Wall -Werror"
+   fi
    LIBS="$LIBS $PERL_LIBS"
  
    AC_CACHE_CHECK([for broken Perl_load_module()],
@@@ -7066,11 -7084,13 +7074,13 @@@ AC_SUBST([LCC_VERSION_STRING]
  
  AC_CONFIG_FILES([src/libcollectdclient/collectd/lcc_features.h])
  
- AM_CFLAGS="-Wall"
- AM_CXXFLAGS="-Wall"
- if test "x$enable_werror" != "xno"; then
-   AM_CFLAGS="$AM_CFLAGS -Werror"
-   AM_CXXFLAGS="$AM_CXXFLAGS -Werror"
+ if test "x$GCC" = "xyes"; then
+   AM_CFLAGS="-Wall"
+   AM_CXXFLAGS="-Wall"
+   if test "x$enable_werror" != "xno"; then
+     AM_CFLAGS="$AM_CFLAGS -Werror"
+     AM_CXXFLAGS="$AM_CXXFLAGS -Werror"
+   fi
  fi
  
  AC_SUBST([AM_CFLAGS])