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

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

 



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

--
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]