Re: [PATCH 1 of 3] tools/configure.ac: add version check for glib2

[Olaf Hering <olaf@aepfle.de>]

On Wed, Jun 13, Ian Campbell wrote:

> On Wed, 2012-06-13 at 09:01 +0100, Olaf Hering wrote:
> > # HG changeset patch
> > # User Olaf Hering <olaf@aepfle.de>
> > # 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[3]: *** [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.

I will resend with this change:

Add a version check to toplevel configure to require at least glib 2.12.
This makes sure configure can detect the condition early instead of
failing later in the middle of tools build when qemu-upstream errors
out.


Olaf