From 61418e5584eff16b9d83bdd8f34393b1f93db2e7 Mon Sep 17 00:00:00 2001 From: Pavel Hrdina Date: Wed, 7 Dec 2016 10:16:19 +0100 Subject: [PATCH] configure: move storage fs check to its own file Signed-off-by: Pavel Hrdina --- configure.ac | 50 ++---------------------- m4/virt-storage-fs.m4 | 89 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 92 insertions(+), 47 deletions(-) create mode 100644 m4/virt-storage-fs.m4 diff --git a/configure.ac b/configure.ac index 611e8da1e4..76845d5474 100644 --- a/configure.ac +++ b/configure.ac @@ -556,7 +556,7 @@ AM_CONDITIONAL([WITH_SECRETS], [test "$with_secrets" = "yes"]) LIBVIRT_STORAGE_ARG_DIR -LIBVIRT_ARG_WITH([STORAGE_FS], [FileSystem backend for the storage driver], [check]) +LIBVIRT_STORAGE_ARG_FS 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]) @@ -589,50 +589,7 @@ if test "$with_osx" = "yes"; then with_storage_fs=no fi -if test "$with_storage_fs" = "yes" || test "$with_storage_fs" = "check"; then - AC_CHECK_HEADER([mntent.h],, - [ - if test "$with_storage_fs" = "check"; then - with_storage_fs=no - AC_MSG_NOTICE([ is required for the FS storage driver, disabling it]) - else - AC_MSG_ERROR([ is required for the FS storage driver]) - fi - ]) -fi - -if test "$with_storage_fs" = "yes" || test "$with_storage_fs" = "check"; then - AC_PATH_PROG([MOUNT], [mount], [], [$LIBVIRT_SBIN_PATH]) - AC_PATH_PROG([UMOUNT], [umount], [], [$LIBVIRT_SBIN_PATH]) - AC_PATH_PROG([MKFS], [mkfs], [], [$LIBVIRT_SBIN_PATH]) - if test "$with_storage_fs" = "yes" ; then - if test -z "$MOUNT" ; then AC_MSG_ERROR([We need mount for FS storage driver]) ; fi - if test -z "$UMOUNT" ; then AC_MSG_ERROR([We need umount for FS storage driver]) ; fi - if test -z "$MKFS" ; then AC_MSG_ERROR([We need mkfs for FS storage driver]) ; fi - else - if test -z "$MOUNT" ; then with_storage_fs=no ; fi - if test -z "$UMOUNT" ; then with_storage_fs=no ; fi - if test -z "$MKFS" ; then with_storage_fs=no ; fi - - if test "$with_storage_fs" = "check" ; then with_storage_fs=yes ; fi - fi - - if test "$with_storage_fs" = "yes" ; then - AC_DEFINE_UNQUOTED([WITH_STORAGE_FS], 1, [whether FS backend for storage driver is enabled]) - AC_DEFINE_UNQUOTED([MOUNT],["$MOUNT"], - [Location or name of the mount program]) - AC_DEFINE_UNQUOTED([UMOUNT],["$UMOUNT"], - [Location or name of the mount program]) - AC_DEFINE_UNQUOTED([MKFS],["$MKFS"], - [Location or name of the mkfs program]) - fi -fi -AM_CONDITIONAL([WITH_STORAGE_FS], [test "$with_storage_fs" = "yes"]) -if test "$with_storage_fs" = "yes"; then - AC_PATH_PROG([SHOWMOUNT], [showmount], [], [$LIBVIRT_SBIN_PATH]) - AC_DEFINE_UNQUOTED([SHOWMOUNT], ["$SHOWMOUNT"], - [Location or name of the showmount program]) -fi +LIBVIRT_STORAGE_CHECK_FS if test "$with_storage_lvm" = "yes" || test "$with_storage_lvm" = "check"; then AC_PATH_PROG([PVCREATE], [pvcreate], [], [$LIBVIRT_SBIN_PATH]) @@ -1186,8 +1143,7 @@ AC_MSG_NOTICE([]) AC_MSG_NOTICE([Storage Drivers]) AC_MSG_NOTICE([]) LIBVIRT_STORAGE_RESULT_DIR -AC_MSG_NOTICE([ FS: $with_storage_fs]) -AC_MSG_NOTICE([ NetFS: $with_storage_fs]) +LIBVIRT_STORAGE_RESULT_FS AC_MSG_NOTICE([ LVM: $with_storage_lvm]) AC_MSG_NOTICE([ iSCSI: $with_storage_iscsi]) AC_MSG_NOTICE([ SCSI: $with_storage_scsi]) diff --git a/m4/virt-storage-fs.m4 b/m4/virt-storage-fs.m4 new file mode 100644 index 0000000000..f2c7130dc4 --- /dev/null +++ b/m4/virt-storage-fs.m4 @@ -0,0 +1,89 @@ +dnl The storage fs check +dnl +dnl Copyright (C) 2016 Red Hat, Inc. +dnl +dnl This library is free software; you can redistribute it and/or +dnl modify it under the terms of the GNU Lesser General Public +dnl License as published by the Free Software Foundation; either +dnl version 2.1 of the License, or (at your option) any later version. +dnl +dnl This library is distributed in the hope that it will be useful, +dnl but WITHOUT ANY WARRANTY; without even the implied warranty of +dnl MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +dnl Lesser General Public License for more details. +dnl +dnl You should have received a copy of the GNU Lesser General Public +dnl License along with this library. If not, see +dnl . +dnl + +AC_DEFUN([LIBVIRT_STORAGE_ARG_FS], [ + LIBVIRT_ARG_WITH([STORAGE_FS], [FileSystem backend for the storage driver], + [check]) +]) + +AC_DEFUN([LIBVIRT_STORAGE_CHECK_FS], [ + if test "$with_storage_fs" = "yes" || test "$with_storage_fs" = "check"; then + AC_CHECK_HEADER([mntent.h], , [ + if test "$with_storage_fs" = "check"; then + with_storage_fs=no + AC_MSG_NOTICE([ is required for the FS storage driver, disabling it]) + else + AC_MSG_ERROR([ is required for the FS storage driver]) + fi + ]) + fi + + if test "$with_storage_fs" = "yes" || test "$with_storage_fs" = "check"; then + AC_PATH_PROG([MOUNT], [mount], [], [$LIBVIRT_SBIN_PATH]) + AC_PATH_PROG([UMOUNT], [umount], [], [$LIBVIRT_SBIN_PATH]) + AC_PATH_PROG([MKFS], [mkfs], [], [$LIBVIRT_SBIN_PATH]) + if test "$with_storage_fs" = "yes" ; then + if test -z "$MOUNT" ; then + AC_MSG_ERROR([We need mount for FS storage driver]) + fi + if test -z "$UMOUNT" ; then + AC_MSG_ERROR([We need umount for FS storage driver]) + fi + if test -z "$MKFS" ; then + AC_MSG_ERROR([We need mkfs for FS storage driver]) + fi + else + if test -z "$MOUNT" ; then + with_storage_fs=no + fi + if test -z "$UMOUNT" ; then + with_storage_fs=no + fi + if test -z "$MKFS" ; then + with_storage_fs=no + fi + + if test "$with_storage_fs" = "check" ; then + with_storage_fs=yes + fi + fi + + if test "$with_storage_fs" = "yes" ; then + AC_DEFINE_UNQUOTED([WITH_STORAGE_FS], 1, + [whether FS backend for storage driver is enabled]) + AC_DEFINE_UNQUOTED([MOUNT], ["$MOUNT"], + [Location or name of the mount program]) + AC_DEFINE_UNQUOTED([UMOUNT], ["$UMOUNT"], + [Location or name of the mount program]) + AC_DEFINE_UNQUOTED([MKFS], ["$MKFS"], + [Location or name of the mkfs program]) + fi + fi + AM_CONDITIONAL([WITH_STORAGE_FS], [test "$with_storage_fs" = "yes"]) + if test "$with_storage_fs" = "yes"; then + AC_PATH_PROG([SHOWMOUNT], [showmount], [], [$LIBVIRT_SBIN_PATH]) + AC_DEFINE_UNQUOTED([SHOWMOUNT], ["$SHOWMOUNT"], + [Location or name of the showmount program]) + fi +]) + +AC_DEFUN([LIBVIRT_STORAGE_RESULT_FS], [ + LIBVIRT_RESULT([FS], [$with_storage_fs]) + LIBVIRT_RESULT([NetFS], [$with_storage_fs]) +]) -- GitLab