April 2015 - MPlayer-dev-eng - lists.mplayerhq.hu

Homepage skin download issue
by Ingo Brückl 09 Sep '15

09 Sep '15

Yesterday, I updated hwswskin to 1.3, but it still isn't downloadable. Is the generation of the skin download tarballs working? Ingo

3 8

mpcommon.c + bug +review
by Hainan Ren (任海男) 09 Jul '15

09 Jul '15

hi, when i get some src codes ,i want to figure how mplayer decode the mp3 format audio and play it, then i find a bug,which i sincerely think so. mpcommon.c calc_a_pts line 551 a_pts = sh_audio->ptr; the there is obviously not a ptr member in sh_audio ,which i think maybe it comes from d_audio thanks,again!

3 2

[PATCH] configure: remove support for internal libdvd*.
by Reimar Döffinger 04 Jun '15

04 Jun '15

--- configure | 134 +++----------------------------------------------------------- 1 file changed, 6 insertions(+), 128 deletions(-) diff --git a/configure b/configure index ba31428..b27b800 100755 --- a/configure +++ b/configure @@ -359,8 +359,6 @@ Optional features: --disable-bluray disable Blu-ray support [autodetect] --disable-dvdnav disable libdvdnav [autodetect] --disable-dvdread disable libdvdread [autodetect] - --disable-dvdread-internal disable internal libdvdread and libdvdnav [autodetect] - --disable-libdvdcss-internal disable internal libdvdcss [autodetect] --disable-cdparanoia disable cdparanoia [autodetect] --disable-cddb disable cddb [autodetect] --disable-bitmap-font disable bitmap font support [enable] @@ -750,8 +748,6 @@ type dvdnav-config >/dev/null 2>&1 && _dvdnavconfig=dvdnav-config _dvdreadconfig="pkg-config dvdread" type dvdread-config >/dev/null 2>&1 && _dvdnavconfig=dvdread-config _dvdread=auto -_dvdread_internal=auto -_libdvdcss_internal=auto _xanim=auto _real=auto _live=auto @@ -921,11 +917,9 @@ for ac_option do ;; --with-dvdnav-config=*) _dvdnavconfig=$(option_value $ac_option) - _dvdread_internal=no ;; --with-dvdread-config=*) _dvdreadconfig=$(option_value $ac_option) - _dvdread_internal=no ;; --extra-cflags=*) @@ -1170,10 +1164,6 @@ for ac_option do --disable-bluray) _bluray=no ;; --enable-dvdread) _dvdread=yes ;; --disable-dvdread) _dvdread=no ;; - --enable-dvdread-internal) _dvdread_internal=yes ;; - --disable-dvdread-internal) _dvdread_internal=no ;; - --enable-libdvdcss-internal) _libdvdcss_internal=yes ;; - --disable-libdvdcss-internal) _libdvdcss_internal=no ;; --enable-dvdnav) _dvdnav=yes ;; --disable-dvdnav) _dvdnav=no ;; --enable-xanim) _xanim=yes ;; @@ -6086,15 +6076,6 @@ fi echores "$_bluray" echocheck "dvdread" -if test "$_dvdread_internal" = auto ; then - _dvdread_internal=no - if linux || freebsd || netbsd || openbsd || dragonfly || sunos || hpux || - darwin || win32 || os2; then - _dvdread_internal=yes - _dvdread=yes - extra_cflags="-Ilibdvdread4 $extra_cflags" - fi -fi if test "$_dvdread" = auto ; then _dvdread=no if test "$_dl" = yes; then @@ -6104,20 +6085,14 @@ if test "$_dvdread" = auto ; then _dvdread=yes extra_cflags="$extra_cflags $_dvdreadcflags" extra_ldflags="$extra_ldflags $_dvdreadlibs" - res_comment="external" fi fi fi -if test "$_dvdread_internal" = yes; then - def_dvdread='#define CONFIG_DVDREAD 1' - inputmodules="dvdread(internal) $inputmodules" - res_comment="internal" -elif test "$_dvdread" = yes; then +if test "$_dvdread" = yes; then def_dvdread='#define CONFIG_DVDREAD 1' - extra_ldflags="$extra_ldflags -ldvdread" - inputmodules="dvdread(external) $inputmodules" - res_comment="external" + extra_ldflags="$extra_ldflags" + inputmodules="dvdread $inputmodules" else def_dvdread='#undef CONFIG_DVDREAD' noinputmodules="dvdread $noinputmodules" @@ -6125,83 +6100,6 @@ fi echores "$_dvdread" -echocheck "internal libdvdcss" -def_broken_mkdir='#undef HAVE_BROKEN_MKDIR' -def_dvdio='#undef DVD_STRUCT_IN_SYS_DVDIO_H' -def_dvd='#undef DVD_STRUCT_IN_DVD_H' -def_cdio='#undef DVD_STRUCT_IN_SYS_CDIO_H' -def_cdrom='#undef DVD_STRUCT_IN_LINUX_CDROM_H' -def_hpux_scsi_h='#undef HPUX_SCTL_IO' -def_sol_scsi_h='#undef SOLARIS_USCSI' -def_sys_uio_h='#undef HAVE_SYS_UIO_H' -_dvdio=no -_dvd=no -_cdio=no -_cdrom=no -_sol_scsi_h=no -_hpux_scsi_h=no -if test "$_libdvdcss_internal" = auto && test "$_dvdread_internal" = yes ; then - _libdvdcss_internal=no - test -d libdvdcss && _libdvdcss_internal=yes - - if header_check linux/cdrom.h ; then - _cdrom=yes - def_cdrom='#define DVD_STRUCT_IN_LINUX_CDROM_H 1' - # FreeBSD 8.1 has broken dvdio.h - elif header_check_broken sys/types.h sys/dvdio.h ; then - _dvdio=yes - def_dvdio='#define DVD_STRUCT_IN_SYS_DVDIO_H 1' - elif header_check dvd.h ; then - _dvd=yes - def_dvd='#define DVD_STRUCT_IN_DVD_H 1' - # at least OpenSolaris has a broken cdio.h - elif header_check_broken sys/types.h sys/cdio.h ; then - _cdio=yes - def_cdio='#define DVD_STRUCT_IN_SYS_CDIO_H 1' - fi - - if sunos; then - header_check sys/scsi/scsi_types.h && - header_check_broken sys/types.h sys/scsi/impl/uscsi.h && - _sol_scsi_h=yes && def_sol_scsi_h='#define SOLARIS_USCSI 1' - elif hpux; then - # also used by AIX, but AIX does not support VCD and/or libdvdread - header_check sys/scsi.h && _hpux_scsi_h=yes && - def_hpux_scsi_h='#define HPUX_SCTL_IO 1' || - _libdvdcss_internal=no - fi - - statement_check_broken stddef.h sys/stat.h 'mkdir(NULL, 0)' || - def_broken_mkdir='#define HAVE_BROKEN_MKDIR 1' - - header_check "sys/uio.h" && - def_sys_uio_h='#define HAVE_SYS_UIO_H 1' - -fi -if test "$_libdvdcss_internal" = yes ; then - if linux || netbsd || openbsd || wine ; then - def_dvd_linux='#define HAVE_LINUX_DVD_STRUCT 1' - openbsd && def_dvd_openbsd='#define HAVE_OPENBSD_DVD_STRUCT 1' - elif freebsd || dragonfly ; then - def_dvd_bsd='#define HAVE_BSD_DVD_STRUCT 1' - elif darwin ; then - def_dvd_darwin='#define DARWIN_DVD_IOCTL' - extra_ldflags="$extra_ldflags -framework IOKit -framework Carbon" - elif cygwin ; then - cflags_libdvdcss="-D_WIN32_IE=0x0500 -mwin32" - elif mingw32 ; then - cflags_libdvdcss="-D_WIN32_IE=0x0500" - libs_mplayer="$libs_mplayer -lshfolder" - fi - cflags_libdvdcss_dvdread="-Ilibdvdcss" - def_dvdcss="#define HAVE_DVDCSS_DVDCSS_H 1" - inputmodules="libdvdcss(internal) $inputmodules" -else - noinputmodules="libdvdcss(internal) $noinputmodules" -fi -echores "$_libdvdcss_internal" - - echocheck "libcdio" if test "$_libcdio" = auto ; then cat > $TMPC << EOF @@ -8123,22 +8021,7 @@ CFLAGS="$CFLAGS -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE64_SOURCE CXXFLAGS=" $CFLAGS -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS" -# This must be the last test to be performed. Any other tests following it -# could fail due to linker errors. libdvdnavmini is intentionally not linked -# against libdvdread (to permit MPlayer to use its own copy of the library). -# So any compilation using the flags added here but not linking against -# libdvdread can fail. echocheck "DVD support (libdvdnav)" -dvdnav_internal=no -if test "$_dvdnav" = auto ; then - if test "$_dvdread_internal" = yes ; then - _dvdnav=yes - dvdnav_internal=yes - res_comment="internal" - else - $_dvdnavconfig --version --minilibs >> $TMPLOG 2>&1 || _dvdnav=no - fi -fi if test "$_dvdnav" = auto ; then _dvdnav=no _dvdnavdir=$($_dvdnavconfig --cflags) @@ -8147,14 +8030,9 @@ if test "$_dvdnav" = auto ; then fi if test "$_dvdnav" = yes ; then def_dvdnav='#define CONFIG_DVDNAV 1' - if test "$dvdnav_internal" = yes ; then - cflags_libdvdnav="-Ilibdvdnav" - inputmodules="dvdnav(internal) $inputmodules" - else - extra_cflags="$extra_cflags $($_dvdnavconfig --cflags)" - extra_ldflags="$extra_ldflags $($_dvdnavconfig --minilibs)" - inputmodules="dvdnav $inputmodules" - fi + extra_cflags="$extra_cflags $($_dvdnavconfig --cflags)" + extra_ldflags="$extra_ldflags $($_dvdnavconfig --libs)" + inputmodules="dvdnav $inputmodules" else def_dvdnav='#undef CONFIG_DVDNAV' noinputmodules="dvdnav $noinputmodules" -- 2.1.1

