[Libguestfs] [PATCH 2/2] configure: simplify check for oUnit v2
Richard W.M. Jones
rjones at redhat.com
Tue May 12 18:55:09 UTC 2015
On Tue, May 12, 2015 at 03:20:57PM +0200, Pino Toscano wrote:
> Instead of parsing the version, try to look for the OUnit2 module within
> the oUnit package.
>
> Followup of commit 46bc79109bb51cfb656863da3635f1c8da5f6412.
> ---
> configure.ac | 15 ++++-----------
> 1 file changed, 4 insertions(+), 11 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 5ff151b..7a609cb 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1135,6 +1135,7 @@ AS_IF([test "x$OCAMLC" != "xno"],[
> OCAML_PKG_gettext=no
> OCAML_PKG_libvirt=no
> OCAML_PKG_oUnit=no
> +ounit_is_v2=no
> AS_IF([test "x$OCAMLC" != "xno"],[
> # Create mllib/common_gettext.ml, gettext functions or stubs.
>
> @@ -1147,17 +1148,9 @@ AS_IF([test "x$OCAMLC" != "xno"],[
> AC_CHECK_OCAML_PKG(libvirt)
> AC_CHECK_OCAML_PKG(oUnit)
>
> - # oUnit >= 2 is required. If it's not this version, discard.
> + # oUnit >= 2 is required, so check that it has OUnit2.
> if test "x$OCAML_PKG_oUnit" != "xno"; then
> - AC_MSG_CHECKING([for oUnit version 2 or above])
> - ounit_version=`$OCAMLFIND query oUnit -format '%v'`
> - ounit_version_major=`echo $ounit_version | $AWK -F. '{print $1}'`
> - if test $ounit_version_major -ge 2; then
> - AC_MSG_RESULT([yes])
> - else
> - AC_MSG_RESULT([no])
> - OCAML_PKG_oUnit=no
> - fi
> + AC_CHECK_OCAML_MODULE(ounit_is_v2,[OUnit.OUnit2],OUnit2,[+oUnit])
> fi
> ])
> AM_CONDITIONAL([HAVE_OCAML_PKG_GETTEXT],
> @@ -1165,7 +1158,7 @@ AM_CONDITIONAL([HAVE_OCAML_PKG_GETTEXT],
> AM_CONDITIONAL([HAVE_OCAML_PKG_LIBVIRT],
> [test "x$OCAMLC" != "xno" && test "x$OCAMLFIND" != "xno" && test "x$OCAML_PKG_libvirt" != "xno"])
> AM_CONDITIONAL([HAVE_OCAML_PKG_OUNIT],
> - [test "x$OCAMLC" != "xno" && test "x$OCAMLFIND" != "xno" && test "x$OCAML_PKG_oUnit" != "xno"])
> + [test "x$OCAMLC" != "xno" && test "x$OCAMLFIND" != "xno" && test "x$OCAML_PKG_oUnit" != "xno" && test "x$ounit_is_v2" != "xno"])
>
> AC_CHECK_PROG([OCAML_GETTEXT],[ocaml-gettext],[ocaml-gettext],[no])
> AM_CONDITIONAL([HAVE_OCAML_GETTEXT],
This one is OK, ACK.
Rich.
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
libguestfs lets you edit virtual machines. Supports shell scripting,
bindings from many languages. http://libguestfs.org
More information about the Libguestfs
mailing list