]> git.ipfire.org Git - thirdparty/kea.git/commitdiff
[4283] Moved clang and missing initializer -Wno-* to improved/fixed configure
authorFrancis Dupont <fdupont@isc.org>
Sat, 2 Jul 2016 09:00:26 +0000 (11:00 +0200)
committerFrancis Dupont <fdupont@isc.org>
Sat, 2 Jul 2016 09:00:26 +0000 (11:00 +0200)
32 files changed:
configure.ac
src/bin/d2/Makefile.am
src/bin/d2/tests/Makefile.am
src/bin/dhcp4/Makefile.am
src/bin/dhcp4/tests/Makefile.am
src/bin/dhcp6/Makefile.am
src/bin/dhcp6/tests/Makefile.am
src/bin/lfc/Makefile.am
src/bin/lfc/tests/Makefile.am
src/bin/perfdhcp/Makefile.am
src/bin/perfdhcp/tests/Makefile.am
src/hooks/dhcp/user_chk/Makefile.am
src/hooks/dhcp/user_chk/tests/Makefile.am
src/lib/asiodns/Makefile.am
src/lib/asiolink/Makefile.am
src/lib/asiolink/tests/Makefile.am
src/lib/cc/Makefile.am
src/lib/dhcp/Makefile.am
src/lib/dhcp/tests/Makefile.am
src/lib/dhcp_ddns/Makefile.am
src/lib/dhcp_ddns/tests/Makefile.am
src/lib/dhcpsrv/Makefile.am
src/lib/dhcpsrv/tests/Makefile.am
src/lib/dns/Makefile.am
src/lib/eval/Makefile.am
src/lib/eval/tests/Makefile.am
src/lib/hooks/Makefile.am
src/lib/hooks/tests/Makefile.am
src/lib/log/compiler/Makefile.am
src/lib/log/tests/Makefile.am
src/lib/stats/Makefile.am
src/lib/util/Makefile.am

index 150fffc772c76817db2d9a746a02db5b35326ca9..535201cf3025499da79aae6177e4f50abcf62bdd 100755 (executable)
@@ -106,6 +106,7 @@ AC_LANG([C++])
 AM_CONDITIONAL(USE_GXX, test "X${GXX}" = "Xyes")
 AC_CHECK_DECL([__SUNPRO_CC], [SUNCXX="yes"], [SUNCXX="no"])
 AC_CHECK_DECL([__clang__], [CLANGPP="yes"], [CLANGPP="no"])
+# USE_CLANGPP is no longer used, keep it by summetry with USE_GXX?
 AM_CONDITIONAL(USE_CLANGPP, test "X${CLANGPP}" = "Xyes")
 
 dnl Determine if we are using GNU sed
@@ -136,9 +137,9 @@ AC_DEFUN([KEA_CXX_TRY_FLAG], [
   AC_MSG_CHECKING([whether $CXX supports $1])
 
   kea_save_CXXFLAGS="$CXXFLAGS"
-  CXXFLAGS="$CXXFLAGS $1"
+  CXXFLAGS="$CXXFLAGS -Werror $1"
 
-  AC_LINK_IFELSE([AC_LANG_SOURCE([int main(void){ return 0;}])],
+  AC_LINK_IFELSE([AC_LANG_SOURCE([int main(){ return 0;}])],
                  [kea_cxx_flag=yes], [kea_cxx_flag=no])
   CXXFLAGS="$kea_save_CXXFLAGS"
 
@@ -169,16 +170,12 @@ fi
 # happen if clang++ is used via ccache.  So, although probably suboptimal,
 # we suppress this particular warning.  Note that it doesn't weaken checks
 # on the source code.
-if test "$CLANGPP" = "yes"; then
+if test "X$CLANGPP" = "Xyes"; then
 CXX_VERSION=`$CXX --version 2> /dev/null | head -1`
 KEA_CXXFLAGS="$KEA_CXXFLAGS -Qunused-arguments"
 fi
 
-KEA_CXX_TRY_FLAG([-Wno-missing-field-initializers],
-       [WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG="-Wno-missing-field-initializers"])
-AC_SUBST(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
-
-# gcc specific settings:
+# gcc/clang specific settings:
 if test "X$GXX" = "Xyes"; then
 CXX_VERSION=`$CXX --version 2> /dev/null | head -1`
 KEA_CXXFLAGS="$KEA_CXXFLAGS -Wall -Wextra -Wnon-virtual-dtor -Wwrite-strings -Woverloaded-virtual -Wno-sign-compare"
@@ -206,13 +203,15 @@ esac
 KEA_CXXFLAGS="$KEA_CXXFLAGS $MULTITHREADING_FLAG"
 
 # Don't use -Werror if configured not to
+werror_extras=
 AC_ARG_WITH(werror,
     AC_HELP_STRING([--with-werror], [Compile using -Werror (default=yes)]),
     [
      case "${withval}" in
          yes) with_werror=1 ;;
          no)  with_werror=0 ;;
-         *)   AC_MSG_ERROR(bad value ${withval} for --with-werror) ;;
+         -*)  with_werror=1; werror_extras=${withval} ;;
+        *)   AC_MSG_ERROR(bad value ${withval} for --with-werror) ;;
      esac],
      [with_werror=1])
 