5 12

MPlayer broken with FFmpeg 2.6 or newer or master branch Libav
by Brad Smith 09 May '15

09 May '15

MPlayer at the moment is broken with FFmpeg 2.6 or newer or master branch Libav. Able to provide a fix? cc -MMD -MP -D_ISOC99_SOURCE -I. -Iffmpeg -O2 -pipe -ffast-math -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE64_SOURCE -I/usr/local/include -DPIC -D_THREAD_SAFE -I/usr/X11R6/include -I/usr/local/include -I/usr/X11R6/include/freetype2 -DZLIB_CONST -I/usr/local/include -I/usr/local/include/opus -I/usr/local/include -I/usr/local/include -c -o libmpcodecs/ad_spdif.o libmpcodecs/ad_spdif.c libmpcodecs/ad_spdif.c:132:15: error: no member named 'raw_packet_buffer_remaining_size' in 'struct AVFormatContext' lavf_ctx->raw_packet_buffer_remaining_size = RAW_PACKET_BUFFER_SIZE; ~~~~~~~~ ^ libmpcodecs/ad_spdif.c:132:50: error: use of undeclared identifier 'RAW_PACKET_BUFFER_SIZE' lavf_ctx->raw_packet_buffer_remaining_size = RAW_PACKET_BUFFER_SIZE; ^ 2 errors generated. -- This message has been scanned for viruses and dangerous content by MailScanner, and is believed to be clean.

