[libvirt] [libvirt-glib] libvirt dependency needs to be bumped
Hey guys, It seems libvirt-glib requires VIR_DOMAIN_EVENT_SHUTDOWN which was only added in libvirt-0.9.8[1]. Hence, configure.ac needs LIBVIRT_REQUIRED=0.9.8 otherwise the build fails in make instead of in configure. Thanks! 1. 38527c9ae0d398a996e1e3c0897a104e3ffd1256 in libvirt.git -- ~Nirbheek Chauhan Gentoo GNOME+Mozilla Team -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
[libvirt] [libvirt-glib PATCH] Add vapi dependency to Makefile.am and fix parallel build
From: Nirbheek Chauhan libvirt-glib-1.0.vapi is needed to generate libvirt-gobject-1.0.vapi Without an explicit dependency, make -jN fails Signed-off-by: Nirbheek Chauhan --- vapi/Makefile.am |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/vapi/Makefile.am b/vapi/Makefile.am index 0af8f6f..2ecc3e3 100644 --- a/vapi/Makefile.am +++ b/vapi/Makefile.am @@ -17,7 +17,7 @@ libvirt-glib-1.0.vapi: $(top_builddir)/libvirt-glib/LibvirtGLib-1.0.gir --library libvirt-glib-1.0 \ $< -libvirt-gobject-1.0.vapi: $(top_builddir)/libvirt-gobject/LibvirtGObject-1.0.gir +libvirt-gobject-1.0.vapi: $(top_builddir)/libvirt-gobject/LibvirtGObject-1.0.gir libvirt-glib-1.0.vapi $(AM_V_GEN)$(VAPIGEN) \ --vapidir=$(builddir) \ --pkg gobject-2.0 \ -- 1.7.3.4 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] [configure] Fix libvirt dependency, and use $VAPIGEN
On Mon, Nov 7, 2011 at 10:10 PM, Eric Blake wrote: > To make it easier for yourself, add this to libvirt-glib/.git/config: > > [format] > subjectprefix = libvirt-glib PATCH > > (or similarly for any other libvrit-SUB project). Then your patches will > automatically be prefixed with designation that it is for a subproject and > not for the top-level libvirt.git, which will help in sorting threads as > long as all the subprojects still share the primary libvir-list mailing > list. > Thank you for the tip! I've added this now. :) -- ~Nirbheek Chauhan Gentoo GNOME+Mozilla Team -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] [configure] Fix libvirt dependency, and use $VAPIGEN
On Mon, Nov 7, 2011 at 3:33 PM, Michal Privoznik wrote: > On 07.11.2011 10:25, nirbh...@gentoo.org wrote: >> This is a rather trivial patch to raise the minimum libvirt dependency; at >> least >> 0.9.4 is required. In addition, it fixes the direct usage of `vapigen`, and >> uses >> the result of the AC_PATH_PROG(VAPIGEN ...) call earlier so that the binary >> to >> be used can be overridden as that macro was meant to allow. >> > Just for the record, this is for libvirt-glib > Ah yes, I forgot to put that in the subject, didn't I? Sorry about that. :-) -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
[libvirt] [PATCH] [configure] Fix libvirt dependency, and use $VAPIGEN
From: Nirbheek Chauhan libvirt-0.9.4 is needed for at least VIR_DOMAIN_EVENT_SUSPENDED_RESTORED, etc from c1ff5dc63dc9d9ec05485bb695cbc66850759ee5 . $VAPIGEN from AC_PATH_PROG should be used instead of the raw binary `vapigen` otherwise it's not possible to override the vapigen binary to be used. --- configure.ac |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index 0ac7a8f..4f87b37 100644 --- a/configure.ac +++ b/configure.ac @@ -10,7 +10,7 @@ AC_CANONICAL_HOST AM_SILENT_RULES([yes]) -LIBVIRT_REQUIRED=0.5.0 +LIBVIRT_REQUIRED=0.9.4 GLIB2_REQUIRED=2.10.0 GOBJECT2_REQUIRED=2.10.0 GIO_REQUIRED=2.10.0 @@ -217,7 +217,7 @@ if test "x$enable_introspection" = "xyes" ; then fi if test "x$enable_vala" != "xno" ; then AC_MSG_CHECKING([$VAPIGEN support for --metadatadir]) -vapigen --help | grep metadatadir 1>/dev/null 2>&1 +$VAPIGEN --help | grep metadatadir 1>/dev/null 2>&1 if test $? != 0 ; then if test "x$enable_vala" == "xcheck" ; then enable_vala=no -- 1.7.3.4 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
[libvirt] [PATCH] [configure] Fix libvirt dependency, and use $VAPIGEN
This is a rather trivial patch to raise the minimum libvirt dependency; at least 0.9.4 is required. In addition, it fixes the direct usage of `vapigen`, and uses the result of the AC_PATH_PROG(VAPIGEN ...) call earlier so that the binary to be used can be overridden as that macro was meant to allow. Thanks! configure.ac |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
