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

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

 



于 2011年09月02日 19:34, Daniel Veillard 写道:
On Fri, Sep 02, 2011 at 07:19:46PM +0800, Osier Yang wrote:
于 2011年09月02日 19:11, Daniel Veillard 写道:
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
Aren't the other 2 patches also ACKed? :-)
   Now yes but at the time I didn't know :-)

Daniel

I pushed these patchset as-is, will add test case and improve
spec file in a following patch later.

Thanks for the reviewing.

Osier

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