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]

 



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

Attachment: signature.asc
Description: OpenPGP 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]