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

[Roger Pau Monne <roger.pau@citrix.com>]

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.

 From my understanding it causes Qemu build to fail, since our version 
of Qemu configure script doesn't check for glib version.

The following commit should be backported to our Qemu tree also 
a52d28afb4e825a5b28815370a268904a4c6dc11.

>> Signed-off-by: Olaf Hering<olaf@aepfle.de>

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@citrix.com>

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@lists.xen.org
>> http://lists.xen.org/xen-devel
>
>


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel