Re: [Qemu-devel] [PATCH v3] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC

[Gerd Hoffmann <kraxel@redhat.com>]

On 08/09/12 15:41, Alon Levy wrote:
> On Mon, Aug 06, 2012 at 09:31:19AM +0200, Gerd Hoffmann wrote:
>>> diff --git a/configure b/configure
>>> index cef0a71..5fcd315 100755
>>> --- a/configure
>>> +++ b/configure
>>> @@ -2630,7 +2630,7 @@ EOF
>>>    spice_cflags=$($pkg_config --cflags spice-protocol spice-server 2>/dev/null)
>>>    spice_libs=$($pkg_config --libs spice-protocol spice-server 2>/dev/null)
>>>    if $pkg_config --atleast-version=0.8.2 spice-server >/dev/null 2>&1 && \
>>> -     $pkg_config --atleast-version=0.8.1 spice-protocol > /dev/null 2>&1 && \
>>> +     $pkg_config --atleast-version=0.12.0 spice-protocol > /dev/null 2>&1 && \
>>
>> I'd prefer to not require cutting-edge spice bits to build ...
> 
> I only increased the protocol requirement, not the server.

Distros tend to update those in sync, so this doesn't help much for the
average user, this change would likely cause spice detection fail on
almost every released distro ...

> Meanwhile I'll add a define based on the pkg-config spice-protocol
> version, which I can test from hw/qxl.c, since the commit adding
> QXL_IO_MONITORS_CONFIG_ASYNC didn't introduce any new define I can
> check.

Just checking "SPICE_SERVER_VERSION >= 0x000b01" doesn't work?  I'd
expect spice-server 0.11.1+ having a dependency on a recent enougth
spice-protocol so you can expect QXL_IO_MONITORS_CONFIG_ASYNC being
present then, no?

thanks,
  Gerd