4 4

AMD E450 VDPAU GPU lockup triggered by r37367/r37368 changes.
by wallak＠free.fr 04 May '15

04 May '15

Dear All, After extensive tests; I've found that the r37367/r37368 changes trigger a high likely GPU lock on my AMD E450 while using VDPAU. The machine may require a reboot once this happens. Have you an idea about the underlying issue? Is there other configurations affected? Best Regards, Wallak, diff -r -u mplayer-2015-r37368/libmpcodecs/vd_ffmpeg.c mplayer-2015-r37366/libmpcodecs/vd_ffmpeg.c --- mplayer-2010000-r37368/libmpcodecs/vd_ffmpeg.c 2015-04-28 22:59:17.000000000 +0200 +++ mplayer-2010000-r37366/libmpcodecs/vd_ffmpeg.c 2015-04-28 22:50:03.000000000 +0200 @@ -579,12 +579,7 @@ } } -// ignore_aspect is used to not bother with aspect changes during get_buffer. -// The aspect can be wrong when get_buffer is called before decoding and it -// should simply not be necessary. -// In particular this stops spurious get_buffer aspect values from causing -// a container value to be overwritten. -static void update_configuration(sh_video_t *sh, enum AVPixelFormat pix_fmt, int ignore_aspect) { +static void update_configuration(sh_video_t *sh, enum AVPixelFormat pix_fmt) { vd_ffmpeg_ctx *ctx = sh->context; AVCodecContext *avctx = ctx->avctx; int aspect_change = av_cmp_q(avctx->sample_aspect_ratio, ctx->last_sample_aspect_ratio); @@ -594,7 +589,7 @@ } // it is possible another vo buffers to be used after vo config() // lavc reset its buffers on width/heigh change but not on aspect change!!! - if ((!ignore_aspect && aspect_change) || + if (aspect_change || pix_fmt != ctx->pix_fmt || !ctx->vo_initialized) { @@ -603,19 +598,16 @@ // this is a special-case HACK for MPEG-1/2 VDPAU that uses neither get_format nor // sets the value correctly in avcodec_open. set_format_params(avctx, pix_fmt); + mp_msg(MSGT_DECVIDEO, MSGL_V, "[ffmpeg] aspect_ratio: %f\\n", aspect); - if (!ignore_aspect) { - mp_msg(MSGT_DECVIDEO, MSGL_V, "[ffmpeg] aspect_ratio: %f\\n", aspect); - - // Do not overwrite sh->original_aspect on the first call, - // if a container aspect is available (which is preferred then). - // But set it even if the sample aspect did not change, since a - // resolution change can cause an aspect change even if the - // _sample_ aspect is unchanged. - if (sh->original_aspect == 0 || (aspect_change && ctx->last_sample_aspect_ratio.den)) - sh->original_aspect = aspect; - ctx->last_sample_aspect_ratio = avctx->sample_aspect_ratio; - } + // Do not overwrite sh->original_aspect on the first call, + // if a container aspect is available (which is preferred then). + // But set it even if the sample aspect did not change, since a + // resolution change can cause an aspect change even if the + // _sample_ aspect is unchanged. + if (sh->original_aspect == 0 || (aspect_change && ctx->last_sample_aspect_ratio.den)) + sh->original_aspect = aspect; + ctx->last_sample_aspect_ratio = avctx->sample_aspect_ratio; ctx->pix_fmt = pix_fmt; ctx->best_csp = pixfmt2imgfmt2(pix_fmt, avctx->codec_id); } @@ -630,7 +622,7 @@ return 0; } -static int init_vo(sh_video_t *sh, enum AVPixelFormat pix_fmt, int ignore_aspect) +static int init_vo(sh_video_t *sh, enum AVPixelFormat pix_fmt) { vd_ffmpeg_ctx *ctx = sh->context; const AVCodecContext *avctx = ctx->avctx; @@ -659,7 +651,7 @@ if (width != sh->disp_w || height != sh->disp_h) ctx->vo_initialized = 0; - update_configuration(sh, pix_fmt, ignore_aspect); + update_configuration(sh, pix_fmt); if (!ctx->vo_initialized) { sh->disp_w = width; @@ -724,7 +716,7 @@ type = MP_IMGTYPE_NUMBERED; } - if(init_vo(sh, avctx->pix_fmt, 1) < 0){ + if(init_vo(sh, avctx->pix_fmt) < 0){ avctx->release_buffer= avcodec_default_release_buffer; goto disable_dr1; } @@ -1055,7 +1047,7 @@ return NULL; // skipped image } - if(init_vo(sh, avctx->pix_fmt, 0) < 0) return NULL; + if(init_vo(sh, avctx->pix_fmt) < 0) return NULL; if(dr1 && pic->opaque){ mpi=pic->opaque; @@ -1134,14 +1126,14 @@ imgfmt = pixfmt2imgfmt2(fmt[i], avctx->codec_id); if(!IMGFMT_IS_HWACCEL(imgfmt) || !is_in_format_list(sh, imgfmt)) continue; mp_msg(MSGT_DECVIDEO, MSGL_V, MSGTR_MPCODECS_TryingPixfmt, i); - if(init_vo(sh, fmt[i], 1) >= 0) { + if(init_vo(sh, fmt[i]) >= 0) { break; } } selected_format = fmt[i]; if (selected_format == PIX_FMT_NONE) { selected_format = avcodec_default_get_format(avctx, fmt); - update_configuration(sh, selected_format, 1); + update_configuration(sh, selected_format); } set_format_params(avctx, selected_format); return selected_format;

