]> git.ipfire.org Git - thirdparty/kea.git/commitdiff
[#2071] propagate all flags from configure to distcheck
authorRazvan Becheriu <razvan@isc.org>
Mon, 30 May 2022 16:02:18 +0000 (19:02 +0300)
committerFrancis Dupont <fdupont@isc.org>
Wed, 10 Aug 2022 14:02:22 +0000 (16:02 +0200)
22 files changed:
Makefile.am
configure.ac
doc/sphinx/Makefile.am
doc/sphinx/arm/dhcp4-srv.rst
m4macros/ax_gssapi.m4
m4macros/ax_sysrepo.m4
src/bin/admin/tests/data/Makefile.am
src/bin/agent/tests/Makefile.am
src/bin/d2/tests/Makefile.am
src/bin/dhcp4/tests/Makefile.am
src/bin/dhcp6/tests/Makefile.am
src/bin/keactrl/Makefile.am
src/bin/lfc/tests/Makefile.am
src/bin/netconf/tests/Makefile.am
src/bin/perfdhcp/tests/Makefile.am
src/lib/dhcpsrv/tests/Makefile.am
src/lib/dhcpsrv/tests/mysql_lease_mgr_unittest.cc
src/lib/dhcpsrv/testutils/mysql_generic_backend_unittest.cc
src/lib/dhcpsrv/testutils/pgsql_generic_backend_unittest.cc
src/lib/hooks/tests/Makefile.am
src/lib/log/interprocess/tests/Makefile.am
src/lib/util/tests/Makefile.am

index ebcf3e7fbedec558f67c0170c2424bc308c0637c..18b34e7eb4e03f04835ad6bca2fa2b393ca82d7b 100644 (file)
@@ -15,6 +15,11 @@ DISTCHECK_PERFDHCP_CONFIGURE_FLAG=@DISTCHECK_PERFDHCP_CONFIGURE_FLAG@
 DISTCHECK_KEA_SHELL_CONFIGURE_FLAG=@DISTCHECK_KEA_SHELL_CONFIGURE_FLAG@
 DISTCHECK_PREMIUM_CONFIGURE_FLAG=@DISTCHECK_PREMIUM_CONFIGURE_FLAG@
 DISTCHECK_CONTRIB_CONFIGURE_FLAG=@DISTCHECK_CONTRIB_CONFIGURE_FLAG@
+DISTCHECK_MYSQL_CONFIGURE_FLAG=@DISTCHECK_MYSQL_CONFIGURE_FLAG@
+DISTCHECK_PGSQL_CONFIGURE_FLAG=@DISTCHECK_PGSQL_CONFIGURE_FLAG@
+DISTCHECK_GSSAPI_CONFIGURE_FLAG=@DISTCHECK_GSSAPI_CONFIGURE_FLAG@
+DISTCHECK_LIBYANG_CONFIGURE_FLAG=@DISTCHECK_LIBYANG_CONFIGURE_FLAG@
+DISTCHECK_SYSREPO_CONFIGURE_FLAG=@DISTCHECK_SYSREPO_CONFIGURE_FLAG@
 
 OVERALL_COVERAGE_DIR=$(abs_top_builddir)/coverage-cpp-html
 
@@ -47,6 +52,21 @@ DISTCHECK_CONFIGURE_FLAGS += $(DISTCHECK_PREMIUM_CONFIGURE_FLAG)
 # Keep the contrib config
 DISTCHECK_CONFIGURE_FLAGS += $(DISTCHECK_CONTRIB_CONFIGURE_FLAG)
 
+# Keep the mysql config
+DISTCHECK_CONFIGURE_FLAGS += $(DISTCHECK_MYSQL_CONFIGURE_FLAG)
+
+# Keep the pgsql config
+DISTCHECK_CONFIGURE_FLAGS += $(DISTCHECK_PGSQL_CONFIGURE_FLAG)
+
+# Keep the gssapi config
+DISTCHECK_CONFIGURE_FLAGS += $(DISTCHECK_GSSAPI_CONFIGURE_FLAG)
+
+# Keep the libyang config
+DISTCHECK_CONFIGURE_FLAGS += $(DISTCHECK_LIBYANG_CONFIGURE_FLAG)
+
+# Keep the sysrepo config
+DISTCHECK_CONFIGURE_FLAGS += $(DISTCHECK_SYSREPO_CONFIGURE_FLAG)
+
 dist_doc_DATA = AUTHORS COPYING ChangeLog README CONTRIBUTING.md platforms.rst code_of_conduct.md
 
 .PHONY: check-valgrind check-valgrind-suppress
@@ -114,9 +134,10 @@ endif
                        --ignore-errors gcov,source,graph \
                        --output report.info; \
                sed --in-place --expression "s|$(abs_top_srcdir)|$(abs_top_builddir)|g" report.info; \
-               "$(GENHTML)" --frames --show-details --title 'Kea code coverage report' --legend \
-                --function-coverage --ignore-errors source --demangle-cpp \
-                --output "$(OVERALL_COVERAGE_DIR)" report.info; \
+               "$(GENHTML)" \
+               --frames --show-details --title 'Kea code coverage report' --legend \
+               --function-coverage --ignore-errors source --demangle-cpp \
+               --output "$(OVERALL_COVERAGE_DIR)" report.info; \
                printf "Generated C++ code coverage report in HTML at %s.\n" "$(OVERALL_COVERAGE_DIR)"; \
        else \
                echo "C++ code coverage not enabled at configuration time." ; \
index 7e02660125b0bc7b179c1db325f93a03c82f04df..d8605f6378e7243270f9e6c93e7393ffef23a20f 100644 (file)
@@ -677,11 +677,14 @@ elif test "${mysql_config}" != "no" ; then
     MYSQL_CONFIG="${withval}"
 fi
 
+DISTCHECK_MYSQL_CONFIGURE_FLAG=
 if test "$MYSQL_CONFIG" != "" ; then
     if test -d "$MYSQL_CONFIG" -o ! -x "$MYSQL_CONFIG" ; then
         AC_MSG_ERROR([MySQL dependencies cannot be found. Please install MySQL libraries or point --with-mysql to mysql_config program if it is located in non-default directory, eg. --with-mysql=/opt/mysql/bin/mysql_config.])
     fi
 
+    DISTCHECK_MYSQL_CONFIGURE_FLAG="--with-mysql=$MYSQL_CONFIG"
+
     MYSQL_CPPFLAGS=`$MYSQL_CONFIG --cflags`
     MYSQL_LIBS=`$MYSQL_CONFIG --libs`
     MYSQL_LIBS="$MYSQL_LIBS $CRYPTO_LIBS"
@@ -689,6 +692,7 @@ if test "$MYSQL_CONFIG" != "" ; then
 
     AC_SUBST(MYSQL_CPPFLAGS)
     AC_SUBST(MYSQL_LIBS)
+    AC_SUBST(DISTCHECK_MYSQL_CONFIGURE_FLAG)
 
     # Check that a simple program using MySQL functions can compile and link.
     CPPFLAGS_SAVED="$CPPFLAGS"
@@ -757,11 +761,14 @@ elif test "${pg_config}" != "no" ; then
     PG_CONFIG="${withval}"
 fi
 
+DISTCHECK_PGSQL_CONFIGURE_FLAG=
 if test "$PG_CONFIG" != "" ; then
     if test -d "$PG_CONFIG" -o ! -x "$PG_CONFIG" ; then
         AC_MSG_ERROR([PostgreSQL dependencies cannot be found. Please install PostgreSQL libraries or point --with-pgsql to pg_config program if it is located in non-default directory, eg. --with-pgsql=/opt/pgsql/bin/pg_config.])
     fi
 
+    DISTCHECK_PGSQL_CONFIGURE_FLAG="--with-pgsql=$PG_CONFIG"
+
     PGSQL_CPPFLAGS=`$PG_CONFIG --cppflags`
     PGSQL_INCLUDEDIR=`$PG_CONFIG --includedir`
     PGSQL_INCLUDEDIR_SERVER=`$PG_CONFIG --includedir-server`
@@ -772,6 +779,7 @@ if test "$PG_CONFIG" != "" ; then
 
     AC_SUBST(PGSQL_CPPFLAGS)
     AC_SUBST(PGSQL_LIBS)
+    AC_SUBST(DISTCHECK_PGSQL_CONFIGURE_FLAG)
 
     # Check that a simple program using PostgreSQL functions can compile and link.
     CPPFLAGS_SAVED="$CPPFLAGS"
index 7fa957bdcd64d2a9a3a97ff9c7440a15f607f840..bfe15ea00215c8e12144f9904d8449c537175ac6 100644 (file)
@@ -188,7 +188,6 @@ update-python-dependencies: ./src/requirements.in
 
 clean-local:
        rm -rf $(sphinxbuilddir)
-       rm -f $(srcdir)/mes-files.txt $(srcdir)/api-files.txt
        rm -f $(srcdir)/kea-messages.rst $(srcdir)/api.rst
        rm -f $(srcdir)/arm/platforms.rst
 
index 534c18e4f7681b3308dfb7d84783b0a06a227e40..3325893a62b52e8fc211659b004729fd786c2c2c 100644 (file)
@@ -6906,7 +6906,7 @@ All supported parameters can be configured via the ``cb_cmds`` hook library
 described in the :ref:`hooks-cb-cmds` section. The general rule is that
 scalar global parameters are set using
 ``remote-global-parameter4-set``; shared-network-specific parameters
-are set using ``remote-network4-set``; and subnet- and pool-level
+are set using ``remote-network4-set``; and subnet-level and pool-level
 parameters are set using ``remote-subnet4-set``. Whenever
 there is an exception to this general rule, it is highlighted in the
 table. Non-scalar global parameters have dedicated commands; for example,
index 2360a3755e947286c247645baa62c30fc07ac8ee..539343edad50bcabcffe727cbffaca31904c528c 100644 (file)
@@ -13,6 +13,7 @@ AC_ARG_WITH([gssapi],
 ENABLE_GSSAPI=no
 GSSAPI_CFLAGS=
 GSSAPI_LIBS=
+DISTCHECK_GSSAPI_CONFIGURE_FLAG="--with-gssapi=$gssapi_path"
 
 AC_MSG_CHECKING([for gssapi support])
 if test "x$gssapi_path" = "x" ; then
@@ -86,6 +87,7 @@ fi
 
 AC_SUBST(GSSAPI_CFLAGS)
 AC_SUBST(GSSAPI_LIBS)
+AC_SUBST(DISTCHECK_GSSAPI_CONFIGURE_FLAG)
 AM_CONDITIONAL([HAVE_GSSAPI], [test $ENABLE_GSSAPI = "yes"])
 
 ])dnl AX_GSS_API
