Re: [libvirt] [PATCH 0/2] fix configure to require yajl if qemu is new enough

Ping. On 09/05/2013 04:18 PM, Eric Blake wrote: > After yet another debug session on IRC about failure to use qemu > because yajl-devel wasn't installed, I came up with these patches. > The first is just preparation (no change to generated code); and > with the second, I validated that uninstalling yajl-devel and then > trying to configure once again gives me the desired: > > checking for yajl_tree_parse in -lyajl... no > configure: error: You must install the libyajl library & headers to compile > libvirt > > [I can't claim build-breaker or trivial rule for either of these > patches, but as I am also the autoconf guru around here, I won't > be surprised if the extent of other's reviews is "yep, I installed > it and it did what you said"...] > > And yes, I tested on RHEL 5 that it still works with autoconf 2.59. > > Eric Blake (2): > build: avoid obsolete AC_HELP_STRING > build: fix regression in requiring yajl for new enough qemu > > configure.ac | 221 > +++++++++++++++++++++++++++++++++++----------------- > m4/virt-apparmor.m4 | 4 +- > m4/virt-lib.m4 | 12 +-- > m4/virt-selinux.m4 | 3 +- > 4 files changed, 158 insertions(+), 82 deletions(-) > -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org