BIND 10 master, updated. 5cbaf8b762a4a90e483715c87568c1f9d1b44264 [master] Use make targets for Valgrind instead of configure switches

[BIND 10 source code commits]

The branch, master has been updated
       via  5cbaf8b762a4a90e483715c87568c1f9d1b44264 (commit)
      from  0557f306a4d0accdd5b6510466749f99073a78f5 (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 5cbaf8b762a4a90e483715c87568c1f9d1b44264
Author: Mukund Sivaraman <muks at isc.org>
Date:   Mon Jun 18 16:24:34 2012 +0530

    [master] Use make targets for Valgrind instead of configure switches

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

Summary of changes:
 Makefile.am  |   18 ++++++++++++++++++
 configure.ac |   24 ++++--------------------
 2 files changed, 22 insertions(+), 20 deletions(-)

-----------------------------------------------------------------------
diff --git a/Makefile.am b/Makefile.am
index 502404d..7024294 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -18,6 +18,24 @@ DISTCHECK_CONFIGURE_FLAGS += $(DISTCHECK_GTEST_CONFIGURE_FLAG)
 
 dist_doc_DATA = AUTHORS COPYING ChangeLog README
 
+.PHONY: check-valgrind check-valgrind-suppress
+
+check-valgrind:
+if HAVE_VALGRIND
+	@VALGRIND_COMMAND="$(VALGRIND) -q --gen-suppressions=all --track-origins=yes --num-callers=48 --leak-check=full --fullpath-after=" \
+	make -C $(abs_top_builddir) check
+else
+	@echo "*** Valgrind is required for check-valgrind ***"; exit 1;
+endif
+
+check-valgrind-suppress:
+if HAVE_VALGRIND
+	@VALGRIND_COMMAND="$(VALGRIND) -q --gen-suppressions=all --error-exitcode=1 --suppressions=$(abs_top_srcdir)/src/valgrind-suppressions --suppressions=$(abs_top_srcdir)/src/valgrind-suppressions.revisit --num-callers=48 --leak-check=full --fullpath-after=" \
+	make -C $(abs_top_builddir) check
+else
+	@echo "*** Valgrind is required for check-valgrind-suppress ***"; exit 1;
+endif
+
 clean-cpp-coverage:
 	@if [ $(USE_LCOV) = yes ] ; then \
 		$(LCOV) --directory . --zerocounters; \
diff --git a/configure.ac b/configure.ac
index fe95c92..70df25d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -986,25 +986,10 @@ AM_COND_IF([ENABLE_LOGGER_CHECKS], [AC_DEFINE([ENABLE_LOGGER_CHECKS], [1], [Chec
 AC_PATH_PROG(VALGRIND, valgrind, no)
 AM_CONDITIONAL(HAVE_VALGRIND, test "x$VALGRIND" != "xno")
 
-AC_ARG_ENABLE(valgrind, [AC_HELP_STRING([--enable-valgrind],
-  [run tests under Valgrind [default=no]])], enable_valgrind=$enableval, enable_valgrind=no)
-AC_ARG_ENABLE(valgrind-suppressions, [AC_HELP_STRING([--enable-valgrind-suppressions],
-  [enable Valgrind suppressions [default=no]])], enable_valgrind_suppressions=$enableval, enable_valgrind_suppressions=no)
-
-use_valgrind=no
-use_valgrind_suppressions=no
-
-if test "x$VALGRIND" != "xno" -a "x$enable_valgrind" != "xno"; then
-   if test "x$enable_valgrind_suppressions" != "xno"; then
-      VALGRIND_EXTRA="--error-exitcode=1 --suppressions=\$(top_srcdir)/src/valgrind-suppressions --suppressions=\$(top_srcdir)/src/valgrind-suppressions.revisit"
-      use_valgrind_suppressions=yes
-   else
-      VALGRIND_EXTRA="--track-origins=yes"
-   fi
-   VALGRIND_COMMAND="$VALGRIND -q --gen-suppressions=all $VALGRIND_EXTRA --num-callers=48 --leak-check=full --fullpath-after="
-   use_valgrind=yes
+found_valgrind="not found"
+if test "x$VALGRIND" != "xno"; then
+   found_valgrind="found"
 fi
-AC_SUBST(VALGRIND_COMMAND)
 
 AC_CONFIG_FILES([Makefile
                  doc/Makefile
@@ -1316,8 +1301,7 @@ Features:
 
 Developer:
   Google Tests:  $gtest_path
-  Valgrind: $use_valgrind
-  Valgrind Suppressions: $use_valgrind_suppressions
+  Valgrind: $found_valgrind
   C++ Code Coverage: $USE_LCOV
   Python Code Coverage: $USE_PYCOVERAGE
   Logger checks: $enable_logger_checks