@@ -236,6 +235,30 @@ if test $with_werror = 1; then
    CXXFLAGS="$CXXFLAGS_SAVED"
 fi
 
+# Added flags after -Werror
+
+# Some versions of GCC warn about some versions of Boost regarding
+# missing initializer for members in its posix_time.
+# https://svn.boost.org/trac/boost/ticket/3477
+# But older GCC compilers don't have the flag.
+KEA_CXX_TRY_FLAG([-Wno-missing-field-initializers],
+       [KEA_CXXFLAGS="$KEA_CXXFLAGS -Wno-missing-field-initializers"])
+
+if test "X$CLANGPP" = "Xyes"; then
+       # This is to workaround unused variables tcout and tcerr in
+       # log4cplus's streams.h and unused parameters from some of the
+       # Boost headers.
+       KEA_CXXFLAGS="$KEA_CXXFLAGS -Wno-unused-variable -Wno-unused-parameter"
+fi
+
+# Add the extras at the very last
+# Note it can be used to re-enable a (fatal) warning
+for extra in $werror_extras; do
+       KEA_CXX_TRY_FLAG([$extra],
+               [KEA_CXXFLAGS="$KEA_CXXFLAGS $extra"],
+               [AC_MSG_ERROR([$CXX does not support $extra"])])
+done
+
 fi                             dnl GXX = yes
 
 # allow building programs with static link.  we need to make it selective
@@ -1114,7 +1137,7 @@ LIBS=$LIBS_SAVED
 AX_BOOST_FOR_KEA
 # Boost offset_ptr is required in one library and not optional right now, so
 # we unconditionally fail here if it doesn't work.
-if test "$BOOST_OFFSET_PTR_WOULDFAIL" = "yes" -a "$werror_ok" = 1; then
+if test "$BOOST_OFFSET_PTR_WOULDFAIL" = "yes" -a X"$werror_ok" = X1; then
     AC_MSG_ERROR([Failed to compile a required header file.  Try upgrading Boost to 1.44 or higher (when using clang++) or specifying --without-werror.  See the ChangeLog entry for Trac no. 2147 for more details.])
 fi
 
index df266ad4a14e88eb9f9b734f613710ad2b2c3e93..0b4e7d58626583c41fe3945fab6a6f6a92ed1f33 100644 (file)
@@ -14,10 +14,6 @@ AM_CPPFLAGS += $(CQL_CPPFLAGS)
 endif
 
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
-if USE_CLANGPP
-# Disable unused parameter warning caused by some Boost headers when compiling with clang
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
 
 if USE_STATIC_LINK
 AM_LDFLAGS = -static
index 6c5dbe4866134832c0d2e79bd11e0e697b040c5a..dc18d81d5edbdfea0b560d0b536f13297c413ed1 100644 (file)
@@ -29,10 +29,6 @@ CLEANFILES = $(builddir)/interfaces.txt $(builddir)/logger_lockfile
 DISTCLEANFILES = d2_process_tests.sh
 
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
-if USE_CLANGPP
-# Disable unused parameter warning caused by some Boost headers when compiling with clang
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
 
 if USE_STATIC_LINK
 AM_LDFLAGS = -static
index 53423e149d6b7ca3e2892b11b3d908d1c574d041..1b8091d415bb123ef46e993e40da69db5435c745 100644 (file)
@@ -15,10 +15,6 @@ AM_CPPFLAGS += $(CQL_CPPFLAGS)
 endif
 
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
-if USE_CLANGPP
-# Disable unused parameter warning caused by some Boost headers when compiling with clang
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
 
 if USE_STATIC_LINK
 AM_LDFLAGS = -static
index ffbf3a3d8d6691f11bcfbba3f99bd08ba57413e7..9f658e14fcd1abc04f1d6221e5b422c3af4bdd53 100644 (file)
@@ -32,10 +32,6 @@ CLEANFILES += *.json *.log
 DISTCLEANFILES = dhcp4_process_tests.sh
 
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
-if USE_CLANGPP
-# Disable unused parameter warning caused by some Boost headers when compiling with clang
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
 
 if USE_STATIC_LINK
 AM_LDFLAGS = -static
index baaf1eae73e8fa42c20bc948cc3ba5827159dbdf..7d335224cf8ebc1925d25ee13893def821c008a3 100644 (file)
@@ -15,10 +15,6 @@ AM_CPPFLAGS += $(CQL_CPPFLAGS)
 endif
 
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
-if USE_CLANGPP
-# Disable unused parameter warning caused by some Boost headers when compiling with clang
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
 
 if USE_STATIC_LINK
 AM_LDFLAGS = -static
index 3573b4ad63d0c95fe1c2015388ad48a07cb55c9f..a85119cc9a4b6bb834daa6d7e018e574ea91469b 100644 (file)
@@ -33,10 +33,6 @@ CLEANFILES += *.json *.log
 DISTCLEANFILES = dhcp6_process_tests.sh
 
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
-if USE_CLANGPP
-# Disable unused parameter warning caused by some Boost headers when compiling with clang
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
 
 if USE_STATIC_LINK
 AM_LDFLAGS = -static
index 9cdd5bae7826f7e691d4af6919057b761e5c069a..7bfeb96df79e136afbd3051fcc6648ae575338fd 100644 (file)
@@ -3,12 +3,7 @@ SUBDIRS = . tests
 AM_CPPFLAGS = -I$(top_srcdir)/src/lib -I$(top_builddir)/src/lib
 AM_CPPFLAGS += -I$(top_srcdir)/src/bin -I$(top_builddir)/src/bin
 AM_CPPFLAGS += $(BOOST_INCLUDES)
-
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
-if USE_CLANGPP
-# Disable unused parameter warning caused by some Boost headers when compiling with clang
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
 
 if USE_STATIC_LINK
 AM_LDFLAGS = -static
index 208f97052080e5535a174360c49dbc6edec476eb..ee20ebf94babbc9670994bf5f1724d24ba981707 100644 (file)
@@ -24,10 +24,6 @@ CLEANFILES = $(builddir)/interfaces.txt $(builddir)/logger_lockfile
 DISTCLEANFILES = 
 
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
-if USE_CLANGPP
-# Disable unused parameter warning caused by some Boost headers when compiling with clang
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
 
 if USE_STATIC_LINK
 AM_LDFLAGS = -static
index bd737903b442e063ec21feae9e3bef7c5a54a996..74d2975babb663f00bc09ebdae8414aa645f1927 100644 (file)
@@ -6,23 +6,11 @@ AM_CPPFLAGS += $(BOOST_INCLUDES)
 
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
 
-# Some versions of GCC warn about some versions of Boost regarding
-# missing initializer for members in its posix_time.
-# https://svn.boost.org/trac/boost/ticket/3477
-# But older GCC compilers don't have the flag.
-AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
-
-if USE_CLANGPP
-# Workaround unused parameters in boost version included in FreeBSD 10
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
-
 AM_LDFLAGS = -lm
 if USE_STATIC_LINK
 AM_LDFLAGS += -static
 endif
 
-
 # convenience archive
 
 noinst_LTLIBRARIES = libperfdhcp.la
@@ -38,16 +26,8 @@ libperfdhcp_la_SOURCES += rate_control.cc rate_control.h
 libperfdhcp_la_SOURCES += stats_mgr.h
 libperfdhcp_la_SOURCES += test_control.cc test_control.h
 
-libperfdhcp_la_CXXFLAGS = $(AM_CXXFLAGS)
-if USE_CLANGPP
-# Disable unused parameter warning caused by some of the
-# Boost headers when compiling with clang.
-libperfdhcp_la_CXXFLAGS += -Wno-unused-parameter
-endif
-
 sbin_PROGRAMS = perfdhcp
 perfdhcp_SOURCES = main.cc
-libkea_perfdhcp___la_CXXFLAGS = $(AM_CXXFLAGS)
 
 perfdhcp_LDFLAGS = $(AM_LDFLAGS) $(CRYPTO_LDFLAGS)
 
index ab0a914fa3acd39d2086dbf28876d1b74e662638..b41141ac6d561c4663821c47442ff1b0ae5c5418 100644 (file)
@@ -35,12 +35,6 @@ run_unittests_SOURCES += command_options_helper.h
 run_unittests_CPPFLAGS = $(AM_CPPFLAGS) $(GTEST_INCLUDES)
 run_unittests_LDFLAGS  = $(AM_LDFLAGS) $(CRYPTO_LDFLAGS) $(GTEST_LDFLAGS)
 
-if USE_CLANGPP
-# Disable unused parameter warning caused by some of the
-# Boost headers when compiling with clang.
-run_unittests_CXXFLAGS = -Wno-unused-parameter
-endif
-
 run_unittests_LDADD  = $(top_builddir)/src/bin/perfdhcp/libperfdhcp.la
 run_unittests_LDADD += $(top_builddir)/src/lib/cfgrpt/libcfgrpt.la
 run_unittests_LDADD += $(top_builddir)/src/lib/dhcp/libkea-dhcp++.la
index 3636a0bebb54d171a6f9811fb63cd6a292b7ec75..41ad7b2e3cfcf7a323d1a4b463f53c46cf8b03b3 100644 (file)
@@ -4,12 +4,6 @@ AM_CPPFLAGS  = -I$(top_builddir)/src/lib -I$(top_srcdir)/src/lib
 AM_CPPFLAGS += $(BOOST_INCLUDES)
 AM_CXXFLAGS  = $(KEA_CXXFLAGS)
 
-# Some versions of GCC warn about some versions of Boost regarding
-# missing initializer for members in its posix_time.
-# https://svn.boost.org/trac/boost/ticket/3477
-# But older GCC compilers don't have the flag.
-AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
-
 # Define rule to build logging source files from message file
 user_chk_messages.h user_chk_messages.cc: s-messages
 s-messages: user_chk_messages.mes
@@ -48,11 +42,6 @@ nodist_libduc_la_SOURCES = user_chk_messages.cc user_chk_messages.h
 
 libduc_la_CXXFLAGS = $(AM_CXXFLAGS)
 libduc_la_CPPFLAGS = $(AM_CPPFLAGS)
-if USE_CLANGPP
-# Disable unused parameter warning caused by some of the
-# Boost headers when compiling with clang.
-libduc_la_CXXFLAGS += -Wno-unused-parameter
-endif
 
 noinst_LTLIBRARIES += libdhcp_user_chk.la
 
index 145e3b11794103e2ab57dce7638f362c7a8d2229..fe1960d745812ecc263b1646f2d83f8fa9e1e134 100644 (file)
@@ -8,12 +8,6 @@ AM_CPPFLAGS += -DINSTALL_PROG=\"$(abs_top_srcdir)/install-sh\"
 
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
 
-# Some versions of GCC warn about some versions of Boost regarding
-# missing initializer for members in its posix_time.
-# https://svn.boost.org/trac/boost/ticket/3477
-# But older GCC compilers don't have the flag.
-AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
-
 USER_CHK_LIB = $(top_builddir)/src/hooks/dhcp/user_chk/libdhcp_user_chk.la
 
 if USE_STATIC_LINK
@@ -44,12 +38,6 @@ libdhcp_user_chk_unittests_CPPFLAGS = $(AM_CPPFLAGS) $(GTEST_INCLUDES) $(LOG4CPL
 libdhcp_user_chk_unittests_LDFLAGS  = $(AM_LDFLAGS) $(CRYPTO_LDFLAGS) $(GTEST_LDFLAGS)
 
 libdhcp_user_chk_unittests_CXXFLAGS = $(AM_CXXFLAGS)
-if USE_CLANGPP
-# This is to workaround unused variables tcout and tcerr in
-# log4cplus's streams.h and unused parameters from some of the
-# Boost headers.
-libdhcp_user_chk_unittests_CXXFLAGS += -Wno-unused-parameter
-endif
 
 libdhcp_user_chk_unittests_LDADD = $(top_builddir)/src/hooks/dhcp/user_chk/libduc.la
 libdhcp_user_chk_unittests_LDADD += $(top_builddir)/src/lib/dhcp/libkea-dhcp++.la
index 3dfd000e8c049d225c2164ab676d54d269fab49d..24dfe05c30cf11a53251de2f5a8e04c3265333be 100644 (file)
@@ -26,12 +26,8 @@ EXTRA_DIST = asiodns_messages.mes
 
 # Note: the ordering matters: -Wno-... must follow -Wextra (defined in
 # KEA_CXXFLAGS)
-libkea_asiodns_la_CPPFLAGS = $(AM_CPPFLAGS)
 libkea_asiodns_la_CXXFLAGS = $(AM_CXXFLAGS)
-if USE_CLANGPP
-# Avoid ASIO header unused-paramet problem with clang
-libkea_asiodns_la_CXXFLAGS += -Wno-unused-parameter
-endif
+libkea_asiodns_la_CPPFLAGS = $(AM_CPPFLAGS)
 libkea_asiodns_la_LIBADD  = $(top_builddir)/src/lib/asiolink/libkea-asiolink.la
 libkea_asiodns_la_LIBADD += $(top_builddir)/src/lib/log/libkea-log.la
 libkea_asiodns_la_LIBADD += $(top_builddir)/src/lib/util/threads/libkea-threads.la
index cc2fa4185236b395fef01ae46aaeb53674a8a806..806aef7c0f96a1c295cda9ed9d887ff288078959 100644 (file)
@@ -9,18 +9,9 @@ CLEANFILES = *.gcno *.gcda
 
 # This is a wrapper library.
 
-libkea_asiolink_la_CPPFLAGS = $(AM_CPPFLAGS)
-# Note: the ordering matters: -Wno-... must follow -Wextra (defined in
-# KEA_CXXFLAGS)
-libkea_asiolink_la_CXXFLAGS = $(AM_CXXFLAGS)
-
 # The ASIO header files have some code fragments that would hit
 # gcc's unused-parameter warning, which would make the build fail
 # with -Werror (our default setting).
-if USE_CLANGPP
-# Avoid this problem with clang
-libkea_asiolink_la_CXXFLAGS += -Wno-unused-parameter
-endif
 
 lib_LTLIBRARIES = libkea-asiolink.la
 
@@ -41,6 +32,10 @@ libkea_asiolink_la_SOURCES += tcp_socket.h
 libkea_asiolink_la_SOURCES += udp_endpoint.h
 libkea_asiolink_la_SOURCES += udp_socket.h
 
+# Note: the ordering matters: -Wno-... must follow -Wextra (defined in
+# KEA_CXXFLAGS)
+libkea_asiolink_la_CXXFLAGS = $(AM_CXXFLAGS)
+libkea_asiolink_la_CPPFLAGS = $(AM_CPPFLAGS)
 libkea_asiolink_la_LIBADD  = $(top_builddir)/src/lib/exceptions/libkea-exceptions.la
 libkea_asiolink_la_LIBADD += $(BOOST_LIBS)
 
index c3be47f887d756c85afbe6cfba714007fa6b5ba7..6e5592a566b61333a5796c0dccd7f08f75a82419 100644 (file)
@@ -8,12 +8,6 @@ if USE_STATIC_LINK
 AM_LDFLAGS = -static
 endif
 
-# Some versions of GCC warn about some versions of Boost regarding
-# missing initializer for members in its posix_time.
-# https://svn.boost.org/trac/boost/ticket/3477
-# But older GCC compilers don't have the flag.
-AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
-
 CLEANFILES = *.gcno *.gcda
 
 TESTS_ENVIRONMENT = \
index 436097a08c1282a439703d6742d67d7eecb4c6f9..8a734761f8b9b0aa5cbc1afa33a4aa6ad9d0458b 100644 (file)
@@ -4,11 +4,6 @@ AM_CPPFLAGS = -I$(top_srcdir)/src/lib -I$(top_builddir)/src/lib
 AM_CPPFLAGS += $(BOOST_INCLUDES)
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
 
-if USE_CLANGPP
-# Workaround unused parameters in boost version included in FreeBSD 10
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
-
 lib_LTLIBRARIES = libkea-cc.la
 libkea_cc_la_SOURCES = data.cc data.h
 libkea_cc_la_SOURCES += command_interpreter.cc command_interpreter.h
index 72c40835454c5c061713344ee738c52323f803d6..ccdb74b2ab6f8b2e906fbd0835174f9ac28b4e3b 100644 (file)
@@ -4,12 +4,6 @@ AM_CPPFLAGS = -I$(top_builddir)/src/lib -I$(top_srcdir)/src/lib
 AM_CPPFLAGS += $(BOOST_INCLUDES)
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
 
-# Some versions of GCC warn about some versions of Boost regarding
-# missing initializer for members in its posix_time.
-# https://svn.boost.org/trac/boost/ticket/3477
-# But older GCC compilers don't have the flag.
-AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
-
 CLEANFILES = *.gcno *.gcda
 
 lib_LTLIBRARIES = libkea-dhcp++.la
@@ -113,9 +107,3 @@ libkea_dhcp___include_HEADERS = \
     pkt_filter_lpf.h \
     protocol_util.h \
     std_option_defs.h
-
-if USE_CLANGPP
-# Disable unused parameter warning caused by some of the
-# Boost headers when compiling with clang.
-libkea_dhcp___la_CXXFLAGS += -Wno-unused-parameter
-endif
index 05928bd7e61212fd2d6d976ed53378de2a1730eb..c0c584e712aecc4a609a19605314db7e1a3aabe5 100644 (file)
@@ -7,12 +7,6 @@ AM_CPPFLAGS += -DINSTALL_PROG=\"$(abs_top_srcdir)/install-sh\"
 
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
 
-# Some versions of GCC warn about some versions of Boost regarding
-# missing initializer for members in its posix_time.
-# https://svn.boost.org/trac/boost/ticket/3477
-# But older GCC compilers don't have the flag.
-AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
-
 if USE_STATIC_LINK
 AM_LDFLAGS = -static
 endif
@@ -102,12 +96,6 @@ libdhcp___unittests_CPPFLAGS = $(AM_CPPFLAGS) $(GTEST_INCLUDES)
 libdhcp___unittests_LDFLAGS  = $(AM_LDFLAGS) $(CRYPTO_LDFLAGS) $(GTEST_LDFLAGS)
 
 libdhcp___unittests_CXXFLAGS = $(AM_CXXFLAGS)
-if USE_CLANGPP
-# This is to workaround unused variables tcout and tcerr in
-# log4cplus's streams.h and unused parameters from some of the
-# Boost headers.
-libdhcp___unittests_CXXFLAGS += -Wno-unused-variable -Wno-unused-parameter
-endif
 
 libdhcp___unittests_LDADD  = $(top_builddir)/src/lib/dhcp/libkea-dhcp++.la
 libdhcp___unittests_LDADD += $(top_builddir)/src/lib/asiolink/libkea-asiolink.la
index e229943f632fdb3ab334dd160f758bee02e3c6e1..b909bd50b8cf03b58a3adf7be41aa1506aac381a 100644 (file)
@@ -4,13 +4,6 @@ AM_CPPFLAGS  = -I$(top_builddir)/src/lib -I$(top_srcdir)/src/lib
 AM_CPPFLAGS += $(BOOST_INCLUDES)
 AM_CXXFLAGS  = $(KEA_CXXFLAGS)
 
-# Some versions of GCC warn about some versions of Boost regarding
-# missing initializer for members in its posix_time.
-# https://svn.boost.org/trac/boost/ticket/3477
-# But older GCC compilers don't have the flag.
-AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
-
-
 # Define rule to build logging source files from message file
 dhcp_ddns_messages.h dhcp_ddns_messages.cc: s-messages
 
@@ -54,12 +47,4 @@ libkea_dhcp_ddns_la_LIBADD += $(top_builddir)/src/lib/log/libkea-log.la
 libkea_dhcp_ddns_la_LIBADD += $(top_builddir)/src/lib/util/threads/libkea-threads.la
 libkea_dhcp_ddns_la_LIBADD += $(top_builddir)/src/lib/util/libkea-util.la
 libkea_dhcp_ddns_la_LIBADD += $(top_builddir)/src/lib/exceptions/libkea-exceptions.la
-
 libkea_dhcp_ddns_la_LIBADD += $(LOG4CPLUS_LIBS) $(CRYPTO_LIBS)
-
-if USE_CLANGPP
-# Disable unused parameter warning caused by some of the
-# Boost headers when compiling with clang.
-libkea_dhcp_ddns_la_CXXFLAGS += -Wno-unused-parameter
-endif
-
index 8a0731de312d59d2b56fa175896bc857d2fbda18..b0538f5564fb713a4024f6f1713589c5dbc9ddfe 100644 (file)
@@ -7,12 +7,6 @@ AM_CPPFLAGS += -DINSTALL_PROG=\"$(abs_top_srcdir)/install-sh\"
 
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
 
-# Some versions of GCC warn about some versions of Boost regarding
-# missing initializer for members in its posix_time.
-# https://svn.boost.org/trac/boost/ticket/3477
-# But older GCC compilers don't have the flag.
-AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
-
 if USE_STATIC_LINK
 AM_LDFLAGS = -static
 endif
@@ -33,15 +27,9 @@ libdhcp_ddns_unittests_SOURCES += test_utils.cc test_utils.h
 
 libdhcp_ddns_unittests_CPPFLAGS = $(AM_CPPFLAGS) $(GTEST_INCLUDES)
 
-libdhcp_ddns_unittests_LDFLAGS  = $(AM_LDFLAGS) $(CRYPTO_LDFLAGS) $(GTEST_LDFLAGS)
-
 libdhcp_ddns_unittests_CXXFLAGS = $(AM_CXXFLAGS)
-if USE_CLANGPP
-# This is to workaround unused variables tcout and tcerr in
-# log4cplus's streams.h and unused parameters from some of the
-# Boost headers.
-libdhcp_ddns_unittests_CXXFLAGS += -Wno-unused-parameter
-endif
+
+libdhcp_ddns_unittests_LDFLAGS  = $(AM_LDFLAGS) $(CRYPTO_LDFLAGS) $(GTEST_LDFLAGS)
 
 libdhcp_ddns_unittests_LDADD = $(top_builddir)/src/lib/dhcp_ddns/libkea-dhcp_ddns.la
 libdhcp_ddns_unittests_LDADD += $(top_builddir)/src/lib/config/libkea-cfgclient.la
index 362169052d9c51edc2f25b2f9bf77ed340972881..70b1152954ddf21475277deb4f3cb7a17567affa 100755 (executable)
@@ -67,12 +67,6 @@ BUILT_SOURCES = alloc_engine_messages.h alloc_engine_messages.cc
 BUILT_SOURCES += dhcpsrv_messages.h dhcpsrv_messages.cc
 BUILT_SOURCES += hosts_messages.h hosts_messages.cc
 
-# Some versions of GCC warn about some versions of Boost regarding
-# missing initializer for members in its posix_time.
-# https://svn.boost.org/trac/boost/ticket/3477
-# But older GCC compilers don't have the flag.
-AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
-
 # Make sure the generated files are deleted in a "clean" operation
 CLEANFILES = *.gcno *.gcda
 CLEANFILES += alloc_engine_messages.h alloc_engine_messages.cc
@@ -208,12 +202,6 @@ if HAVE_CQL
 libkea_dhcpsrv_la_LDFLAGS += $(CQL_LIBS)
 endif
 
-if USE_CLANGPP
-# Disable unused parameter warning caused by some of the
-# Boost headers when compiling with clang.
-libkea_dhcpsrv_la_CXXFLAGS += -Wno-unused-parameter
-endif
-
 # The message file should be in the distribution
 EXTRA_DIST += alloc_engine_messages.mes
 EXTRA_DIST += dhcpsrv_messages.mes
index aea9898f7cd8d75e99a48ade2a3a9646d5dcf23d..2d65439e4dd2c9de538444008339fc9dbdb571f2 100755 (executable)
@@ -14,12 +14,6 @@ if USE_STATIC_LINK
 AM_LDFLAGS = -static
 endif
 
-# Some versions of GCC warn about some versions of Boost regarding
-# missing initializer for members in its posix_time.
-# https://svn.boost.org/trac/boost/ticket/3477
-# But older GCC compilers don't have the flag.
-AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
-
 CLEANFILES = *.gcno *.gcda
 
 TESTS_ENVIRONMENT = \
@@ -144,6 +138,8 @@ if HAVE_CQL
 libdhcpsrv_unittests_CPPFLAGS += $(CQL_CPPFLAGS)
 endif
 
+libdhcpsrv_unittests_CXXFLAGS = $(AM_CXXFLAGS)
+
 libdhcpsrv_unittests_LDFLAGS  = $(AM_LDFLAGS) $(CRYPTO_LDFLAGS) $(GTEST_LDFLAGS)
 if HAVE_MYSQL
 libdhcpsrv_unittests_LDFLAGS  += $(MYSQL_LIBS)
@@ -155,14 +151,6 @@ if HAVE_CQL
 libdhcpsrv_unittests_LDFLAGS  += $(CQL_LIBS)
 endif
 
-libdhcpsrv_unittests_CXXFLAGS = $(AM_CXXFLAGS)
-if USE_CLANGPP
-# This is to workaround unused variables tcout and tcerr in
-# log4cplus's streams.h and unused parameters from some of the
-# Boost headers.
-libdhcpsrv_unittests_CXXFLAGS += -Wno-unused-variable -Wno-unused-parameter
-endif
-
 libdhcpsrv_unittests_LDADD  = $(top_builddir)/src/lib/dhcpsrv/testutils/libdhcpsrvtest.la
 libdhcpsrv_unittests_LDADD += $(top_builddir)/src/lib/dhcpsrv/libkea-dhcpsrv.la
 libdhcpsrv_unittests_LDADD += $(top_builddir)/src/lib/eval/libkea-eval.la
index d0bc492a827c47b063e21e7b7a176dc45f2bbce4..76c534fabbcabf5a980ec79639eb5db5192e716a 100644 (file)
@@ -6,11 +6,6 @@ AM_CPPFLAGS = -I$(top_srcdir)/src/lib -I$(top_builddir)/src/lib
 AM_CPPFLAGS += $(BOOST_INCLUDES)
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
 
-if USE_CLANGPP
-# Workaround unused parameters in boost version included in FreeBSD 10
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
-
 CLEANFILES = *.gcno *.gcda
 CLEANFILES += s-rdatacode
 # These two are created with rrtype/class.h, so not explicitly listed in
index ff059de15bef687a472b432cc129e82e7c19eba4..2c8dd8ccd7c65cf88a0ec5ed8fecb8247fa56ad4 100644 (file)
@@ -4,12 +4,6 @@ AM_CPPFLAGS = -I$(top_builddir)/src/lib -I$(top_srcdir)/src/lib
 AM_CPPFLAGS += $(BOOST_INCLUDES)
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
 
-# Some versions of GCC warn about some versions of Boost regarding
-# missing initializer for members in its posix_time.
-# https://svn.boost.org/trac/boost/ticket/3477
-# But older GCC compilers don't have the flag.
-AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
-
 # GCC 4.4 emits warning about breaking strict aliasing rule.
 # This warning is a result of a GCC bug:
 # https://gcc.gnu.org/bugzilla/show_bug.cgi?id=41874
@@ -17,11 +11,6 @@ AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
 # Disabling the strict aliasing rule suppresses this warning.
 AM_CXXFLAGS += $(WARNING_GCC_44_STRICT_ALIASING_CFLAG)
 
-if USE_CLANGPP
-# Workaround unused parameters in boost version included in FreeBSD 10
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
-
 lib_LTLIBRARIES = libkea-eval.la
 libkea_eval_la_SOURCES  =
 libkea_eval_la_SOURCES += eval_log.cc eval_log.h
index a492686eff61bf22236d01dc1fbbcbf539ae44a1..f3a882efdec19596d04aa383568cc304e5b37c2a 100644 (file)
@@ -6,12 +6,6 @@ AM_CPPFLAGS += -DLOGGING_SPEC_FILE=\"$(abs_top_srcdir)/src/lib/dhcpsrv/logging.s
 
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
 
-# Some versions of GCC warn about some versions of Boost regarding
-# missing initializer for members in its posix_time.
-# https://svn.boost.org/trac/boost/ticket/3477
-# But older GCC compilers don't have the flag.
-AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
-
 if USE_STATIC_LINK
 AM_LDFLAGS = -static
 endif
@@ -46,14 +40,6 @@ libeval_unittests_LDADD   += $(top_builddir)/src/lib/exceptions/libkea-exception
 libeval_unittests_LDADD   += $(top_builddir)/src/lib/testutils/libkea-testutils.la
 libeval_unittests_LDADD   += $(CRYPTO_LIBS) $(LOG4CPLUS_LIBS)
 libeval_unittests_LDADD   += $(BOOST_LIBS) $(GTEST_LDADD)
-
-if USE_CLANGPP
-# This is to workaround unused variables tcout and tcerr in
-# log4cplus's streams.h and unused parameters from some of the
-# Boost headers.
-libeval_unittests_CXXFLAGS += -Wno-unused-variable -Wno-unused-parameter
-endif
-
 endif
 
 noinst_PROGRAMS = $(TESTS)
index 01f1d4565fcb0f5611e835d88a6b414b2fbc849d..de4c5d6635fa417bdeeeb78ed2aea6f79d7bbc29 100644 (file)
@@ -4,13 +4,6 @@ AM_CPPFLAGS  = -I$(top_builddir)/src/lib -I$(top_srcdir)/src/lib
 AM_CPPFLAGS += $(BOOST_INCLUDES)
 AM_CXXFLAGS  = $(KEA_CXXFLAGS)
 
-# Some versions of GCC warn about some versions of Boost regarding
-# missing initializer for members in its posix_time.
-# https://svn.boost.org/trac/boost/ticket/3477
-# But older GCC compilers don't have the flag.
-AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
-
-
 # Define rule to build logging source files from message file
 hooks_messages.h hooks_messages.cc: s-messages
 
@@ -74,9 +67,3 @@ libkea_hooks_include_HEADERS = \
     library_handle.h \
     hooks.h \
     server_hooks.h
-
-if USE_CLANGPP
-# Disable unused parameter warning caused by some of the
-# Boost headers when compiling with clang.
-libkea_hooks_la_CXXFLAGS += -Wno-unused-parameter
-endif
index 22b2a224e73bfcd04501bf3a00229081af1a5797..d9016a6565c9831fb7590177feb58a03dd672a12 100644 (file)
@@ -4,11 +4,6 @@ AM_CPPFLAGS  = -I$(top_builddir)/src/lib -I$(top_srcdir)/src/lib
 AM_CPPFLAGS += $(BOOST_INCLUDES)
 
 AM_CXXFLAGS  = $(KEA_CXXFLAGS)
-# Some versions of GCC warn about some versions of Boost regarding
-# missing initializer for members in its posix_time.
-# https://svn.boost.org/trac/boost/ticket/3477
-# But older GCC compilers don't have the flag.     
-AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
 
 # Kea libraries against which the test user libraries are linked.
 HOOKS_LIB      = $(top_builddir)/src/lib/hooks/libkea-hooks.la
@@ -19,11 +14,6 @@ EXCEPTIONS_LIB = $(top_builddir)/src/lib/exceptions/libkea-exceptions.la
 
 ALL_LIBS       = $(HOOKS_LIB) $(LOG_LIB) $(THREADS_LIB) $(UTIL_LIB) $(EXCEPTIONS_LIB) $(LOG4CPLUS_LIBS)
 
-if USE_CLANGPP
-# see ../Makefile.am
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
-
 # Files to clean include the file created by testing.
 CLEANFILES = *.gcno *.gcda $(builddir)/marker_file.dat
 
index aa5398de91b048672145b1fd496cb8a5b9ed6bee..45474a5c897015554bf97438561f5183bc2cb611 100644 (file)
@@ -5,12 +5,6 @@ AM_CPPFLAGS += $(BOOST_INCLUDES)
 
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
 
-# This check is for clang complaining about unused parameter
-# in boost version included in FreeBSD 10.
-if USE_CLANGPP
-AM_CXXFLAGS +=-Wno-unused-parameter
-endif
-
 if USE_STATIC_LINK
 AM_LDFLAGS = -static
 endif
index 41f84e7d9e7694bfd72a174a6d558de511ce82a1..03806f987e12c6b4c5352532cf116a20ff71475a 100644 (file)
@@ -10,11 +10,6 @@ if USE_STATIC_LINK
 AM_LDFLAGS += -static
 endif
 
-if USE_CLANGPP
-# Workaround unused parameters in boost version included in FreeBSD 10
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
-
 CLEANFILES = *.gcno *.gcda *.lock
 
 EXTRA_DIST = log_test_messages.mes
@@ -76,10 +71,6 @@ TESTS =
 
 # Define the flags used in each set of tests
 TESTSCXXFLAGS  = $(AM_CXXFLAGS)
-if USE_CLANGPP
-# Workaround unused variables tcout and tcerr in log4cplus's streams.h.
-TESTSCXXFLAGS += -Wno-unused-variable -Wno-unused-parameter
-endif
 TESTSCPPFLAGS  = $(AM_CPPFLAGS) $(GTEST_INCLUDES) $(LOG4CPLUS_INCLUDES)
 TESTSLDFLAGS  = $(AM_LDFLAGS) $(GTEST_LDFLAGS)
 
index c0b3336a973178c37ab531ff8f36acbb7b20c0c6..8b2deccb88711fb3bde0e197bed684715a995c81 100644 (file)
@@ -4,12 +4,6 @@ AM_CPPFLAGS = -I$(top_srcdir)/src/lib -I$(top_builddir)/src/lib
 AM_CPPFLAGS += $(BOOST_INCLUDES)
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
 
-if USE_CLANGPP
-# Workaround unused parameters in boost version included in FreeBSD 10
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
-
-
 lib_LTLIBRARIES = libkea-stats.la
 libkea_stats_la_SOURCES = observation.h observation.cc
 libkea_stats_la_SOURCES += context.h context.cc
index 61817770ff86b334778b4484940e15cf17de8c75..b6ba4a095ab2e31c5d26cf060c13152a793c3cad 100644 (file)
@@ -4,13 +4,8 @@ SUBDIRS = . io unittests tests python threads
 
 AM_CPPFLAGS = -I$(top_srcdir)/src/lib -I$(top_builddir)/src/lib
 AM_CPPFLAGS += $(BOOST_INCLUDES)
-
 AM_CXXFLAGS = $(KEA_CXXFLAGS)
 
-if USE_CLANGPP
-AM_CXXFLAGS += -Wno-unused-parameter
-endif
-
 lib_LTLIBRARIES = libkea-util.la
 libkea_util_la_SOURCES  = boost_time_utils.h boost_time_utils.cc
 libkea_util_la_SOURCES += csv_file.h csv_file.cc