On Fri, Apr 08, 2016 at 06:04:51PM +0200, Andrea Bolognani wrote:
virt-host-validate, just like virt-login-shell, doesn't make
sense
on Windows, so we should avoid building it.
Make the tool optional and build it by default on all platforms
except Windows, erroring out if the user attempts to build it
anyway.
---
configure.ac | 29 ++++++++++++++++++-----------
m4/virt-host-validate.m4 | 40 ++++++++++++++++++++++++++++++++++++++++
tools/Makefile.am | 9 ++++++++-
3 files changed, 66 insertions(+), 12 deletions(-)
create mode 100644 m4/virt-host-validate.m4
[...]
diff --git a/tools/Makefile.am b/tools/Makefile.am
index 930e6c3..16a1aed 100644
--- a/tools/Makefile.am
+++ b/tools/Makefile.am
@@ -64,7 +64,7 @@ confdir = $(sysconfdir)/libvirt
conf_DATA =
bin_SCRIPTS = virt-xml-validate virt-pki-validate
-bin_PROGRAMS = virsh virt-host-validate virt-admin
+bin_PROGRAMS = virsh virt-admin
libexec_SCRIPTS = libvirt-guests.sh
dist_man1_MANS = \
virt-host-validate.1 \
@@ -89,6 +89,13 @@ EXTRA_DIST += \
virt-login-shell.1
endif ! WITH_LOGIN_SHELL
+if WITH_HOST_VALIDATE
+bin_PROGRAMS += virt-host-validate
+dist_man1_MANS += virt-host-validate.1
+else ! WITH_HOST_VALIDATE
+EXTRA_DIST += virt-host-validate.1
+endif ! WITH_HOST_VALIDATE
+
I haven't looked that much into it, but I don't think we should put the
generated man pages into the dist, but rather the .pod files and that
are there by default, right? Could just removing the:
EXTRA_DIST += virt-host-validate.1
be enough to fix what Cole saw happen?