Re: [PATCH 2/2] configure: Make virt-host-validate optional

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Apr 11, 2016 at 11:29:51AM +0200, Andrea Bolognani wrote:
On Mon, 2016-04-11 at 10:48 +0200, Martin Kletzander wrote:
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?

We want to ship the generated man pages so that users don't need
to have pod2man(1) installed on their system.

Anyway, the problem is just that a hunk got lost while shuffling
commits around. After squashing it in distcheck works again for
me.

Cheers.


diff --git a/tools/Makefile.am b/tools/Makefile.am
index ab37d70..560a9a5 100644
--- a/tools/Makefile.am
+++ b/tools/Makefile.am
@@ -67,7 +67,6 @@ bin_SCRIPTS = virt-xml-validate virt-pki-validate
 bin_PROGRAMS = virsh virt-admin
 libexec_SCRIPTS = libvirt-guests.sh
 dist_man1_MANS = \
-               virt-host-validate.1 \
                virt-pki-validate.1 \
                virt-xml-validate.1 \
                virsh.1 \
-- 

Yes, or that ^^ if we don't require pod2man =) That's fine with me, but
I don't have mingw build working, so I'll leave it to others to properly
ACK it.

Andrea Bolognani
Software Engineer - Virtualization Team

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]