[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] [PATCH 1 of 3] tools/configure.ac: add version check for glib2
Ian Campbell wrote:
On Wed, 2012-06-13 at 09:56 +0100, Roger Pau Monne wrote:Ian Campbell wrote:On Wed, 2012-06-13 at 09:01 +0100, Olaf Hering wrote:# HG changeset patch # User Olaf Hering<olaf@xxxxxxxxx> # Date 1339572293 -7200 # Node ID ea554d05821b95a7e96e4a25cbf953c5abe35aeb # Parent a70b35deb2b5592cc1b2363860f21bb2c7049885 tools/configure.ac: add version check for glib2 xen-unstable fails to build in a SLES10SP4 environment since a long time because the included version of glib is slightly older than the required glib version. According to the docs glib version 2.12 includes base64 support, but SLES10 is shipped with glib 2.8.6: qemu-timer-common.o: In function `init_get_clock': /usr/src/packages/BUILD/xen-4.2.25432/non-dbg/tools/qemu-xen-dir/qemu-timer-common.c:57: undefined reference to `clock_gettime' qga/guest-agent-commands.o: In function `qmp_guest_file_write': qga/guest-agent-commands.c:249: undefined reference to `g_base64_decode' qga/guest-agent-commands.o: In function `qmp_guest_file_read': qga/guest-agent-commands.c:224: undefined reference to `g_base64_encode' collect2: ld returned 1 exit status make: *** [qemu-ga] Error 1 Add a version check to configure to require at least glib 2.12 to build qemu-upstream.Does this cause configure to fail or does it cause us to just not build qemu-upstream? I think the former (which is fine with me) but your last sentence suggests that latter.From my understanding it causes Qemu build to fail, since our version of Qemu configure script doesn't check for glib version.But this patch makes it do that check, right?
Yes, we currently check for glib, but we don't require any specific version. This patch sets the necessary glib version for Qemu-upstream compilation to succeed as a requirement for our configure script.
The following commit should be backported to our Qemu tree also a52d28afb4e825a5b28815370a268904a4c6dc11.Signed-off-by: Olaf Hering<olaf@xxxxxxxxx>Anyway, since we check for glib already, I think this should be applied, so at least we check for the required version Acked-by: Roger Pau MonnÃ<roger.pau@xxxxxxxxxx> Please rerun autoconf after applying this.diff -r a70b35deb2b5 -r ea554d05821b tools/configure.ac --- a/tools/configure.ac +++ b/tools/configure.ac @@ -115,7 +115,7 @@ AX_PATH_PROG_OR_FAIL([BCC], [bcc]) AX_PATH_PROG_OR_FAIL([IASL], [iasl]) AX_CHECK_UUID AX_CHECK_CURSES -PKG_CHECK_MODULES(glib, glib-2.0) +PKG_CHECK_MODULES(glib, [glib-2.0>= 2.12]) # Check library path AX_DEFAULT_LIB _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
_______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
Lists.xenproject.org is hosted with RackSpace, monitoring our