Bug#758359: mplayer2: Increase MP_INPUT_BUFFER_PADDING_SIZE

2014-10-04 Thread Alessio Treglia
Hi,

and thanks for the report.

Have you tested the patch with libav too?

Cheers!

-- 
Alessio Treglia  | www.alessiotreglia.com
Debian Developer | ales...@debian.org
Ubuntu Core Developer|  quadris...@ubuntu.com
0416 0004 A827 6E40 BB98 90FB E8A4 8AE5 311D 765A
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers

Bug#758359: mplayer2: Increase MP_INPUT_BUFFER_PADDING_SIZE

2014-10-04 Thread Andreas Cadhalpun

Hi,

On 04.10.2014 17:53, Alessio Treglia wrote:

Have you tested the patch with libav too?


Yes, it still works with Libav, as well.

You might even want to increase the input buffer padding size to 64 (as 
mplayer does), just in case this is increased again in the future.


Best regards,
Andreas

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers


Bug#758359: mplayer2: Increase MP_INPUT_BUFFER_PADDING_SIZE

2014-08-16 Thread Andreas Cadhalpun

Source: mplayer2
Version: 2.0-728-g2c378c7-2
Severity: wishlist
Tags: patch
Usertags: reintroducing-ffmpeg

Dear maintainer,

I am working on reintroducing FFmpeg to Debian [1-2].

FFmpeg needs 32-Byte padding. The attached patch changes mplayer2's
MP_INPUT_BUFFER_PADDING_SIZE to that.
It might even make sense to change it directly to 64, as the padding
requirement might increase further.

Please apply this patch to facilitate building your package with FFmpeg
in Debian.

If you want to facilitate this even further, you can also add
lib*-ffmpeg-dev alternatives to the Libav build-dependencies.

While the FFmpeg package is still waiting in the NEW queue [3], it can
already be built from the git repository [4].

Best regards,
Andreas


1: https://lists.debian.org/debian-devel/2014/07/msg01010.html
2: https://bugs.debian.org/729203
3: https://ftp-master.debian.org/new/ffmpeg_7:2.3.1-1.html
4: https://anonscm.debian.org/cgit/collab-maint/ffmpeg.git

diff --git a/debian/patches/INPUT_BUFFER_PADDING.patch b/debian/patches/INPUT_BUFFER_PADDING.patch
new file mode 100644
index 000..28de569
--- /dev/null
+++ b/debian/patches/INPUT_BUFFER_PADDING.patch
@@ -0,0 +1,17 @@
+Description: Increase FFMPEG_INPUT_BUFFER_PADDING_SIZE to 32
+ This is needed to be able to build with FFmpeg 2.3.
+
+Author: Andreas Cadhalpun andreas.cadhal...@googlemail.com
+Last-Update: 2014-07-20
+
+--- mplayer2-2.0-728-g2c378c7.orig/libmpdemux/demuxer.h
 mplayer2-2.0-728-g2c378c7/libmpdemux/demuxer.h
+@@ -121,7 +121,7 @@ enum timestamp_type {
+ #define SEEK_BACKWARD (1  3)
+ 
+ // demux_lavf can pass lavf buffers using FF_INPUT_BUFFER_PADDING_SIZE instead
+-#define MP_INPUT_BUFFER_PADDING_SIZE 16
++#define MP_INPUT_BUFFER_PADDING_SIZE 32
+ 
+ typedef struct demux_stream {
+ int buffer_pos;// current buffer position
diff --git a/debian/patches/series b/debian/patches/series
index bb73a73..ea46fe4 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,2 +1,3 @@
 ftbfs-libav10.patch
 samba4.patch
+INPUT_BUFFER_PADDING.patch

___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers