On Fri, 2016-04-08 at 14:03 -0400, Cole Robinson wrote:
On 04/08/2016 12:04 PM, Andrea Bolognani wrote:
>
> The recent fixes to virt-host-validate have caused the mingw
> build to fail[1].
>
> Instead of working around Windows' quirks in the tool, just
> stop building it altogether - it's not like it made any
> sense to have it available on that OS anyway.
>
> Cheers.
>
>
> [1]
https://ci.centos.org/view/libvirt-project/job/libvirt-mingw/1496/
>
> Andrea Bolognani (2):
> tools: Reorganize conditional bits
> configure: Make virt-host-validate optional
>
> configure.ac | 29 ++++++++++++++++++-----------
> m4/virt-host-validate.m4 | 40 ++++++++++++++++++++++++++++++++++++++++
> tools/Makefile.am | 35 ++++++++++++++++++-----------------
> 3 files changed, 76 insertions(+), 28 deletions(-)
> create mode 100644 m4/virt-host-validate.m4
I applied them both, tried 'make distcheck', and hit:
/usr/bin/install -c -m 644 ../../../tools/virt-host-validate.1
../../../tools/virt-pki-validate.1 ../../../tools/virt-xml-validate.1
../../../tools/virsh.1 ../../../tools/virt-admin.1
../../../tools/virt-login-shell.1 ../../../tools/virt-host-validate.1
'/home/crobinso/src/libvirt/libvirt-1.3.4/_inst/share/man/man1'
/usr/bin/install: will not overwrite just-created
'/home/crobinso/src/libvirt/libvirt-1.3.4/_inst/share/man/man1/virt-host-validate.1'
with '../../../tools/virt-host-validate.1'
Makefile:2863: recipe for target 'install-man1' failed
make[4]: *** [install-man1] Error 1
make[4]: *** Waiting for unfinished jobs....
Makefiles suck :)
I just realized Martin didn't include you when replying.
Would you mind trying again after squashing in the hunk
from[1]? That should fix the failure you're seeing.
Cheers.
[
1] https://www.redhat.com/archives/libvir-list/2016-April/msg00485.html
--
Andrea Bolognani
Software Engineer - Virtualization Team