BIND 10 master, updated. bdb2771f51482995116460fb6594ba69b13e725e [1671] Mention the default setting

BIND 10 source code commits bind10-changes at lists.isc.org
Tue Feb 28 09:54:26 UTC 2012


The branch, master has been updated
       via  bdb2771f51482995116460fb6594ba69b13e725e (commit)
       via  8684a411d7718a71ad9fb616f56b26436c4f03e5 (commit)
      from  555c57669226bca31e961412fde174f04db876ab (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit bdb2771f51482995116460fb6594ba69b13e725e
Author: Mukund Sivaraman <muks at banu.com>
Date:   Tue Feb 28 09:03:23 2012 +0530

    [1671] Mention the default setting

commit 8684a411d7718a71ad9fb616f56b26436c4f03e5
Author: Mukund Sivaraman <muks at banu.com>
Date:   Tue Feb 28 08:59:04 2012 +0530

    [1671] Support --with-werror configure switch

-----------------------------------------------------------------------

Summary of changes:
 configure.ac |   29 +++++++++++++++++++++--------
 1 files changed, 21 insertions(+), 8 deletions(-)

-----------------------------------------------------------------------
diff --git a/configure.ac b/configure.ac
index a988754..af8c2fe 100644
--- a/configure.ac
+++ b/configure.ac
@@ -288,8 +288,6 @@ AC_DEFUN([BIND10_CXX_TRY_FLAG], [
   AC_MSG_RESULT([$bind10_cxx_flag])
 ])
 
-werror_ok=0
-
 # SunStudio compiler requires special compiler options for boost
 # (http://blogs.sun.com/sga/entry/boost_mini_howto)
 if test "$SUNCXX" = "yes"; then
@@ -315,19 +313,34 @@ case "$host" in
 	;;
 esac
 
+# Don't use -Werror if configured not to
+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) ;;
+     esac],
+     [with_werror=1])
+
+werror_ok=0
+
 # Certain versions of gcc (g++) have a bug that incorrectly warns about
 # the use of anonymous name spaces even if they're closed in a single
 # translation unit.  For these versions we have to disable -Werror.
-CXXFLAGS_SAVED="$CXXFLAGS"
-CXXFLAGS="$CXXFLAGS $B10_CXXFLAGS -Werror"
-AC_MSG_CHECKING(for in-TU anonymous namespace breakage)
-AC_TRY_COMPILE([namespace { class Foo {}; }
-namespace isc {class Bar {Foo foo_;};} ],,
+if test $with_werror = 1; then
+   CXXFLAGS_SAVED="$CXXFLAGS"
+   CXXFLAGS="$CXXFLAGS $B10_CXXFLAGS -Werror"
+   AC_MSG_CHECKING(for in-TU anonymous namespace breakage)
+   AC_TRY_COMPILE([namespace { class Foo {}; }
+   namespace isc {class Bar {Foo foo_;};} ],,
 	[AC_MSG_RESULT(no)
 	 werror_ok=1
 	 B10_CXXFLAGS="$B10_CXXFLAGS -Werror"],
 	[AC_MSG_RESULT(yes)])
-CXXFLAGS="$CXXFLAGS_SAVED"
+   CXXFLAGS="$CXXFLAGS_SAVED"
+fi
 
 # Python 3.2 has an unused parameter in one of its headers. This
 # has been reported, but not fixed as of yet, so we check if we need



More information about the bind10-changes mailing list