[libvirt] [PATCH 1/2] build: properly handle ./configure --with-libpcap

[Eric Blake]

On 12/13/2010 06:44 AM, Diego Elio Pettenò wrote:
> Without this fix, ./configure --with-libpcap will cause --with-libpcap=yes
> to be implicitly passed down, which cause yes/bin/pcap-config to be
> searched for rather than /usr/bin/pcap-config.
> 
> Also output pcap: no when pcap is not found or disabled.
> ---
>  configure.ac |   30 ++++++++++++++++--------------
>  1 files changed, 16 insertions(+), 14 deletions(-)

You forgot to run 'make syntax-check', which flagged a problem here:

> @@ -2406,7 +2408,7 @@ AC_MSG_NOTICE([  xmlrpc: $XMLRPC_CFLAGS $XMLRPC_LIBS])
>  else
>  AC_MSG_NOTICE([  xmlrpc: no])
>  fi
> -if test "$with_qemu" = "yes" ; then
> +if test "$with_qemu" = "yes" -a "$LIBPCAP_FOUND" != "no"; then

This must be test abc && test xyz, not test abc -a xyz, to avoid
portability problems.

ACK with that nit fixed, so I pushed it.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20101213/760b64a1/attachment-0001.sig>