All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: QEMU Developers <qemu-devel@nongnu.org>,
	Libvirt <libvir-list@redhat.com>,
	Gerd Hoffmann <kraxel@redhat.com>, Fam Zheng <famz@redhat.com>
Subject: Re: [Qemu-devel] [PATCH 3/3] ui: remove support for SDL1.2 in favour of SDL2
Date: Wed, 22 Aug 2018 10:53:30 +0100	[thread overview]
Message-ID: <20180822095330.GA29063@redhat.com> (raw)
In-Reply-To: <CAFEAcA8XNE5GOtfAi8+pDvYOdeNOU0eHT-xSSA8RK3F94PUv+A@mail.gmail.com>

On Wed, Aug 08, 2018 at 02:51:01PM +0100, Peter Maydell wrote:
> On 8 August 2018 at 11:49, Daniel P. Berrangé <berrange@redhat.com> wrote:
> > SDL1.2 was deprecated in the 2.12.0 release with:
> >
> >   commit e52c6ba34149b4f39c3fd60e59ee32b809db2bfa
> >   Author: Daniel P. Berrange <berrange@redhat.com>
> >   Date:   Mon Jan 15 14:25:33 2018 +0000
> >
> >     ui: deprecate use of SDL 1.2 in favour of 2.0 series
> >
> >     The SDL 2.0 release was made in Aug, 2013:
> >
> >       https://www.libsdl.org/release/
> >
> >     That will soon be 4 + 1/2 years ago, which is enough time to consider
> >     the 2.0 series widely supported.
> >
> >     Thus we deprecate the SDL 1.2 support, which will allow us to delete it
> >     in the last release of 2018. By this time, SDL 2.0 will be more than 5
> >     years old.
> >
> >     Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
> >     Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> >     Message-id: 20180115142533.24585-1-berrange@redhat.com
> >     Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> >
> > It is thus able to be removed in the 3.1.0 release.
> 
> At least one of the BSD VMs in tests/vm/ is still using SDL1.2.
> I think we should update that VM before we drop SDL1.2 support.

I confirmed that the freebsd & netbsd images still work as SDL just gets
automatically disabled by configure. The openbsd image breaks hard though
because SDL is the only available audio backend on openbsd, effectively
making SDL a mandatory requirement.

So we definitely need to update the openbsd image to have SDL2.


Regards,
Daniel
-- 
|: 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 :|

  parent reply	other threads:[~2018-08-22  9:53 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-08 10:49 [Qemu-devel] [PATCH 0/3] ui: remove deprecated UI frontends Daniel P. Berrangé
2018-08-08 10:49 ` [Qemu-devel] [PATCH 1/3] ui: remove support for GTK2 in favour of GTK3 Daniel P. Berrangé
2018-08-08 10:49 ` [Qemu-devel] [PATCH 2/3] ui: increase min required GTK3 version to 3.14.0 Daniel P. Berrangé
2018-08-08 10:49 ` [Qemu-devel] [PATCH 3/3] ui: remove support for SDL1.2 in favour of SDL2 Daniel P. Berrangé
2018-08-08 13:51   ` Peter Maydell
2018-08-08 13:55     ` Daniel P. Berrangé
2018-08-22  9:53     ` Daniel P. Berrangé [this message]
2018-08-08 11:24 ` [Qemu-devel] [PATCH 0/3] ui: remove deprecated UI frontends Gerd Hoffmann
2018-08-08 12:58   ` Daniel P. Berrangé
2018-08-21  7:46 ` Gerd Hoffmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180822095330.GA29063@redhat.com \
    --to=berrange@redhat.com \
    --cc=famz@redhat.com \
    --cc=kraxel@redhat.com \
    --cc=libvir-list@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.