[svn] commit: r3877 - in /trunk: configure.ac src/lib/nsas/Makefile.am src/lib/nsas/tests/Makefile.am

BIND 10 source code commits bind10-changes at lists.isc.org
Fri Dec 17 13:45:39 UTC 2010


Author: jreed
Date: Fri Dec 17 13:45:38 2010
New Revision: 3877

Log:
Check if compiler accepts -Wno-missing-field-initializers.
Use it for nsas build even if not GCC.
(It was already used by default for nsas for past few days,
but that breaks old GCC. This will fix build for Solaris GCC.)

Modified:
    trunk/configure.ac
    trunk/src/lib/nsas/Makefile.am
    trunk/src/lib/nsas/tests/Makefile.am

Modified: trunk/configure.ac
==============================================================================
--- trunk/configure.ac (original)
+++ trunk/configure.ac Fri Dec 17 13:45:38 2010
@@ -195,6 +195,19 @@
 # specify the default warning flags in CXXFLAGS and let specific modules
 # "override" the default.
 
+# This may be used to try compiler flags.
+AC_DEFUN([BIND10_CXX_TRY_FLAG], [
+  AC_MSG_CHECKING([whether $CXX supports $1])
+
+  bind10_save_CXXFLAGS="$CXXFLAGS"
+  CXXFLAGS="$CXXFLAGS $1"
+
+  AC_COMPILE_IFELSE([ ], [bind10_cxx_flag=yes], [bind10_cxx_flag=no])
+  CXXFLAGS="$bind10_save_CXXFLAGS"
+
+  AC_MSG_RESULT([$bind10_cxx_flag])
+])
+
 werror_ok=0
 
 # SunStudio compiler requires special compiler options for boost
@@ -203,6 +216,10 @@
 CXXFLAGS="$CXXFLAGS -library=stlport4 -features=tmplife -features=tmplrefstatic"
 MULTITHREADING_FLAG="-mt"
 fi
+
+BIND10_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:
 if test "X$GXX" = "Xyes"; then

Modified: trunk/src/lib/nsas/Makefile.am
==============================================================================
--- trunk/src/lib/nsas/Makefile.am (original)
+++ trunk/src/lib/nsas/Makefile.am Fri Dec 17 13:45:38 2010
@@ -7,12 +7,11 @@
 AM_CPPFLAGS += $(SQLITE_CFLAGS)
 AM_CXXFLAGS = $(B10_CXXFLAGS)
 
-if USE_GXX
 # 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
-AM_CXXFLAGS += -Wno-missing-field-initializers
-endif
+# But older GCC compilers don't have the flag.
+AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
 
 if USE_CLANGPP
 # clang++ complains about unused function parameters in some boost header

Modified: trunk/src/lib/nsas/tests/Makefile.am
==============================================================================
--- trunk/src/lib/nsas/tests/Makefile.am (original)
+++ trunk/src/lib/nsas/tests/Makefile.am Fri Dec 17 13:45:38 2010
@@ -11,12 +11,11 @@
 AM_LDFLAGS += -static
 endif
 
-if USE_GXX
 # 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
-AM_CXXFLAGS += -Wno-missing-field-initializers
-endif
+# But older GCC compilers don't have the flag.     
+AM_CXXFLAGS += $(WARNING_NO_MISSING_FIELD_INITIALIZERS_CFLAG)
 
 if USE_CLANGPP
 # see ../Makefile.am




More information about the bind10-changes mailing list