[libvirt] [libvirt-glib] Make vala optional by default
Daniel P. Berrange
berrange at redhat.com
Fri Oct 21 09:49:51 UTC 2011
On Fri, Oct 21, 2011 at 10:59:12AM +0200, Christophe Fergeau wrote:
> 04368c60c added vala binding generation to libvirt-glib. To do
> that, it needs vapigen, so it checks for its presence during
> configure.
> However, configure errors out when vapigen is not found
> unless --disable-vala has been explicitly passed. In particular,
> vapigen must be available when running autogen.sh/configure without
> arguments.
>
> This commit changes this behaviour to disable building of vala
> bindings when vapigen can't be found and no arguments were
> passed to configure. It will still error out when --enable-vala was
> specified and vapigen is not available.
> ---
> configure.ac | 11 ++++++-----
> 1 files changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index c474f0b..b3fa808 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -205,12 +205,13 @@ if test "x$enable_introspection" = "xyes" ; then
> if test "x$enable_vala" != "xno" ; then
> AC_PATH_PROG(VAPIGEN, vapigen, no)
> if test "x$VAPIGEN" != "xno"; then
> - if test "x$enable_vala" == "xcheck" ; then
> - enable_vala=yes
> - AC_SUBST(VAPIGEN)
> - fi
> + enable_vala=yes
> else
> - AC_MSG_ERROR([Cannot find the "vapigen" binary in your PATH])
> + if test "x$enable_vala" != "xcheck" ; then
> + AC_MSG_ERROR([Cannot find the "vapigen" binary in your PATH])
> + else
> + enable_vala=no
> + fi
> fi
> fi
> fi
ACK
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
More information about the libvir-list
mailing list