[libvirt] [PATCH] configure: tweak logic flow of virtport check

Justin Clift jclift at redhat.com
Wed Sep 29 14:22:03 UTC 2010


This fixes a small logic bug, where passing --without-macvtap
on the configure line, or otherwise indicating a lack of
support for macvtap, causes configure to bail.
---
 configure.ac |   13 ++++++++++++-
 1 files changed, 12 insertions(+), 1 deletions(-)

diff --git a/configure.ac b/configure.ac
index 8dcd756..f9b0d0f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2083,10 +2083,21 @@ AC_ARG_WITH([virtualport],
   [with_virtualport=${withval}],
   [with_virtualport=check])
 
-if test "$with_virtualport" != "no"; then
+dnl Warn the user and error out if they requested virtualport support with configure
+dnl options, but the required macvtap support isn't available
+
+if test "$with_virtualport" = "yes"; then
     if test "$with_macvtap" = "no"; then
         AC_MSG_ERROR([--with-virtualport requires --with-macvtap])
     fi
+fi
+
+dnl virtualport checks
+
+if test "$with_macvtap" != "yes"; then
+    with_virtualport=no
+fi
+if test "$with_virtualport" != "no"; then
     AC_MSG_CHECKING([whether to compile with virtual port support])
     AC_TRY_COMPILE([ #include <sys/socket.h>
                      #include <linux/rtnetlink.h> ],
-- 
1.7.2.3




More information about the libvir-list mailing list