Re: [Qemu-devel] [PATCH] configure: require glib 2.12, 2.20 for mingw32

[Blue Swirl]

On Thu, Apr 5, 2012 at 11:01, Paolo Bonzini <address@hidden> wrote:
> These are pretty sane requirements to move forward with glib usage.
> 2.12 is the version found in RHEL/CentOS 5, and 2.20 is the
> first version to support g_poll.  Without g_poll, we cannot
> integrate well with the glib main loop.

It looks like 2.20 was released in 2009, so for win32 this is reasonable too.

Thanks, applied.

>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
>  configure |   12 ++++++++++--
>  1 file changed, 10 insertions(+), 2 deletions(-)
>
> diff --git a/configure b/configure
> index 4b3adc9..6f850a6 100755
> --- a/configure
> +++ b/configure
> @@ -1993,13 +1993,21 @@ fi
>
>  ##########################################
>  # glib support probe
> -if $pkg_config --modversion gthread-2.0 > /dev/null 2>&1 ; then
> +
> +if test "$mingw32" = yes; then
> +    # g_poll is required in order to integrate with the glib main loop.
> +    glib_req_ver=2.20
> +else
> +    glib_req_ver=2.12
> +fi
> +if $pkg_config --atleast-version=$glib_req_ver gthread-2.0 > /dev/null 2>&1
> +then
>     glib_cflags=`$pkg_config --cflags gthread-2.0 2>/dev/null`
>     glib_libs=`$pkg_config --libs gthread-2.0 2>/dev/null`
>     LIBS="$glib_libs $LIBS"
>     libs_qga="$glib_libs $libs_qga"
>  else
> -    echo "glib-2.0 required to compile QEMU"
> +    echo "glib-$glib_req_ver required to compile QEMU"
>     exit 1
>  fi
>
> --
> 1.7.9.3
>