[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH v3] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC |
Date: |
Thu, 09 Aug 2012 15:50:12 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.6esrpre) Gecko/20120714 Thunderbird/10.0.6 |
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
- Re: [Qemu-devel] [PATCH v3] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC, Gerd Hoffmann, 2012/08/06
- Re: [Qemu-devel] [PATCH v3] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC, Alon Levy, 2012/08/09
- Re: [Qemu-devel] [PATCH v3] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC,
Gerd Hoffmann <=
- Re: [Qemu-devel] [PATCH v3] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC, Alon Levy, 2012/08/09
- [Qemu-devel] [PATCH v4 1/2] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC, Alon Levy, 2012/08/09
- [Qemu-devel] [PATCH v4 2/2] configure: print spice-protocol and spice-server versions, Alon Levy, 2012/08/09
- Re: [Qemu-devel] [PATCH v4 1/2] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC, Gerd Hoffmann, 2012/08/15
- Re: [Qemu-devel] [PATCH v4 1/2] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC, Alon Levy, 2012/08/15
- Re: [Qemu-devel] [PATCH v4 1/2] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC, Gerd Hoffmann, 2012/08/15