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

Jonathan Lebon jlebon at redhat.com
Tue Sep 10 17:43:51 UTC 2013


I can confirm that configure now happily yells at me.
Thanks,

Jonathan

----- Original Message -----
> 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
> 
> 




More information about the libvir-list mailing list