Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Josh de Kock
On 14/09/2016 20:07, James Almer wrote: [...] I'm fine with that if others are (Especially Marton and Josh, since they develop/maintain ffplay). It's a good compromise. But much like ffserver it should similarly be marked as deprecated and essentially unmaintained somewhere. I'm fine with

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Michael Niedermayer
On Wed, Sep 14, 2016 at 04:07:50PM -0300, James Almer wrote: > On 9/14/2016 3:51 PM, Michael Niedermayer wrote: > > On Wed, Sep 14, 2016 at 06:00:00PM +0200, Carl Eugen Hoyos wrote: > >> 2016-09-14 17:54 GMT+02:00 Josh de Kock : > >>> We discussed this in IRC and thought that

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread James Almer
On 9/14/2016 3:51 PM, Michael Niedermayer wrote: > On Wed, Sep 14, 2016 at 06:00:00PM +0200, Carl Eugen Hoyos wrote: >> 2016-09-14 17:54 GMT+02:00 Josh de Kock : >>> We discussed this in IRC and thought that ffmpeg was a >>> big enough library, and it switching to SDL2 may help

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Michael Niedermayer
On Wed, Sep 14, 2016 at 06:00:00PM +0200, Carl Eugen Hoyos wrote: > 2016-09-14 17:54 GMT+02:00 Josh de Kock : > > We discussed this in IRC and thought that ffmpeg was a > > big enough library, and it switching to SDL2 may help > > distros to consider supporting SDL2 > > If this

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 12:18 PM, Carl Eugen Hoyos wrote: > 2016-09-14 18:15 GMT+02:00 James Almer : > > > Configure failed to detect it > > configure requires 2.0.1 > > Carl Eugen (who wonders how Ronald tests...) ffmpeg (the tool), of course! :)

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Marton Balint
On Wed, 14 Sep 2016, Carl Eugen Hoyos wrote: 2016-09-14 18:15 GMT+02:00 James Almer : Configure failed to detect it configure requires 2.0.1 YUV texture updates require 2.0.1 as far as I remember. Regards, Marton ___

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread James Almer
On 9/14/2016 1:12 PM, Clément Bœsch wrote: > On Wed, Sep 14, 2016 at 05:34:05PM +0200, Carl Eugen Hoyos wrote: >> 2016-09-12 20:58 GMT+02:00 Josh de Kock : >>> From: Marton Balint >>> >>> Depends on the other SDL patches. >> >> This works fine on my antique

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Hendrik Leppkes
Am 14.09.2016 18:00 schrieb "Carl Eugen Hoyos" : > > 2016-09-14 17:54 GMT+02:00 Josh de Kock : > > We discussed this in IRC and thought that ffmpeg was a > > big enough library, and it switching to SDL2 may help > > distros to consider supporting SDL2 > > If

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 18:15 GMT+02:00 James Almer : > Configure failed to detect it configure requires 2.0.1 Carl Eugen (who wonders how Ronald tests...) ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread James Almer
On 9/14/2016 1:00 PM, Carl Eugen Hoyos wrote: > 2016-09-14 17:54 GMT+02:00 Josh de Kock : >> We discussed this in IRC and thought that ffmpeg was a >> big enough library, and it switching to SDL2 may help >> distros to consider supporting SDL2 > > If this really were the only

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Clément Bœsch
On Wed, Sep 14, 2016 at 05:34:05PM +0200, Carl Eugen Hoyos wrote: > 2016-09-12 20:58 GMT+02:00 Josh de Kock : > > From: Marton Balint > > > > Depends on the other SDL patches. > > This works fine on my antique desktop system (where an > extra repository provides

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Ronald S. Bultje
Hi, On Wed, Sep 14, 2016 at 12:04 PM, Carl Eugen Hoyos wrote: > 2016-09-14 17:59 GMT+02:00 Michael Niedermayer : > > On Wed, Sep 14, 2016 at 05:34:05PM +0200, Carl Eugen Hoyos wrote: > >> 2016-09-12 20:58 GMT+02:00 Josh de Kock : >

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread James Almer
On 9/14/2016 12:34 PM, Carl Eugen Hoyos wrote: > 2016-09-12 20:58 GMT+02:00 Josh de Kock : >> From: Marton Balint >> >> Depends on the other SDL patches. > > This works fine on my antique desktop system (where an > extra repository provides sdl2 2.0.3) but not

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 17:59 GMT+02:00 Michael Niedermayer : > On Wed, Sep 14, 2016 at 05:34:05PM +0200, Carl Eugen Hoyos wrote: >> 2016-09-12 20:58 GMT+02:00 Josh de Kock : >> > From: Marton Balint >> > >> > Depends on the other SDL patches. >> >>

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Michael Niedermayer
On Wed, Sep 14, 2016 at 05:34:05PM +0200, Carl Eugen Hoyos wrote: > 2016-09-12 20:58 GMT+02:00 Josh de Kock : > > From: Marton Balint > > > > Depends on the other SDL patches. > > This works fine on my antique desktop system (where an > extra repository provides

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Carl Eugen Hoyos
2016-09-14 17:54 GMT+02:00 Josh de Kock : > We discussed this in IRC and thought that ffmpeg was a > big enough library, and it switching to SDL2 may help > distros to consider supporting SDL2 If this really were the only reason, I sincerely hope we wouldn't even consider this

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Josh de Kock
On 14/09/2016 16:34, Carl Eugen Hoyos wrote: [...] If a supported Linux distribution does not ship a sufficiently recent sdl2, I am not sure if this is such a good idea. Does the patch offer new functionality? It doesn't add any extra functionality, but SDL1 has been unmaintained for more than

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Carl Eugen Hoyos
2016-09-12 20:58 GMT+02:00 Josh de Kock : > From: Marton Balint > > Depends on the other SDL patches. This works fine on my antique desktop system (where an extra repository provides sdl2 2.0.3) but not on my laptop that I did not consider eol so far, it only

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Josh de Kock
On Wed, Sep 14, 2016, at 10:12 AM, Michael Niedermayer wrote: > On Tue, Sep 13, 2016 at 01:45:12PM -0300, James Almer wrote: > > > > Tested on Windows (compiled with mingw-w64) and works fine. > > patchwork uses "^Tested-by:" to detect patches having been tested > should the regexp be

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-14 Thread Michael Niedermayer
On Tue, Sep 13, 2016 at 01:45:12PM -0300, James Almer wrote: > On 9/12/2016 3:58 PM, Josh de Kock wrote: > > From: Marton Balint > > > > Depends on the other SDL patches. > > > > Signed-off-by: Josh de Kock > > --- > > configure | 6 +- > > ffplay.c | 594

Re: [FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-13 Thread James Almer
On 9/12/2016 3:58 PM, Josh de Kock wrote: > From: Marton Balint > > Depends on the other SDL patches. > > Signed-off-by: Josh de Kock > --- > configure | 6 +- > ffplay.c | 594 > ++ > 2 files

[FFmpeg-devel] [PATCH] ffplay: convert ffplay to use SDL2

2016-09-12 Thread Josh de Kock
From: Marton Balint Depends on the other SDL patches. Signed-off-by: Josh de Kock --- configure | 6 +- ffplay.c | 594 ++ 2 files changed, 249 insertions(+), 351 deletions(-) diff --git