3 4

[PATCH] vo and ao "exit" devices
by Dylan Simon 02 May '15

02 May '15

Someone mentioned that they wanted mplayer to exit if it failed to open their audio device, rather than continuing with no audio. These (trivial but harmless) output devices allow this to happen by putting them at the end of the output chain (e.g., "-ao alsa,pulse,exit"). Apparently someone else suggested this approach as well, a while ago. I will readily admit that this is silly, but it works. I have no particular attachment to name, description, or approach.

3 4

[PATCH] Add Keywords entry to desktop file
by Robbie Harwood 30 Apr '15

30 Apr '15

As per [0], it is desirable (though not required[1]) for .desktop files to contain a "Keywords" entry for ease of indexing. [0]: https://wiki.gnome.org/Initiatives/GnomeGoals/DesktopFileKeywords [1]: http://standards.freedesktop.org/desktop-entry-spec/latest/ar01s05.html Signed-off-by: Robbie Harwood <rharwood(a)club.cc.cmu.edu>

3 2

[PATCH 2/2] Fix compilation with recent FFmpeg on ARM
by Laurent 20 Apr '15

20 Apr '15

Not sure about this one: HAVE_SECTION_DATA_REL_RO. It is defined for Android and Linux, see: http://lists.libav.org/pipermail/libav-commits/2014-December/015860.html So I lazily hardcoded it: diff --git a/configure b/configure index de7c6e4..77115ed 100755 --- a/configure +++ b/configure @@ -9363,6 +9363,7 @@ $def_gnutls #define HAVE_MM_EMPTY 0 #define HAVE_PPC4XX 0 #define HAVE_RDTSC 0 +#define HAVE_SECTION_DATA_REL_RO 1 #define HAVE_SIMD_ALIGN_16 1 #define HAVE_STRERROR_R 0 #define HAVE_STRPTIME 0

4 8

Subversion post-commit hook error
by Ingo Brückl 18 Apr '15

18 Apr '15

I just got Warning: post-commit hook failed (exit code 1) with output: You need version 1.5.0 or better of the Subversion Python bindings. and guess that's on the server, right? Ingo

3 3

[PATCH] Fix some spelling errors
by Robbie Harwood 17 Apr '15

17 Apr '15

Hi, This patch fixes a couple spelling errors uncovered by Lintian. Thanks!

2 1