Re: [PATCH 1/3] storage: Add mkfs and libblkid to build system

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 31, 2011 at 10:34:46PM +0800, Osier Yang wrote:
> ---
>  configure.ac    |   26 ++++++++++++++++++++++++++
>  libvirt.spec.in |    4 ++++
>  2 files changed, 30 insertions(+), 0 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index 3c08a19..5e3539f 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -73,6 +73,7 @@ OPENWSMAN_REQUIRED="2.2.3"
>  LIBPCAP_REQUIRED="1.0.0"
>  LIBNL_REQUIRED="1.1"
>  LIBSSH2_REQUIRED="1.0"
> +LIBBLKID_REQUIRED="2.17"
>  
>  dnl Checks for C compiler.
>  AC_PROG_CC
> @@ -1643,12 +1644,15 @@ fi
>  if test "$with_storage_fs" = "yes" || test "$with_storage_fs" = "check"; then
>    AC_PATH_PROG([MOUNT], [mount], [], [$PATH:/sbin:/usr/sbin])
>    AC_PATH_PROG([UMOUNT], [umount], [], [$PATH:/sbin:/usr/sbin])
> +  AC_PATH_PROG([MKFS], [mkfs], [], [$PATH:/sbin:/usr/sbin])
>    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
> @@ -1659,6 +1663,8 @@ if test "$with_storage_fs" = "yes" || test "$with_storage_fs" = "check"; then
>          [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"])
> @@ -2304,6 +2310,26 @@ if test "$with_nwfilter" = "yes" ; then
>  fi
>  AM_CONDITIONAL([WITH_NWFILTER], [test "$with_nwfilter" = "yes"])
>  
> +dnl libblkid is used by several storage drivers; therefore we probe
> +dnl for it unconditionally.
> +AC_ARG_WITH([libblkid],
> +  [AS_HELP_STRING([--with-libblkid],
> +    [use libblkid to scan for filesystems and partitions @<:@default=check@:>@])],
> +  [],
> +  [with_libblkid=check])
> +
> +if test "x$with_libblkid" = "xyes" || test "x$with_libblkid" = "xcheck"; then
> +   PKG_CHECK_MODULES([BLKID],
> +     [blkid >= $LIBBLKID_REQUIRED],
> +     [with_libblkid="yes"],
> +     [with_libblkid="no"])
> +fi
> +
> +if test "x$with_libblkid" = "xyes"; then
> +  AC_DEFINE([HAVE_LIBBLKID], [1], [libblkid is present])
> +fi
> +AM_CONDITIONAL([HAVE_LIBBLKID], [test "x$with_libblkid" = "xyes"])
> +
>  AC_ARG_WITH([qemu-user],
>    AC_HELP_STRING([--with-qemu-user], [username to run QEMU system instance as @<:@default=root@:>@]),
>    [QEMU_USER=${withval}],
> diff --git a/libvirt.spec.in b/libvirt.spec.in
> index f4da969..a7bcfd7 100644
> --- a/libvirt.spec.in
> +++ b/libvirt.spec.in
> @@ -281,6 +281,10 @@ Requires: PolicyKit >= 0.6
>  %endif
>  %if %{with_storage_fs}
>  Requires: nfs-utils
> +# For mkfs
> +Requires: util-linux-ng
> +# For pool-build probing for existing pools
> +BuildRequires: libblkid-devel >= 2.17

  We may have to tweak this if it's not available for example on RHEL5
but okay for now

>  # For glusterfs
>  %if 0%{?fedora} >= 11
>  Requires: glusterfs-client >= 2.0.1

  ACK, but should be commited without the other 2 patches

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel@xxxxxxxxxxxx  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/

--
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]