Thu Nov 3 14:41:28 2016 UTC ()
Make mpv agnostic regarding possible mismatches of built vs running
dependencies. Given the existence of BUILDLINK_A[BP]I_DEPENDS and
PKGREVISION bumps our use in pkgsrc can be considered a legitimate
one.

Delete the print_libav_versions() check to reflect that.

Bump PKGREVISION.

Thanks to <wiz> for noticing that!


(leot)
diff -r1.41 -r1.42 pkgsrc/multimedia/mpv/Makefile
diff -r1.27 -r1.28 pkgsrc/multimedia/mpv/distinfo
diff -r0 -r1.1 pkgsrc/multimedia/mpv/patches/patch-player_main.c

cvs diff -r1.41 -r1.42 pkgsrc/multimedia/mpv/Makefile (expand / switch to context diff)
--- pkgsrc/multimedia/mpv/Makefile 2016/10/20 08:27:01 1.41
+++ pkgsrc/multimedia/mpv/Makefile 2016/11/03 14:41:28 1.42
@@ -1,6 +1,7 @@
-# $NetBSD: Makefile,v 1.41 2016/10/20 08:27:01 leot Exp $
+# $NetBSD: Makefile,v 1.42 2016/11/03 14:41:28 leot Exp $
 
 DISTNAME=	mpv-0.21.0
+PKGREVISION=	1
 CATEGORIES=	multimedia
 MASTER_SITES=	${MASTER_SITE_GITHUB:=mpv-player/}
 GITHUB_TAG=	v${PKGVERSION_NOREV}

cvs diff -r1.27 -r1.28 pkgsrc/multimedia/mpv/distinfo (expand / switch to context diff)
--- pkgsrc/multimedia/mpv/distinfo 2016/10/20 08:27:01 1.27
+++ pkgsrc/multimedia/mpv/distinfo 2016/11/03 14:41:28 1.28
@@ -1,6 +1,7 @@
-$NetBSD: distinfo,v 1.27 2016/10/20 08:27:01 leot Exp $
+$NetBSD: distinfo,v 1.28 2016/11/03 14:41:28 leot Exp $
 
 SHA1 (mpv-0.21.0.tar.gz) = 8f90df723faab7fe019c9b29898f656688e22796
 RMD160 (mpv-0.21.0.tar.gz) = 7ca393d2d80cc3f56cf93a871e39b332883a0899
 SHA512 (mpv-0.21.0.tar.gz) = f4072fc3a351643f94c64c8da43560458d53072e61d099670903319f21576ccbdfa053a57c1ac09fe51e067e5055e1bb691ee93645ea2dea04b95fd96d937d16
 Size (mpv-0.21.0.tar.gz) = 2812584 bytes
+SHA1 (patch-player_main.c) = fefba566f5130d24b748b7ab4ecc48bf42bc6bdd

File Added: pkgsrc/multimedia/mpv/patches/Attic/patch-player_main.c
$NetBSD: patch-player_main.c,v 1.1 2016/11/03 14:41:28 leot Exp $

Avoid to check mismatched built vs running libraries version.
The use in pkgsrc can be considered a legitimate one.

--- player/main.c.orig	2016-10-19 21:10:13.000000000 +0000
+++ player/main.c
@@ -429,18 +429,6 @@ int mp_initialize(struct MPContext *mpct
 
     handle_deprecated_options(mpctx);
 
-    if (!print_libav_versions(mp_null_log, 0)) {
-        // Using mismatched libraries can be legitimate, but even then it's
-        // a bad idea. We don't acknowledge its usefulness and stability.
-        print_libav_versions(mpctx->log, MSGL_FATAL);
-        MP_FATAL(mpctx, "\nmpv was compiled against a different version of "
-                 "FFmpeg/Libav than the shared\nlibrary it is linked against. "
-                 "This is most likely a broken build and could\nresult in "
-                 "misbehavior and crashes.\n\nmpv does not support this "
-                 "configuration and will not run - rebuild mpv instead.\n");
-        return -1;
-    }
-
     if (!mpctx->playlist->first && !opts->player_idle_mode)
         return -3;