[libvirt] [PATCH 1/3] storage: Add mkfs and libblkid to build system
Osier Yang
jyang at redhat.com
Fri Sep 2 13:20:08 UTC 2011
于 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
More information about the libvir-list
mailing list