[libvirt] [PATCH 03/16] bootstrap.conf: drop gnulib tests from libvirt

Pavel Hrdina phrdina at redhat.com
Thu Jan 9 08:16:33 UTC 2020


We are in process of removing gnulib and adopting meson as our build
system.  In order to help with the transition let's drop gnulib tests.

This will also help with the fact that before we will be able to drop
gnulib completely we will store output of bootstrap in git.

Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
---
 .gitignore               |  2 --
 Makefile.am              |  2 +-
 bootstrap.conf           | 13 -------------
 configure.ac             |  1 -
 gnulib/tests/Makefile.am | 32 --------------------------------
 5 files changed, 1 insertion(+), 49 deletions(-)
 delete mode 100644 gnulib/tests/Makefile.am

diff --git a/.gitignore b/.gitignore
index 2d6e3e3194..2139d176da 100644
--- a/.gitignore
+++ b/.gitignore
@@ -27,12 +27,10 @@ Makefile.in
 
 # gnulib related ignores
 !/gnulib/lib/Makefile.am
-!/gnulib/tests/Makefile.am
 *.rej
 *~
 /gnulib/lib/*
 /gnulib/m4/*
-/gnulib/tests/*
 
 # git related ignores
 *.orig
diff --git a/Makefile.am b/Makefile.am
index 0d7ccc74db..8a8eecb697 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -23,7 +23,7 @@ GENHTML = genhtml
 # so force it explicitly
 DISTCHECK_CONFIGURE_FLAGS = --enable-werror
 
-SUBDIRS = . gnulib/lib include/libvirt src tools docs gnulib/tests \
+SUBDIRS = . gnulib/lib include/libvirt src tools docs \
   tests po examples
 
 XZ_OPT ?= -v -T0
diff --git a/bootstrap.conf b/bootstrap.conf
index ae9ecb4039..13d0e77514 100644
--- a/bootstrap.conf
+++ b/bootstrap.conf
@@ -143,13 +143,10 @@ fi
 gnulib_name=libgnu
 m4_base=m4
 source_base=gnulib/lib
-tests_base=gnulib/tests
 gnulib_tool_option_extras="\
  --lgpl=2\
- --with-tests\
  --makefile-name=gnulib.mk\
  --avoid=pt_chown\
- --avoid=lock-tests\
 "
 local_gl_dir=gnulib/local
 
@@ -184,16 +181,6 @@ gnulib_extra_files="
         doc/INSTALL
 "
 
-
-bootstrap_post_import_hook()
-{
-  # Change paths in gnulib/tests/gnulib.mk from "../../.." to "../..",
-  # and make tests conditional by changing "TESTS" to "GNULIB_TESTS".
-  m=gnulib/tests/gnulib.mk
-  sed 's,\.\./\.\./\.\.,../..,g; s/^TESTS /GNULIB_TESTS /' $m > $m-t
-  mv -f $m-t $m
-}
-
 bootstrap_epilogue()
 {
     echo "$0: done.  Now you can run 'mkdir build && cd build && ../configure'."
diff --git a/configure.ac b/configure.ac
index cbfb3fe1aa..2be909b7ce 100644
--- a/configure.ac
+++ b/configure.ac
@@ -922,7 +922,6 @@ AC_CONFIG_FILES([run],
 AC_CONFIG_FILES([\
         Makefile src/Makefile include/libvirt/Makefile docs/Makefile \
         gnulib/lib/Makefile \
-        gnulib/tests/Makefile \
         .color_coded \
         .ycm_extra_conf.py \
         libvirt.pc \
diff --git a/gnulib/tests/Makefile.am b/gnulib/tests/Makefile.am
deleted file mode 100644
index 7062cbaf87..0000000000
--- a/gnulib/tests/Makefile.am
+++ /dev/null
@@ -1,32 +0,0 @@
-## Makefile for gnulib/lib
-
-## Copyright (C) 2011, 2013 Red Hat, Inc.
-##
-## This library is free software; you can redistribute it and/or
-## modify it under the terms of the GNU Lesser General Public
-## License as published by the Free Software Foundation; either
-## version 2.1 of the License, or (at your option) any later version.
-##
-## This library is distributed in the hope that it will be useful,
-## but WITHOUT ANY WARRANTY; without even the implied warranty of
-## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
-## Lesser General Public License for more details.
-##
-## You should have received a copy of the GNU Lesser General Public
-## License along with this library.  If not, see
-## <http://www.gnu.org/licenses/>.
-
-include gnulib.mk
-
-GNULIB_TESTS0 =
-GNULIB_TESTS1 = $(GNULIB_TESTS)
-if WITH_EXPENSIVE_TESTS
-## Automake requires that at least one conditional call out all tests to
-## be run, for those tests to be shipped in the tarball
-TESTS = $(GNULIB_TESTS)
-endif WITH_EXPENSIVE_TESTS
-## However, we want to change the set of tests based on the make environment,
-## where the default was set at configure time.  Use GNU make constructs to
-## hide our actions from Automake, so we don't get it too confused.
-VIR_TEST_EXPENSIVE ?= $(VIR_TEST_EXPENSIVE_DEFAULT)
-$(eval TESTS=$(GNULIB_TESTS$(VIR_TEST_EXPENSIVE)))
-- 
2.24.1




More information about the libvir-list mailing list