Re: [PATCH 6/6] ui/spice: support multi plane dmabuf scanout

["Daniel P. Berrangé" <berrange@redhat.com>]

On Mon, Mar 24, 2025 at 01:30:16PM +0400, Marc-André Lureau wrote:
> Hi
> 
> On Mon, Mar 24, 2025 at 12:20 PM <yuq825@gmail.com> wrote:
> >
> > From: Qiang Yu <yuq825@gmail.com>

Please provide non-empty commit messages.

> >
> > Signed-off-by: Qiang Yu <yuq825@gmail.com>
> > ---
> >  meson.build        |  2 +-
> >  ui/spice-display.c | 65 +++++++++++++++++++++++-----------------------
> >  2 files changed, 34 insertions(+), 33 deletions(-)
> >
> > diff --git a/meson.build b/meson.build
> > index 9d9c11731f..b87704a83b 100644
> > --- a/meson.build
> > +++ b/meson.build
> > @@ -1329,7 +1329,7 @@ if get_option('spice') \
> >               .require(pixman.found(),
> >                        error_message: 'cannot enable SPICE if pixman is not available') \
> >               .allowed()
> > -  spice = dependency('spice-server', version: '>=0.14.0',
> > +  spice = dependency('spice-server', version: '>=0.14.3',
> 
> I am okay with bumping dependency requirements, but the nicer
> alternative would be to allow the current version and check the
> existence of the new API/function instead.

Bumping the min version must be a separate commit that lists
the current versions in our supported build platforms[1], in order
to demonstrate the that the chosen min version doesn't exclude
any supported platform.

See commit 34d55725e664445ccd5621165b1ef805197a530e for how
this should be done.

0.14.3 looks like it would probably be fine as a choice though,
as Ubuntu 20.04 has aged out of our support matrix.

With regards,
Daniel

[1] https://www.qemu.org/docs/master/about/build-platforms.html
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|