[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-4.0 v3] configure: bump spice-server require
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH for-4.0 v3] configure: bump spice-server required version to 0.12.5 |
Date: |
Thu, 29 Nov 2018 12:23:04 +0100 |
User-agent: |
NeoMutt/20180716 |
> > seems the spice-protocol dep is there due to qemu itself needing it:
> >
> > commit 358689fe299c306f1d81bea57a5067d0abb56699
> > Author: Michal Privoznik <address@hidden>
> > Date: Fri Mar 1 08:43:18 2013 +0100
> >
> > configure: Require at least spice-protocol-0.12.3
> >
> > As of 5a49d3e9 we assume SPICE_PORT_EVENT_BREAK to be defined.
> > However, it is defined not in 0.12.2 what we require now, but in
> > 0.12.3. Therefore in order to prevent build failure we must
> > adjust our minimal requirements.
> >
> > Signed-off-by: Stefan Hajnoczi <address@hidden>
> >
> > That makes sense. So, when spice-server 0.12.5 requires spice-protocol
> > 0.12.8+ anyway I think we can savely drop the spice-protocol check.
> >
> > cheers,
> > Gerd
> >
>
> My idea was more to revert (kind of) 7e3efdac75caca0b283f8e76ad24c924b4718e7b:
>
>
> commit 7e3efdac75caca0b283f8e76ad24c924b4718e7b
> Author: Alon Levy <address@hidden>
> Date: Wed Mar 7 16:19:03 2012 +0200
>
> spice: require spice-protocol >= 0.8.1
>
> Requiring spice-server >= 0.8.2 is not enough since spice-server.pc
> doesn't require spice-protocol (any version). Until that is fixed
> upstream an explicit requirement in qemu fixes compilation broken since
>
> commit 2e1a98c9c1b90ca093278c6b43244dc46604d7b7
> Author: Alon Levy <address@hidden>
> Date: Fri Feb 24 23:19:30 2012 +0200
>
> qxl: introduce QXLCookie
>
> Reported-by: Peter Maydell <address@hidden>
>
> Signed-off-by: Alon Levy <address@hidden>
> Signed-off-by: Gerd Hoffmann <address@hidden>
>
> diff --git a/configure b/configure
> index 0111774cb0..62aa7609e1 100755
> --- a/configure
> +++ b/configure
> @@ -2592,6 +2592,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 && \
> compile_prog "$spice_cflags" "$spice_libs" ; then
> spice="yes"
> libs_softmmu="$libs_softmmu $spice_libs"
>
>
> The rationale does not apply any more, spice-server depends on spice-protocol.
Fine with me too.
cheers,
Gerd