[libvirt] [PATCH 08/89] configure: use LIBVIRT_ARG_WITH(_ALT) macros
Pavel Hrdina
phrdina at redhat.com
Fri Dec 16 09:10:36 UTC 2016
Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
---
configure.ac | 381 ++++++++++++++---------------------------------
m4/virt-apparmor.m4 | 6 +-
m4/virt-driver-bhyve.m4 | 5 +-
m4/virt-driver-uml.m4 | 5 +-
m4/virt-driver-vz.m4 | 5 +-
m4/virt-host-validate.m4 | 5 +-
m4/virt-init-script.m4 | 8 +-
m4/virt-lib.m4 | 21 +--
m4/virt-login-shell.m4 | 5 +-
m4/virt-nss.m4 | 7 +-
m4/virt-selinux.m4 | 6 +-
m4/virt-wireshark.m4 | 7 +-
12 files changed, 134 insertions(+), 327 deletions(-)
diff --git a/configure.ac b/configure.ac
index 3f8e785078..c4f0623b8c 100644
--- a/configure.ac
+++ b/configure.ac
@@ -96,14 +96,8 @@ AC_SUBST([LIBVIRT_VERSION])
AC_SUBST([LIBVIRT_VERSION_INFO])
AC_SUBST([LIBVIRT_VERSION_NUMBER])
-AC_ARG_WITH([packager],
- [AS_HELP_STRING([--with-packager],
- [Extra packager name])],
- [],[with_packager=no])
-AC_ARG_WITH([packager-version],
- [AS_HELP_STRING([--with-packager-version],
- [Extra packager version])],
- [],[with_packager_version=no])
+LIBVIRT_ARG_WITH_ALT([PACKAGER], [Extra packager name], [no])
+LIBVIRT_ARG_WITH_ALT([PACKAGER_VERSION], [Extra packager version], [no])
if test "x$with_packager" != "xno"
then
AC_DEFINE_UNQUOTED([PACKAGER], ["$with_packager"],
@@ -416,24 +410,22 @@ LIBVIRT_CHECK_EXTERNAL_PROGRAMS
dnl Specific dir for HTML output ?
-AC_ARG_WITH([html-dir], [AS_HELP_STRING([--with-html-dir=path],
- [path to base html directory, default $datadir/doc/html])],
- [HTML_DIR=$withval], [HTML_DIR='$(datadir)/doc'])
-
-AC_ARG_WITH([html-subdir], [AS_HELP_STRING([--with-html-subdir=path],
- [directory used under html-dir, default $PACKAGE-$VERSION/html])],
- [test "x$withval" != "x" && HTML_DIR="$HTML_DIR/$withval"],
- [HTML_DIR="$HTML_DIR/\$(PACKAGE)-\$(VERSION)/html"])
+LIBVIRT_ARG_WITH_ALT([HTML_DIR], [path to base html directory],
+ ['$(datadir)/doc'])
+LIBVIRT_ARG_WITH_ALT([HTML_SUBDIR], [directory used under html-dir],
+ ['$(PACKAGE)-$(VERSION)/html'])
+if test "x$with_html_subdir" != "x" ; then
+ HTML_DIR="$with_html_dir/$with_html_subdir"
+else
+ HTML_DIR="$with_html_dir"
+fi
AC_SUBST([HTML_DIR])
dnl Specific XML catalog file for validation of generated html
-AC_ARG_WITH([xml-catalog-file],
- [AS_HELP_STRING([--with-xml-catalog-file=path],
- [path to XML catalog file for validating
- generated html, default /etc/xml/catalog])],
- [XML_CATALOG_FILE=$withval],
- [XML_CATALOG_FILE='/etc/xml/catalog'])
-AC_SUBST([XML_CATALOG_FILE])
+LIBVIRT_ARG_WITH_ALT([XML_CATALOG_FILE],
+ [path to XML catalog file for validating generated html],
+ ['/etc/xml/catalog'])
+AC_SUBST([XML_CATALOG_FILE], [$with_xml_catalog_file])
dnl if --prefix is /usr, don't use /usr/var for localstatedir
dnl or /usr/etc for sysconfdir
@@ -447,75 +439,27 @@ if test "$prefix" = "/usr" && test "$sysconfdir" = '${prefix}/etc' ; then
fi
dnl Allow to build without Xen, QEMU/KVM, test or remote driver
-AC_ARG_WITH([xen],
- [AS_HELP_STRING([--with-xen],
- [add XEN support @<:@default=check@:>@])])
-m4_divert_text([DEFAULTS], [with_xen=check])
-AC_ARG_WITH([xen-inotify],
- [AS_HELP_STRING([--with-xen-inotify],
- [add XEN inotify support @<:@default=check@:>@])])
-m4_divert_text([DEFAULTS], [with_xen_inotify=check])
-AC_ARG_WITH([qemu],
- [AS_HELP_STRING([--with-qemu],
- [add QEMU/KVM support @<:@default=yes@:>@])])
-m4_divert_text([DEFAULTS], [with_qemu=yes])
-AC_ARG_WITH([openvz],
- [AS_HELP_STRING([--with-openvz],
- [add OpenVZ support @<:@default=check@:>@])])
-m4_divert_text([DEFAULTS], [with_openvz=check])
-AC_ARG_WITH([vmware],
- [AS_HELP_STRING([--with-vmware],
- [add VMware support @<:@default=yes@:>@])])
-m4_divert_text([DEFAULTS], [with_vmware=yes])
-AC_ARG_WITH([phyp],
- [AS_HELP_STRING([--with-phyp],
- [add PHYP support @<:@default=check@:>@])])
-m4_divert_text([DEFAULTS], [with_phyp=check])
-AC_ARG_WITH([xenapi],
- [AS_HELP_STRING([--with-xenapi],
- [add XenAPI support @<:@default=check@:>@])])
-m4_divert_text([DEFAULTS], [with_xenapi=check])
-AC_ARG_WITH([libxl],
- [AS_HELP_STRING([--with-libxl],
- [add libxenlight support @<:@default=check@:>@])])
-m4_divert_text([DEFAULTS], [with_libxl=check])
-AC_ARG_WITH([vbox],
- [AS_HELP_STRING([--with-vbox=@<:@PFX@:>@],
- [VirtualBox XPCOMC location @<:@default=yes@:>@])])
-m4_divert_text([DEFAULTS], [with_vbox=yes])
-AC_ARG_WITH([lxc],
- [AS_HELP_STRING([--with-lxc],
- [add Linux Container support @<:@default=check@:>@])])
-m4_divert_text([DEFAULTS], [with_lxc=check])
-AC_ARG_WITH([esx],
- [AS_HELP_STRING([--with-esx],
- [add ESX support @<:@default=check@:>@])])
-m4_divert_text([DEFAULTS], [with_esx=check])
-AC_ARG_WITH([hyperv],
- [AS_HELP_STRING([--with-hyperv],
- [add Hyper-V support @<:@default=check@:>@])])
-m4_divert_text([DEFAULTS], [with_hyperv=check])
-AC_ARG_WITH([test],
- [AS_HELP_STRING([--with-test],
- [add test driver support @<:@default=yes@:>@])])
-m4_divert_text([DEFAULTS], [with_test=yes])
-AC_ARG_WITH([remote],
- [AS_HELP_STRING([--with-remote],
- [add remote driver support @<:@default=yes@:>@])])
-m4_divert_text([DEFAULTS], [with_remote=yes])
-AC_ARG_WITH([libvirtd],
- [AS_HELP_STRING([--with-libvirtd],
- [add libvirtd support @<:@default=yes@:>@])])
-m4_divert_text([DEFAULTS], [with_libvirtd=yes])
-AC_ARG_WITH([chrdev-lock-files],
- [AS_HELP_STRING([--with-chrdev-lock-files],
- [location for UUCP style lock files for character devices
- (use auto for default paths on some platforms) @<:@default=auto@:>@])])
-m4_divert_text([DEFAULTS], [with_chrdev_lock_files=auto])
-AC_ARG_WITH([pm-utils],
- [AS_HELP_STRING([--with-pm-utils],
- [use pm-utils for power management @<:@default=yes@:>@])])
-m4_divert_text([DEFAULTS], [with_pm_utils=check])
+LIBVIRT_ARG_WITH([XEN], [XEN], [check])
+LIBVIRT_ARG_WITH([XEN_INOTIFY], [XEN inotify], [check])
+LIBVIRT_ARG_WITH([QEMU], [QEMU/KVM], [yes])
+LIBVIRT_ARG_WITH([OPENVZ], [OpenVZ], [check])
+LIBVIRT_ARG_WITH([VMWARE], [VMware], [yes])
+LIBVIRT_ARG_WITH([PHYP], [PHYP], [check])
+LIBVIRT_ARG_WITH([XENAPI], [XenAPI], [check])
+LIBVIRT_ARG_WITH([LIBXL], [libxenlight], [check])
+LIBVIRT_ARG_WITH([VBOX], [VirtualBox XPCOMC], [yes])
+LIBVIRT_ARG_WITH([LXC], [Linux Container], [check])
+LIBVIRT_ARG_WITH([ESX], [ESX], [check])
+LIBVIRT_ARG_WITH([HYPERV], [Hyper-V], [check])
+LIBVIRT_ARG_WITH([TEST], [test driver], [yes])
+LIBVIRT_ARG_WITH([REMOTE], [remote driver], [yes])
+LIBVIRT_ARG_WITH([LIBVIRTD], [libvirtd], [yes])
+
+LIBVIRT_ARG_WITH_ALT([CHRDEV_LOCK_FILES],
+ [location for UUCP style lock files for character devices ]
+ [(use auto for default paths on some platforms)],
+ [auto])
+LIBVIRT_ARG_WITH_ALT([PM_UTILS], [use pm-utils for power management], [check])
dnl
dnl in case someone want to build static binaries
@@ -537,10 +481,7 @@ fi
LIBVIRT_CHECK_INIT_SCRIPT
AC_MSG_CHECKING([for whether to install sysctl config])
-AC_ARG_WITH([sysctl],
- [AS_HELP_STRING([--with-sysctl@<:@=yes/no@:>@],
- [Whether to install sysctl configs @<:@default=check@:>@])],
- [],[with_sysctl=check])
+LIBVIRT_ARG_WITH_ALT([SYSCTL], [Whether to install sysctl configs], [check])
if test "$with_sysctl" = "yes" || test "$with_sysctl" = "check"
then
@@ -1033,16 +974,16 @@ LIBXML_CFLAGS=""
LIBXML_LIBS=""
LIBXML_FOUND="no"
-AC_ARG_WITH([libxml], [AS_HELP_STRING([--with-libxml=@<:@PFX@:>@],
- [libxml2 location])])
+LIBVIRT_ARG_WITH_ALT([LIBXML], [libxml2 (>= 2.6.0) location], [check])
+
if test "x$with_libxml" = "xno" ; then
AC_MSG_CHECKING(for libxml2 libraries >= $LIBXML_REQUIRED)
AC_MSG_ERROR([libxml2 >= $LIBXML_REQUIRED is required for libvirt])
-elif test "x$with_libxml" = "x" && test "x$PKG_CONFIG" != "x" ; then
+elif test "x$with_libxml" != "xno" && test "x$PKG_CONFIG" != "x" ; then
PKG_CHECK_MODULES(LIBXML, libxml-2.0 >= $LIBXML_REQUIRED, [LIBXML_FOUND=yes], [LIBXML_FOUND=no])
fi
if test "$LIBXML_FOUND" = "no" ; then
- if test "x$with_libxml" != "x" ; then
+ if test "x$with_libxml" != "xyes" ; then
LIBXML_CONFIG=$with_libxml/bin/$LIBXML_CONFIG
fi
AC_MSG_CHECKING(libxml2 $LIBXML_CONFIG >= $LIBXML_REQUIRED )
@@ -1077,11 +1018,9 @@ CFLAGS="$old_CFLAGS"
LIBS="$old_LIBS"
-AC_ARG_WITH([tls-priority],
- [AS_HELP_STRING([--with-tls-priority],
- [set the default TLS session priority string @<:@default=NORMAL@:>@])],
- [],
- [with_tls_priority=NORMAL])
+LIBVIRT_ARG_WITH_ALT([TLS_PRIORITY],
+ [set the default TLS session priority string],
+ [NORMAL])
AC_DEFINE_UNQUOTED([TLS_PRIORITY], ["$with_tls_priority"],
[TLS default priority string])
@@ -1091,11 +1030,8 @@ dnl PolicyKit library
POLKIT_CFLAGS=
POLKIT_LIBS=
PKCHECK_PATH=
-AC_ARG_WITH([polkit],
- [AS_HELP_STRING([--with-polkit],
- [use PolicyKit for UNIX socket access checks @<:@default=check@:>@])],
- [],
- [with_polkit=check])
+LIBVIRT_ARG_WITH_ALT([POLKIT], [use PolicyKit for UNIX socket access checks],
+ [check])
with_polkit0=no
with_polkit1=no
@@ -1162,11 +1098,8 @@ AC_SUBST([POLKIT_CFLAGS])
AC_SUBST([POLKIT_LIBS])
dnl firewalld
-AC_ARG_WITH([firewalld],
- [AS_HELP_STRING([--with-firewalld],
- [enable firewalld support @<:@default=check@:>@])],
- [],
- [with_firewalld=check])
+LIBVIRT_ARG_WITH([FIREWALLD], [firewalld], [check])
+
if test "x$with_firewalld" = "xcheck" ; then
with_firewalld=$with_dbus
fi
@@ -1202,11 +1135,7 @@ fi
AM_CONDITIONAL([VIR_CHRDEV_LOCK_FILE_PATH], [test "$with_chrdev_lock_files" != "no"])
-AC_ARG_WITH([secdriver-selinux],
- [AS_HELP_STRING([--with-secdriver-selinux],
- [use SELinux security driver @<:@default=check@:>@])],
- [],
- [with_secdriver_selinux=check])
+LIBVIRT_ARG_WITH_ALT([SECDRIVER_SELINUX], [use SELinux security driver], [check])
if test "$with_selinux" != "yes" ; then
if test "$with_secdriver_selinux" = "check" ; then
@@ -1242,11 +1171,7 @@ fi
AM_CONDITIONAL([WITH_SECDRIVER_SELINUX], [test "$with_secdriver_selinux" != "no"])
-AC_ARG_WITH([secdriver-apparmor],
- [AS_HELP_STRING([--with-secdriver-apparmor],
- [use AppArmor security driver @<:@default=check@:>@])],
- [],
- [with_secdriver_apparmor=check])
+LIBVIRT_ARG_WITH_ALT([SECDRIVER_APPARMOR], [use AppArmor security driver], [check])
if test "$with_apparmor" != "yes" ; then
if test "$with_secdriver_apparmor" = "check" ; then
@@ -1261,22 +1186,14 @@ elif test "with_secdriver_apparmor" != "no" ; then
fi
AM_CONDITIONAL([WITH_SECDRIVER_APPARMOR], [test "$with_secdriver_apparmor" != "no"])
-AC_ARG_WITH([apparmor-profiles],
- [AS_HELP_STRING([--with-apparmor-profiles],
- [install apparmor profiles @<:@default=no@:>@])],
- [with_apparmor_profiles=yes],
- [with_apparmor_profiles=no])
+LIBVIRT_ARG_WITH_ALT([APPARMOR_PROFILES], [install apparmor profiles], [no])
if test "$with_apparmor" = "no"; then
with_apparmor_profiles="no"
fi
AM_CONDITIONAL([WITH_APPARMOR_PROFILES], [test "$with_apparmor_profiles" != "no"])
dnl DTrace static probes
-AC_ARG_WITH([dtrace],
- [AS_HELP_STRING([--with-dtrace],
- [use dtrace for static probing @<:@default=check@:>@])],
- [],
- [with_dtrace=check])
+LIBVIRT_ARG_WITH_ALT([DTRACE], [use dtrace for static probing], [check])
if test "$with_dtrace" != "no" ; then
AC_PATH_PROG([DTRACE], [dtrace], [], [$LIBVIRT_SBIN_PATH])
@@ -1297,11 +1214,7 @@ AM_CONDITIONAL([WITH_DTRACE_PROBES], [test "$with_dtrace" != "no"])
dnl numad
-AC_ARG_WITH([numad],
- [AS_HELP_STRING([--with-numad],
- [use numad to manage CPU placement dynamically @<:@default=check@:>@])],
- [],
- [with_numad=check])
+LIBVIRT_ARG_WITH_ALT([NUMAD], [use numad to manage CPU placement dynamically], [check])
if test "$with_numad" != "no" ; then
fail=0
@@ -1336,12 +1249,11 @@ LIBPCAP_CFLAGS=""
LIBPCAP_LIBS=""
LIBPCAP_FOUND="no"
-AC_ARG_WITH([libpcap], [AS_HELP_STRING([--with-libpcap=@<:@PFX@:>@],
- [libpcap location])])
+LIBVIRT_ARG_WITH_ALT([LIBPCAP], [libpcap location], [check])
if test "$with_qemu" = "yes"; then
case $with_libpcap in
no) LIBPCAP_CONFIG= ;;
- ''|yes) LIBPCAP_CONFIG="pcap-config" ;;
+ ''|yes|check) LIBPCAP_CONFIG="pcap-config" ;;
*) LIBPCAP_CONFIG="$with_libpcap/bin/pcap-config" ;;
esac
AS_IF([test "x$LIBPCAP_CONFIG" != "x"], [
@@ -1413,10 +1325,7 @@ AM_CONDITIONAL([WITH_PM_UTILS], [test "$with_pm_utils" = "yes"])
dnl check if the network driver should be compiled
-AC_ARG_WITH([network],
- [AS_HELP_STRING([--with-network],
- [with virtual network driver @<:@default=yes@:>@])],
- [],[with_network=yes])
+LIBVIRT_ARG_WITH_ALT([NETWORK], [with virtual network driver], [yes])
dnl there's no use compiling the network driver without the libvirt
dnl daemon, nor compiling it for MacOS X, where it breaks the compile
@@ -1438,10 +1347,7 @@ fi
AM_CONDITIONAL([WITH_BRIDGE], [test "$with_bridge" = "yes"])
-AC_ARG_WITH([secrets],
- [AS_HELP_STRING([--with-secrets],
- [with local secrets management driver @<:@default=yes@:>@])],
- [],[with_secrets=yes])
+LIBVIRT_ARG_WITH([SECRETS], [local secrets management driver], [yes])
if test "$with_libvirtd" = "no"; then
with_secrets=no
@@ -1452,51 +1358,17 @@ fi
AM_CONDITIONAL([WITH_SECRETS], [test "$with_secrets" = "yes"])
-AC_ARG_WITH([storage-dir],
- [AS_HELP_STRING([--with-storage-dir],
- [with directory backend for the storage driver @<:@default=yes@:>@])],
- [],[with_storage_dir=yes])
-AC_ARG_WITH([storage-fs],
- [AS_HELP_STRING([--with-storage-fs],
- [with FileSystem backend for the storage driver @<:@default=check@:>@])],
- [],[with_storage_fs=check])
-AC_ARG_WITH([storage-lvm],
- [AS_HELP_STRING([--with-storage-lvm],
- [with LVM backend for the storage driver @<:@default=check@:>@])],
- [],[with_storage_lvm=check])
-AC_ARG_WITH([storage-iscsi],
- [AS_HELP_STRING([--with-storage-iscsi],
- [with iSCSI backend for the storage driver @<:@default=check@:>@])],
- [],[with_storage_iscsi=check])
-AC_ARG_WITH([storage-scsi],
- [AS_HELP_STRING([--with-storage-scsi],
- [with SCSI backend for the storage driver @<:@default=check@:>@])],
- [],[with_storage_scsi=check])
-AC_ARG_WITH([storage-mpath],
- [AS_HELP_STRING([--with-storage-mpath],
- [with mpath backend for the storage driver @<:@default=check@:>@])],
- [],[with_storage_mpath=check])
-AC_ARG_WITH([storage-disk],
- [AS_HELP_STRING([--with-storage-disk],
- [with GPartd Disk backend for the storage driver @<:@default=check@:>@])],
- [],[with_storage_disk=check])
-AC_ARG_WITH([storage-rbd],
- [AS_HELP_STRING([--with-storage-rbd],
- [with RADOS Block Device backend for the storage driver
- @<:@default=check@:>@])],
- [],[with_storage_rbd=check])
-AC_ARG_WITH([storage-sheepdog],
- [AS_HELP_STRING([--with-storage-sheepdog],
- [with Sheepdog backend for the storage driver @<:@default=check@:>@])],
- [],[with_storage_sheepdog=check])
-AC_ARG_WITH([storage-gluster],
- [AS_HELP_STRING([--with-storage-gluster],
- [with Gluster backend for the storage driver @<:@default=check@:>@])],
- [],[with_storage_gluster=check])
-AC_ARG_WITH([storage-zfs],
- [AS_HELP_STRING([--with-storage-zfs],
- [with ZFS backend for the storage driver @<:@default=check@:>@])],
- [],[with_storage_zfs=check])
+LIBVIRT_ARG_WITH([STORAGE_DIR], [directory backend for the storage driver], [yes])
+LIBVIRT_ARG_WITH([STORAGE_FS], [FileSystem backend for the storage driver], [check])
+LIBVIRT_ARG_WITH([STORAGE_LVM], [LVM backend for storage driver], [check])
+LIBVIRT_ARG_WITH([STORAGE_ISCSI], [iSCSI backend for the storage driver], [check])
+LIBVIRT_ARG_WITH([STORAGE_SCSI], [SCSI backend for the storage driver], [check])
+LIBVIRT_ARG_WITH([STORAGE_MPATH], [mpath backend for the storage driver], [check])
+LIBVIRT_ARG_WITH([STORAGE_DISK], [GPartd Disk backend for the storage driver], [check])
+LIBVIRT_ARG_WITH([STORAGE_RBD], [RADOS Block Device backend for the storage driver], [check])
+LIBVIRT_ARG_WITH([STORAGE_SHEEPDOG], [with Sheepdog backend for the storage driver], [check])
+LIBVIRT_ARG_WITH([STORAGE_GLUSTER], [Gluster backend for the storage driver], [check])
+LIBVIRT_ARG_WITH([STORAGE_ZFS], [ZFS backend for the storage driver], [check])
if test "$with_libvirtd" = "no"; then
with_storage_dir=no
@@ -1917,25 +1789,22 @@ if test -z "$PERL"; then
AC_MSG_ERROR([Failed to find perl.])
fi
-AC_ARG_WITH([test-suite],
- [AS_HELP_STRING([--with-test-suite],
- [build test suite by default @<:@default=check@:>@])],
- [case "${withval}" in
- yes|no|check) ;;
- *) AC_MSG_ERROR([bad value ${withval} for tests option]) ;;
- esac],
- [withval=check])
+LIBVIRT_ARG_WITH_ALT([TEST_SUITE], [build test suite by default], [check])
+case "$with_test_suite" in
+ yes|no|check) ;;
+ *) AC_MSG_ERROR([bad value ${withval} for tests option]) ;;
+esac
AC_MSG_CHECKING([Whether to build test suite by default])
-if test "$withval" = "check" ; then
+if test "$with_test_suite" = "check" ; then
if test -d $srcdir/.git ; then
- withval=yes
+ with_test_suite=yes
else
- withval=no
+ with_test_suite=no
fi
fi
-AC_MSG_RESULT([$withval])
-AM_CONDITIONAL([WITH_TESTS], [test "$withval" = "yes"])
+AC_MSG_RESULT([$with_test_suite])
+AM_CONDITIONAL([WITH_TESTS], [test "$with_test_suite" = "yes"])
AC_ARG_ENABLE([expensive-tests],
[AS_HELP_STRING([--enable-expensive-tests],
@@ -2121,11 +1990,8 @@ AM_CONDITIONAL([WITH_WIN_ICON], [test "$WINDRES" != ""])
dnl Driver-Modules library
-AC_ARG_WITH([driver-modules],
- [AS_HELP_STRING([--with-driver-modules],
- [build drivers as loadable modules @<:@default=check@:>@])],
- [],
- [with_driver_modules=check])
+LIBVIRT_ARG_WITH_ALT([DRIVER_MODULES], [build drivers as loadable modules],
+ [check])
if test "$with_libvirtd" = "no" ; then
with_driver_modules=no
@@ -2187,10 +2053,7 @@ fi
AM_CONDITIONAL([WITH_NWFILTER], [test "$with_nwfilter" = "yes"])
dnl check if the interface driver should be compiled
-AC_ARG_WITH([interface],
- [AS_HELP_STRING([--with-interface],
- [with host interface driver @<:@default=check@:>@])],
- [],[with_interface=check])
+LIBVIRT_ARG_WITH([INTERFACE], [host interface driver], [check])
dnl Don't compile the interface driver without libvirtd
if test "$with_libvirtd" = "no" ; then
@@ -2219,18 +2082,20 @@ else
default_qemu_group=root
fi
-AC_ARG_WITH([qemu-user],
- [AS_HELP_STRING([--with-qemu-user],
- [username to run QEMU system instance as
- @<:@default=platform dependent@:>@])],
- [QEMU_USER=${withval}],
- [QEMU_USER=${default_qemu_user}])
-AC_ARG_WITH([qemu-group],
- [AS_HELP_STRING([--with-qemu-group],
- [groupname to run QEMU system instance as
- @<:@default=platform dependent@:>@])],
- [QEMU_GROUP=${withval}],
- [QEMU_GROUP=${default_qemu_group}])
+LIBVIRT_ARG_WITH_ALT([QEMU_USER], [username to run QEMU system instance as],
+ ['platform dependent'])
+LIBVIRT_ARG_WITH_ALT([QEMU_GROUP], [groupname to run QEMU system instance as],
+ ['platform dependent'])
+if test "x$with_qemu_user" = "xplatform dependent" ; then
+ QEMU_USER="$default_qemu_user"
+else
+ QEMU_USER="$with_qemu_user"
+fi
+if test "x$with_qemu_group" = "xplatform dependent" ; then
+ QEMU_GROUP="$default_qemu_group"
+else
+ QEMU_GROUP="$with_qemu_group"
+fi
AC_DEFINE_UNQUOTED([QEMU_USER], ["$QEMU_USER"], [QEMU user account])
AC_DEFINE_UNQUOTED([QEMU_GROUP], ["$QEMU_GROUP"], [QEMU group account])
@@ -2238,11 +2103,7 @@ AC_PATH_PROG([QEMU_BRIDGE_HELPER], [qemu-bridge-helper], [/usr/libexec/qemu-brid
[/usr/libexec:/usr/lib/qemu:/usr/lib])
AC_DEFINE_UNQUOTED([QEMU_BRIDGE_HELPER], ["$QEMU_BRIDGE_HELPER"], [QEMU bridge helper])
-AC_ARG_WITH([macvtap],
- [AS_HELP_STRING([--with-macvtap],
- [enable macvtap device @<:@default=check@:>@])],
- [with_macvtap=${withval}],
- [with_macvtap=check])
+LIBVIRT_ARG_WITH_ALT([MACVTAP], [enable macvtap device], [check])
AC_MSG_CHECKING([whether to compile with macvtap support])
if test "$with_macvtap" != "no" ; then
@@ -2272,11 +2133,7 @@ if test "$with_macvtap" = yes; then
fi
-AC_ARG_WITH([virtualport],
- [AS_HELP_STRING([--with-virtualport],
- [enable virtual port support @<:@default=check@:>@])],
- [with_virtualport=${withval}],
- [with_virtualport=check])
+LIBVIRT_ARG_WITH_ALT([VIRTUALPORT], [enable virtual port support], [check])
dnl Warn the user and error out if they requested virtualport support with configure
dnl options, but the required macvtap support isn't available
@@ -2467,30 +2324,22 @@ test "x$lv_cv_static_analysis" = xyes && t=1
AC_DEFINE_UNQUOTED([STATIC_ANALYSIS], [$t],
[Define to 1 when performing static analysis.])
-AC_ARG_WITH([default-editor],
- [AS_HELP_STRING([--with-default-editor],
- [Editor to use for interactive commands
- @<:@default=vi@:>@])],
- [DEFAULT_EDITOR=${withval}],
- [DEFAULT_EDITOR=vi])
-AC_DEFINE_UNQUOTED([DEFAULT_EDITOR], ["$DEFAULT_EDITOR"], [Default editor to use])
-
-AC_ARG_WITH([loader-nvram],
- [AS_HELP_STRING([--with-loader-nvram],
- [Pass list of pairs of <loader>:<nvram> paths. Both
- pairs and list items are separated by a colon.
- @<:default=paths to OVMF and its clones@:>@])],
- [if test "$withval" = "no"; then
- withval=""
- else
- l=`echo $withval | tr ':' '\n' | wc -l`
- if test "`expr $l % 2`" -ne 0; then
- AC_MSG_ERROR([Malformed --with-loader-nvram argument])
- fi
- fi
- AC_DEFINE_UNQUOTED([DEFAULT_LOADER_NVRAM],
- ["$withval"],
- [List of loader:nvram pairs])])
+LIBVIRT_ARG_WITH_ALT([DEFAULT_EDITOR], [Editor to use for interactive commands], [vi])
+AC_DEFINE_UNQUOTED([DEFAULT_EDITOR], ["$with_default_editor"], [Default editor to use])
+
+LIBVIRT_ARG_WITH_ALT([LOADER_NVRAM],
+ [Pass list of pairs of <loader>:<nvram> paths.
+ Both pairs and list items are separated by a colon.],
+ [''])
+if test "x$with_loader_nvram" != "xno" && \
+ test "x$with_loader_nvram" != "x" ; then
+ l=$(echo $with_loader_nvram | tr ':' '\n' | wc -l)
+ if test $(expr $l % 2) -ne 0 ; then
+ AC_MSG_ERROR([Malformed --with-loader-nvram argument])
+ fi
+ AC_DEFINE_UNQUOTED([DEFAULT_LOADER_NVRAM], [$with_loader_nvram],
+ [List of loader:nvram pairs])
+fi
# Some GNULIB base64 symbols clash with a kerberos library
AC_DEFINE_UNQUOTED([isbase64],[libvirt_gl_isbase64],[Hack to avoid symbol clash])
@@ -2669,10 +2518,10 @@ AC_MSG_NOTICE([ Use -Werror: $set_werror])
AC_MSG_NOTICE([ Warning Flags: $WARN_CFLAGS])
AC_MSG_NOTICE([ DTrace: $with_dtrace])
AC_MSG_NOTICE([ numad: $with_numad])
-AC_MSG_NOTICE([ XML Catalog: $XML_CATALOG_FILE])
+AC_MSG_NOTICE([ XML Catalog: $with_xml_catalog_file])
LIBVIRT_RESULT_INIT_SCRIPT
AC_MSG_NOTICE([ Char device locks: $with_chrdev_lock_files])
-AC_MSG_NOTICE([ Default Editor: $DEFAULT_EDITOR])
+AC_MSG_NOTICE([ Default Editor: $with_default_editor])
AC_MSG_NOTICE([ Loader/NVRAM: $with_loader_nvram])
LIBVIRT_RESULT_LOGIN_SHELL
LIBVIRT_RESULT_HOST_VALIDATE
diff --git a/m4/virt-apparmor.m4 b/m4/virt-apparmor.m4
index 3b7fc37aa7..eb017d181b 100644
--- a/m4/virt-apparmor.m4
+++ b/m4/virt-apparmor.m4
@@ -21,11 +21,7 @@ AC_DEFUN([LIBVIRT_CHECK_APPARMOR],[
LIBVIRT_CHECK_LIB([APPARMOR], [apparmor],
[aa_change_profile], [sys/apparmor.h])
- AC_ARG_WITH([apparmor_mount],
- [AS_HELP_STRING([--with-apparmor-mount],
- [set AppArmor mount point @<:@default=check@:>@])],
- [],
- [with_apparmor_mount=check])
+ LIBVIRT_ARG_WITH_ALT([APPARMOR_MOUNT], [set AppArmor mount point], [check])
if test "$with_apparmor" = "yes"; then
AC_DEFINE_UNQUOTED([APPARMOR_DIR],
diff --git a/m4/virt-driver-bhyve.m4 b/m4/virt-driver-bhyve.m4
index 11d3a667ee..c83eff08e9 100644
--- a/m4/virt-driver-bhyve.m4
+++ b/m4/virt-driver-bhyve.m4
@@ -18,10 +18,7 @@ dnl <http://www.gnu.org/licenses/>.
dnl
AC_DEFUN([LIBVIRT_DRIVER_CHECK_BHYVE],[
- AC_ARG_WITH([bhyve],
- [AS_HELP_STRING([--with-bhyve],
- [add BHyVe support @<:@default=check@:>@])])
- m4_divert_text([DEFAULTS], [with_bhyve=check])
+ LIBVIRT_ARG_WITH([BHYVE], [BHyVe], [check])
if test "$with_bhyve" != "no"; then
AC_PATH_PROG([BHYVE], [bhyve], [], [$PATH:/usr/sbin])
diff --git a/m4/virt-driver-uml.m4 b/m4/virt-driver-uml.m4
index 7b252e61ea..bc3f7d5a45 100644
--- a/m4/virt-driver-uml.m4
+++ b/m4/virt-driver-uml.m4
@@ -18,10 +18,7 @@ dnl <http://www.gnu.org/licenses/>.
dnl
AC_DEFUN([LIBVIRT_DRIVER_CHECK_UML],[
- AC_ARG_WITH([uml],
- [AS_HELP_STRING([--with-uml],
- [add UML support @<:@default=check@:>@])])
- m4_divert_text([DEFAULTS], [with_uml=check])
+ LIBVIRT_ARG_WITH([UML], [UML], [check])
if test "$with_libvirtd" = "no" || test "$with_linux" = "no"; then
if test "$with_uml" = "yes"; then
diff --git a/m4/virt-driver-vz.m4 b/m4/virt-driver-vz.m4
index 7b7a8d5a94..a51857467e 100644
--- a/m4/virt-driver-vz.m4
+++ b/m4/virt-driver-vz.m4
@@ -18,10 +18,7 @@ dnl <http://www.gnu.org/licenses/>.
dnl
AC_DEFUN([LIBVIRT_DRIVER_CHECK_VZ],[
- AC_ARG_WITH([vz],
- [AS_HELP_STRING([--with-vz],
- [add Virtuozzo support @<:@default=check@:>@])])
- m4_divert_text([DEFAULTS], [with_vz=check])
+ LIBVIRT_ARG_WITH([VZ], [Virtuozzo], [check])
if test "$with_vz" = "yes" ||
test "$with_vz" = "check"; then
diff --git a/m4/virt-host-validate.m4 b/m4/virt-host-validate.m4
index 8c42303095..3aca34deee 100644
--- a/m4/virt-host-validate.m4
+++ b/m4/virt-host-validate.m4
@@ -16,10 +16,7 @@ dnl License along with this library. If not, see
dnl <http://www.gnu.org/licenses/>.
AC_DEFUN([LIBVIRT_CHECK_HOST_VALIDATE], [
- AC_ARG_WITH([host_validate],
- [AS_HELP_STRING([--with-host-validate],
- [build virt-host-validate @<:@default=check@:>@])])
- m4_divert_text([DEFAULTS], [with_host_validate=check])
+ LIBVIRT_ARG_WITH_ALT([HOST_VALIDATE], [build virt-host-validate], [check])
if test "x$with_host_validate" != "xno"; then
if test "x$with_win" = "xyes"; then
diff --git a/m4/virt-init-script.m4 b/m4/virt-init-script.m4
index c307b0e9b2..1eb43a9288 100644
--- a/m4/virt-init-script.m4
+++ b/m4/virt-init-script.m4
@@ -18,11 +18,9 @@ dnl <http://www.gnu.org/licenses/>.
dnl
AC_DEFUN([LIBVIRT_CHECK_INIT_SCRIPT],[
- AC_ARG_WITH([init-script],
- [AS_HELP_STRING([--with-init-script@<:@=STYLE@:>@],
- [Style of init script to install: redhat, systemd, systemd+redhat,
- upstart, check, none @<:@default=check@:>@])],
- [],[with_init_script=check])
+ LIBVIRT_ARG_WITH_ALT([INIT_SCRIPT],
+ [Style of init script to install: redhat, systemd,
+ systemd+redhat, upstart, check, none], [check])
AC_MSG_CHECKING([for init script type])
diff --git a/m4/virt-lib.m4 b/m4/virt-lib.m4
index 87c23d4552..473d184880 100644
--- a/m4/virt-lib.m4
+++ b/m4/virt-lib.m4
@@ -60,13 +60,9 @@ AC_DEFUN([LIBVIRT_CHECK_LIB],[
m4_pushdef([make_var], [WITH_]check_name)
m4_pushdef([cflags_var], check_name[_CFLAGS])
m4_pushdef([libs_var], check_name[_LIBS])
- m4_pushdef([arg_var], [with-]check_name_dash)
m4_pushdef([with_var], [with_]check_name_lc)
- m4_divert_text([DEFAULTS], [with_var][=check])
- AC_ARG_WITH(check_name_dash,
- [AS_HELP_STRING([--arg_var],
- [with lib]]m4_dquote(library_name)[[ support @<:@default=check@:>@])])
+ LIBVIRT_ARG_WITH(check_name, library_name, [check])
old_LIBS=$LIBS
old_CFLAGS=$CFLAGS
@@ -121,7 +117,6 @@ AC_DEFUN([LIBVIRT_CHECK_LIB],[
fi
m4_popdef([with_var])
- m4_popdef([arg_var])
m4_popdef([libs_var])
m4_popdef([cflags_var])
m4_popdef([make_var])
@@ -190,15 +185,11 @@ AC_DEFUN([LIBVIRT_CHECK_LIB_ALT],[
m4_pushdef([make_var], [WITH_]check_name)
m4_pushdef([cflags_var], check_name[_CFLAGS])
m4_pushdef([libs_var], check_name[_LIBS])
- m4_pushdef([arg_var], [with-]check_name_dash)
m4_pushdef([with_var], [with_]check_name_lc)
m4_pushdef([config_var_alt], [WITH_]check_name_alt)
m4_pushdef([make_var_alt], [WITH_]check_name_alt)
- m4_divert_text([DEFAULTS], [with_var][=check])
- AC_ARG_WITH(check_name_dash,
- [AS_HELP_STRING([--arg_var],
- [with lib]]m4_dquote(library_name)[[ support @<:@default=check@:>@])])
+ LIBVIRT_ARG_WITH(check_name, library_name, [check])
old_LIBS=$LIBS
old_CFLAGS=$CFLAGS
@@ -269,7 +260,6 @@ AC_DEFUN([LIBVIRT_CHECK_LIB_ALT],[
m4_popdef([make_var_alt])
m4_popdef([config_var_alt])
m4_popdef([with_var])
- m4_popdef([arg_var])
m4_popdef([libs_var])
m4_popdef([cflags_var])
m4_popdef([make_var])
@@ -320,13 +310,9 @@ AC_DEFUN([LIBVIRT_CHECK_PKG],[
m4_pushdef([make_var], [WITH_]check_name)
m4_pushdef([cflags_var], check_name[_CFLAGS])
m4_pushdef([libs_var], check_name[_LIBS])
- m4_pushdef([arg_var], [with-]check_name_dash)
m4_pushdef([with_var], [with_]check_name_lc)
- m4_divert_text([DEFAULTS], [with_var][=check])
- AC_ARG_WITH(check_name_dash,
- [AS_HELP_STRING([--arg_var],
- [with ]]m4_dquote(pc_name)[[ (>= ]]m4_dquote(pc_version)[[) support @<:@default=check@:>@])])
+ LIBVIRT_ARG_WITH(check_name, pc_name, [check], pc_version)
fail=0
if test "x$with_var" != "xno" ; then
@@ -351,7 +337,6 @@ AC_DEFUN([LIBVIRT_CHECK_PKG],[
AM_CONDITIONAL(make_var, [test "x$with_var" = "xyes"])
m4_popdef([with_var])
- m4_popdef([arg_var])
m4_popdef([libs_var])
m4_popdef([cflags_var])
m4_popdef([make_var])
diff --git a/m4/virt-login-shell.m4 b/m4/virt-login-shell.m4
index fb5b488d22..a2b90dae6e 100644
--- a/m4/virt-login-shell.m4
+++ b/m4/virt-login-shell.m4
@@ -16,10 +16,7 @@ dnl License along with this library. If not, see
dnl <http://www.gnu.org/licenses/>.
AC_DEFUN([LIBVIRT_CHECK_LOGIN_SHELL], [
- AC_ARG_WITH([login_shell],
- [AS_HELP_STRING([--with-login-shell],
- [build virt-login-shell @<:@default=check@:>@])])
- m4_divert_text([DEFAULTS], [with_login_shell=check])
+ LIBVIRT_ARG_WITH_ALT([LOGIN_SHELL], [build virt-login-shell], [check])
if test "x$with_login_shell" != "xno"; then
if test "x$with_linux" != "xyes"; then
diff --git a/m4/virt-nss.m4 b/m4/virt-nss.m4
index bf5c416c31..f95010d692 100644
--- a/m4/virt-nss.m4
+++ b/m4/virt-nss.m4
@@ -18,10 +18,9 @@ dnl <http://www.gnu.org/licenses/>.
dnl
AC_DEFUN([LIBVIRT_CHECK_NSS],[
- AC_ARG_WITH([nss-plugin],
- [AS_HELP_STRING([--with-nss-plugin],
- [enable Name Service Switch plugin for resolving guest IP addresses])],
- [], [with_nss_plugin=check])
+ LIBVIRT_ARG_WITH_ALT([NSS_PLUGIN],
+ [enable Name Service Switch plugin for resolving guest
+ IP addresses], [check])
bsd_nss=no
fail=0
diff --git a/m4/virt-selinux.m4 b/m4/virt-selinux.m4
index a65e978465..afc4772c2d 100644
--- a/m4/virt-selinux.m4
+++ b/m4/virt-selinux.m4
@@ -21,11 +21,7 @@ AC_DEFUN([LIBVIRT_CHECK_SELINUX],[
LIBVIRT_CHECK_LIB([SELINUX], [selinux],
[fgetfilecon_raw], [selinux/selinux.h])
- AC_ARG_WITH([selinux_mount],
- [AS_HELP_STRING([--with-selinux-mount],
- [set SELinux mount point @<:@default=check@:>@])],
- [],
- [with_selinux_mount=check])
+ LIBVIRT_ARG_WITH_ALT([SELINUX_MOUNT], [set SELinux mount point], [check])
if test "$with_selinux" = "yes"; then
# libselinux changed signatures between 2.2 and 2.3
diff --git a/m4/virt-wireshark.m4 b/m4/virt-wireshark.m4
index d5d7404382..89976ba3f9 100644
--- a/m4/virt-wireshark.m4
+++ b/m4/virt-wireshark.m4
@@ -20,10 +20,9 @@ dnl
AC_DEFUN([LIBVIRT_CHECK_WIRESHARK],[
LIBVIRT_CHECK_PKG([WIRESHARK_DISSECTOR], [wireshark], [1.11.3])
- AC_ARG_WITH([ws-plugindir],
- [AS_HELP_STRING([--with-ws-plugindir],
- [wireshark plugins directory for use when installing wireshark plugin])],
- [], [with_ws_plugindir=check])
+ LIBVIRT_ARG_WITH_ALT([WS_PLUGINDIR],
+ [wireshark plugins directory for use when installing
+ wireshark plugin], [check])
dnl Check for system location of wireshark plugins
if test "x$with_wireshark_dissector" != "xno" ; then
--
2.11.0
More information about the libvir-list
mailing list