index 415125e6b2277191fae937a31eec152f2504c516..50b30713c15f41b4d6f91e027326657373bbfbb5 100644 (file)
@@ -4,10 +4,14 @@ AC_DEFUN([AX_SYSREPO], [
     [AS_HELP_STRING([--with-libyang[[=PATH]]], [optional path to the libyang installation directory])],
     [with_libyang="${withval}"])
 
+    DISTCHECK_LIBYANG_CONFIGURE_FLAG="--with-libyang=$with_libyang"
+
   AC_ARG_WITH([sysrepo],
     [AS_HELP_STRING([--with-sysrepo[[=PATH]]], [optional path to the sysrepo installation directory])],
     [with_sysrepo="${withval}"])
 
+    DISTCHECK_SYSREPO_CONFIGURE_FLAG="--with-sysrepo=$with_sysrepo"
+
   # If --with-libyang was omitted, assume it was passed and give it the value
   # from --with-sysrepo.
   if test -z "${with_libyang}"; then
@@ -220,4 +224,6 @@ AC_DEFUN([AX_SYSREPO], [
   AC_SUBST(SRPD_PLUGINS_PATH)
   AC_SUBST(SYSREPO_VERSION)
   AC_SUBST(SYSREPOCPP_VERSION)
+  AC_SUBST(DISTCHECK_LIBYANG_CONFIGURE_FLAG)
+  AC_SUBST(DISTCHECK_SYSREPO_CONFIGURE_FLAG)
 ])
index 3e9f88dbbbb47d590f49d936e4d8142cd6e2584d..7ce62898ec142787aa66ac0a3740a094d5c13dee 100644 (file)
@@ -1,3 +1,3 @@
 EXTRA_DIST = \
-    lease4_dump_test.reference.csv \
-    lease6_dump_test.reference.csv
+       lease4_dump_test.reference.csv \
+       lease6_dump_test.reference.csv
index 7cfedfeb111d1e04bbce784125704a42f5b10ac6..40ac32765c36e5d482273c088aac11aeed97f3da 100644 (file)
@@ -7,7 +7,7 @@ EXTRA_DIST += testdata/hiddens
 EXTRA_DIST += testdata/hiddenu
 
 TESTS_ENVIRONMENT = \
-        $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
+       $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
 
 # Shell tests
 SHTESTS = ca_process_tests.sh
index 0da992b6ec1efb4641eba4890efa91cee755c1f1..b47513278ebc3afb882161f55d77092e739cf203 100644 (file)
@@ -6,7 +6,7 @@ EXTRA_DIST += testdata/d2_cfg_tests.json
 EXTRA_DIST += testdata/get_config.json
 
 TESTS_ENVIRONMENT = \
-        $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
+       $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
 
 # Shell tests
 SHTESTS = d2_process_tests.sh
index 8e5808bee5e62b1355f8e6d69f5f315ea27286b1..291d29f2847a1391fc14336c754ad7ee59d2a24f 100644 (file)
@@ -4,7 +4,7 @@ SUBDIRS = .
 EXTRA_DIST = get_config_unittest.cc.skel
 
 TESTS_ENVIRONMENT = \
-        $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
+       $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
 
 # Shell tests
 SHTESTS = dhcp4_process_tests.sh
index e3518f1ad42999965370fc54b34355969d8d6567..78c71663124fd9593b7305606bae7257825a251d 100644 (file)
@@ -4,7 +4,7 @@ SUBDIRS = .
 EXTRA_DIST = get_config_unittest.cc.skel
 
 TESTS_ENVIRONMENT = \
-        $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
+       $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
 
 # Shell tests
 SHTESTS = dhcp6_process_tests.sh
index e85392d056993da98d1894a0d697df2ceb51c445..ca9f255726953ea48f41e9a5a12b0aa6a3419843 100644 (file)
@@ -3,8 +3,9 @@ SUBDIRS = . tests
 # Install keactrl in sbin and the keactrl.conf required by the keactrl
 # in etc. keactrl will look for its configuration file in the etc folder.
 sbin_SCRIPTS  = keactrl
-KEA_CONFIGFILES = kea-dhcp4.conf kea-dhcp6.conf kea-dhcp-ddns.conf \
-                  kea-ctrl-agent.conf
+KEA_CONFIGFILES = \
+       kea-dhcp4.conf kea-dhcp6.conf kea-dhcp-ddns.conf kea-ctrl-agent.conf
+
 if HAVE_SYSREPO
 KEA_CONFIGFILES += kea-netconf.conf
 endif
@@ -13,9 +14,10 @@ CONFIGFILES = keactrl.conf $(KEA_CONFIGFILES)
 
 DISTCLEANFILES = keactrl keactrl.conf
 CLEANFILES = $(KEA_CONFIGFILES)
-EXTRA_DIST = keactrl.in keactrl.conf.in kea-dhcp4.conf.pre \
-             kea-dhcp6.conf.pre kea-dhcp-ddns.conf.pre \
-             kea-ctrl-agent.conf.pre kea-netconf.conf.pre
+EXTRA_DIST = \
+       keactrl.in keactrl.conf.in kea-dhcp4.conf.pre \
+       kea-dhcp6.conf.pre kea-dhcp-ddns.conf.pre \
+       kea-ctrl-agent.conf.pre kea-netconf.conf.pre
 
 # *.conf files are not really sources used for building other targets, but we need
 # these files to be generated before make install is called.
@@ -24,23 +26,23 @@ BUILT_SOURCES = $(KEA_CONFIGFILES)
 
 kea-dhcp4.conf: kea-dhcp4.conf.pre
        $(top_builddir)/tools/path_replacer.sh \
-    $(top_srcdir)/src/bin/keactrl/kea-dhcp4.conf.pre $@
+       $(top_srcdir)/src/bin/keactrl/kea-dhcp4.conf.pre $@
 
 kea-dhcp6.conf: kea-dhcp6.conf.pre
        $(top_builddir)/tools/path_replacer.sh \
-    $(top_srcdir)/src/bin/keactrl/kea-dhcp6.conf.pre $@
+       $(top_srcdir)/src/bin/keactrl/kea-dhcp6.conf.pre $@
 
 kea-dhcp-ddns.conf: kea-dhcp-ddns.conf.pre
        $(top_builddir)/tools/path_replacer.sh \
-    $(top_srcdir)/src/bin/keactrl/kea-dhcp-ddns.conf.pre $@
+       $(top_srcdir)/src/bin/keactrl/kea-dhcp-ddns.conf.pre $@
 
 kea-ctrl-agent.conf: kea-ctrl-agent.conf.pre
        $(top_builddir)/tools/path_replacer.sh \
-    $(top_srcdir)/src/bin/keactrl/kea-ctrl-agent.conf.pre $@
+       $(top_srcdir)/src/bin/keactrl/kea-ctrl-agent.conf.pre $@
 
 kea-netconf.conf: kea-netconf.conf.pre
        $(top_builddir)/tools/path_replacer.sh \
-    $(top_srcdir)/src/bin/keactrl/kea-netconf.conf.pre $@
+       $(top_srcdir)/src/bin/keactrl/kea-netconf.conf.pre $@
 
 if INSTALL_CONFIGURATIONS
 
index 65593436c0292f2c39e862a391ac2580b78e233a..edba11847ddbfa9805a880f75fd8fa9e87a1a2b9 100644 (file)
@@ -15,7 +15,7 @@ AM_LDFLAGS = -static
 endif
 
 TESTS_ENVIRONMENT = \
-        $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
+       $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
 
 TESTS =
 if HAVE_GTEST
index e25e62a3b73f1f834930138a8c047f869cbbf275..594472579f30a8e89bc3775b9b8475999f25f6f5 100644 (file)
@@ -24,7 +24,7 @@ AM_LDFLAGS = -static
 endif
 
 TESTS_ENVIRONMENT = \
-        $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
+       $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
 
 TESTS =
 if HAVE_GTEST
index 72202a8c4f80b1db47675487bce2be54bf6003c1..c62bd70b1fa8546e9d9f2ccc132cd9ae1a8e9fe0 100644 (file)
@@ -17,7 +17,7 @@ CLEANFILES = *.gcno *.gcda
 CLEANFILES += test1.hex test2.hex test3.hex test4.hex test5.hex
 
 TESTS_ENVIRONMENT = \
-        $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
+       $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
 
 TESTS =
 if HAVE_GTEST
index 804a497a732f0a7499a0dc7314a925410b761e10..8252ebad23679e6f74a816af458b3b7a64919371 100644 (file)
@@ -17,7 +17,7 @@ CLEANFILES = *.gcno *.gcda
 DISTCLEANFILES = test_libraries.h
 
 TESTS_ENVIRONMENT = \
-        $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
+       $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
 
 TESTS =
 if HAVE_GTEST
index 9383db8b63583f8827bbb778efb5309080e237d7..0b4f839abf47ff4cfa3f9428a904d9052b3f1592 100644 (file)
@@ -10,6 +10,7 @@
 #include <dhcpsrv/lease_mgr_factory.h>
 #include <dhcpsrv/mysql_lease_mgr.h>
 #include <dhcpsrv/testutils/test_utils.h>
+#include <dhcpsrv/testutils/mysql_generic_backend_unittest.h>
 #include <dhcpsrv/tests/generic_lease_mgr_unittest.h>
 #include <exceptions/exceptions.h>
 #include <mysql/mysql_connection.h>
@@ -1073,6 +1074,25 @@ TEST_F(MySqlLeaseMgrTest, leaseStatsQueryAttribution6MultiThreading) {
     testLeaseStatsQueryAttribution6();
 }
 
+/// @brief This test is a basic check for the generic backend test class,
+///        rather than any production code check.
+TEST_F(MySqlGenericBackendTest, leaseCount) {
+
+    // Create database connection parameter list
+    DatabaseConnection::ParameterMap params;
+    params["name"] = "keatest";
+    params["user"] = "keatest";
+    params["password"] = "keatest";
+
+    // Create and open the database connection
+    MySqlConnection conn(params);
+    conn.openDatabase();
+
+    // Check that the countRows is working. It's used extensively in other
+    // tests, so basic check is enough here.
+    EXPECT_EQ(0, countRows(conn, "lease4"));
+}
+
 /// @brief Checks that no exceptions are thrown when inquiring about JSON
 /// support and prints an informative message.
 TEST_F(MySqlLeaseMgrTest, isJsonSupported) {
index ace663e7b7f03dbe9488ce625caded941b145457..ac142d0c6fc59cb7acf5488758291b16a33160bd 100644 (file)
@@ -6,8 +6,10 @@
 
 #include <config.h>
 #include <dhcpsrv/testutils/mysql_generic_backend_unittest.h>
+#include <mysql/testutils/mysql_schema.h>
 
 using namespace isc::db;
+using namespace isc::db::test;
 
 namespace isc {
 namespace dhcp {
@@ -15,6 +17,7 @@ namespace test {
 
 MySqlGenericBackendTest::MySqlGenericBackendTest()
     : GenericBackendTest() {
+    createMySQLSchema();
 }
 
 size_t
index 412ff0486e69f796ae8af4c02c8673858cb85b6f..8d588a1049dedb42e3f8992e33c966b60888f528 100644 (file)
@@ -6,8 +6,10 @@
 
 #include <config.h>
 #include <dhcpsrv/testutils/pgsql_generic_backend_unittest.h>
+#include <pgsql/testutils/pgsql_schema.h>
 
 using namespace isc::db;
+using namespace isc::db::test;
 
 namespace isc {
 namespace dhcp {
@@ -15,6 +17,7 @@ namespace test {
 
 PgSqlGenericBackendTest::PgSqlGenericBackendTest()
     : GenericBackendTest() {
+    createPgSQLSchema();
 }
 
 size_t
index b6fe8b9a8f03424149ff4d0c9b63eab5f1eed3c0..c9ea1a1ba1cd6ba88659ad9598319477fe69024d 100644 (file)
@@ -38,8 +38,9 @@ if HAVE_GTEST
 # to unexpected errors. For this reason, the --enable-static-link option is
 # ignored for unit tests built here.
 
-noinst_LTLIBRARIES = libnvl.la  libivl.la libfxl.la libbcl.la liblcl.la \
-                     liblecl.la libucl.la libfcl.la libpcl.la libacl.la
+noinst_LTLIBRARIES = \
+       libnvl.la  libivl.la libfxl.la libbcl.la liblcl.la \
+       liblecl.la libucl.la libfcl.la libpcl.la libacl.la
 
 # -rpath /nowhere is a hack to trigger libtool to not create a
 # convenience archive, resulting in shared modules
index 5a53327338c6f2a29bcc17663392a0e7f5023769..a1f8a731137d10ad386b9bfe88783c58d4efa3e7 100644 (file)
@@ -15,7 +15,7 @@ endif
 CLEANFILES = *.gcno *.gcda
 
 TESTS_ENVIRONMENT = \
-        $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
+       $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
 
 TESTS =
 if HAVE_GTEST
index 56aff6a664203d7168b24b1615bd6eb8bac83f9d..fd696504d6b260d3b188dc76560c27119191f33f 100644 (file)
@@ -18,7 +18,7 @@ CLEANFILES = *.gcno *.gcda
 CLEANFILES += *.csv
 
 TESTS_ENVIRONMENT = \
-        $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
+       $(LIBTOOL) --mode=execute $(VALGRIND_COMMAND)
 
 TESTS =
 if HAVE_GTEST