From patchwork Fri Mar 6 16:00:19 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Robert Smith X-Patchwork-Id: 20553 Return-Path: X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id D76F327BBEA; Fri, 6 Mar 2020 16:01:38 +0000 (GMT) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, MAILING_LIST_MULTI,T_DKIM_INVALID,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.2 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTP id 4B27627BBE4 for ; Fri, 6 Mar 2020 16:01:38 +0000 (GMT) Received: from localhost ([::1]:38534 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jAFPt-0003My-Pe for patchwork@mira.cbaines.net; Fri, 06 Mar 2020 11:01:37 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:38605) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jAFPP-0003Ij-86 for guix-patches@gnu.org; Fri, 06 Mar 2020 11:01:12 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1jAFPK-0008Q5-27 for guix-patches@gnu.org; Fri, 06 Mar 2020 11:01:07 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:39799) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1jAFPJ-0008PM-LY for guix-patches@gnu.org; Fri, 06 Mar 2020 11:01:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1jAFPJ-0000CC-K6 for guix-patches@gnu.org; Fri, 06 Mar 2020 11:01:01 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#39814] [PATCH v2] gnu: anki: Fix mpv audio playback References: <20200227200909.16365-1-robertsmith@posteo.net> In-Reply-To: <20200227200909.16365-1-robertsmith@posteo.net> Resent-From: Robert Smith Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Fri, 06 Mar 2020 16:01:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 39814 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: kuba@kadziolka.net Cc: 39814@debbugs.gnu.org, Robert Smith Received: via spool by 39814-submit@debbugs.gnu.org id=B39814.1583510446724 (code B ref 39814); Fri, 06 Mar 2020 16:01:01 +0000 Received: (at 39814) by debbugs.gnu.org; 6 Mar 2020 16:00:46 +0000 Received: from localhost ([127.0.0.1]:45772 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1jAFOy-0000BX-VR for submit@debbugs.gnu.org; Fri, 06 Mar 2020 11:00:46 -0500 Received: from mout02.posteo.de ([185.67.36.66]:56861) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1jAFOt-0000BD-Qm for 39814@debbugs.gnu.org; Fri, 06 Mar 2020 11:00:39 -0500 Received: from submission (posteo.de [89.146.220.130]) by mout02.posteo.de (Postfix) with ESMTPS id 340F9240102 for <39814@debbugs.gnu.org>; Fri, 6 Mar 2020 17:00:29 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.net; s=2017; t=1583510429; bh=liDpYY+JXakk1Nj66HKodP3E67sXcvnaJ++AgI5dHvI=; h=From:To:Cc:Subject:Date:From; b=cRW3DSpf2f57/5NO3OOKe3wCvrWgTAzPQLefHI4nTRyKvXVpUOhet6NkEyxFDxck/ l42Oyab9ErlKUfxC6/GZy0iZXOwCuDUwLnQC4sqSdQIwaG6qlHk9KVb0rIJTv1Jd8Z eRDALe6BcPuDf7eDgSX/NnRwnaJ/cVBxzsgHRXjXXoSQReVxWwm7tuwfftiMcUAalN 4gA6jSfzBwB/MZhPQlaYrCyueoyRz0FTA3vdhryB4AHphAD3dy5F7+S0/tkF1Hm1qd BMnwE6fW+9r3AqNEI/VJxsLkuxfCVlY2Hh6yZSIkDuc4rFDlIl2sCRh5mb352+jZRY RAkas0s3VDmig== Received: from customer (localhost [127.0.0.1]) by submission (posteo.de) with ESMTPSA id 48Yslc2mvqz9rxV; Fri, 6 Mar 2020 17:00:28 +0100 (CET) From: Robert Smith Date: Fri, 6 Mar 2020 17:00:19 +0100 Message-Id: <20200306160019.13440-1-robertsmith@posteo.net> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.51.188.43 X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: "Guix-patches" X-getmail-retrieved-from-mailbox: Patches * gnu/packages/patches/anki-mpv-args.patch: New file. * gnu/local.mk (dist_patch_DATA): Adjust accordingly. * gnu/packages/education.scm (anki): Adjust accordingly, wrap anki's PATH to include mpv executable. --- Thanks for the feedback Jakub, I've rewritten the patch based on the upstream commit and modified the commit message. --- gnu/local.mk | 1 + gnu/packages/education.scm | 6 +++- gnu/packages/patches/anki-mpv-args.patch | 42 ++++++++++++++++++++++++ 3 files changed, 48 insertions(+), 1 deletion(-) create mode 100644 gnu/packages/patches/anki-mpv-args.patch diff --git a/gnu/local.mk b/gnu/local.mk index 857345cfad..bd4796d41d 100644 --- a/gnu/local.mk +++ b/gnu/local.mk @@ -716,6 +716,7 @@ dist_patch_DATA = \ %D%/packages/patches/akonadi-timestamps.patch \ %D%/packages/patches/allegro-mesa-18.2.5-and-later.patch \ %D%/packages/patches/amule-crypto-6.patch \ + %D%/packages/patches/anki-mpv-args.patch \ %D%/packages/patches/antiword-CVE-2014-8123.patch \ %D%/packages/patches/antlr3-3_1-fix-java8-compilation.patch \ %D%/packages/patches/antlr3-3_3-fix-java8-compilation.patch \ diff --git a/gnu/packages/education.scm b/gnu/packages/education.scm index 20fc9a3706..d8427f6eab 100644 --- a/gnu/packages/education.scm +++ b/gnu/packages/education.scm @@ -738,7 +738,8 @@ adjust the level of difficulty.") (uri (string-append "https://apps.ankiweb.net/downloads/archive/anki-" version "-source.tgz")) (sha256 - (base32 "1gfr51rnllkyzli73p4r51h5ypzfa3m7lic3m3rzpywmqwrxs07k")))) + (base32 "1gfr51rnllkyzli73p4r51h5ypzfa3m7lic3m3rzpywmqwrxs07k")) + (patches (search-patches "anki-mpv-args.patch")))) (build-system gnu-build-system) (arguments `(#:make-flags (list (string-append "PREFIX=" %output)) @@ -779,6 +780,9 @@ adjust the level of difficulty.") (wrap-program program `("QTWEBENGINEPROCESS_PATH" = (,qtwebengineprocess)) + `("PATH" prefix (,(string-append + (assoc-ref inputs "mpv") + "/bin"))) `("PYTHONPATH" = ,site-packages))) (find-files bin "."))) #t))))) diff --git a/gnu/packages/patches/anki-mpv-args.patch b/gnu/packages/patches/anki-mpv-args.patch new file mode 100644 index 0000000000..21b9bd0aba --- /dev/null +++ b/gnu/packages/patches/anki-mpv-args.patch @@ -0,0 +1,42 @@ +Modified from upstream commit: +https://github.com/ankitects/anki/commit/ccd715013609133c55e83924734efa78abc03326 +Fixes mpv argument syntax (support for old syntax removed in mpv 0.31): +https://anki.tenderapp.com/discussions/ankidesktop/38186-mpvprocesserror-unable-to-start-process +Necessary because we are currently unable to upgrade anki to the +latest version in guix (NPM dependencies currently unpackaged). +--- + anki/mpv.py | 4 ++-- + anki/sound.py | 1 - + 2 files changed, 2 insertions(+), 3 deletions(-) + +diff --git a/anki/mpv.py b/anki/mpv.py +index f53d9d0..563fddc 100644 +--- a/anki/mpv.py ++++ b/anki/mpv.py +@@ -104,9 +104,9 @@ class MPVBase: + """ + self.argv = [self.executable] + self.argv += self.default_argv +- self.argv += ["--input-ipc-server", self._sock_filename] ++ self.argv += ["--input-ipc-server="+self._sock_filename] + if self.window_id is not None: +- self.argv += ["--wid", str(self.window_id)] ++ self.argv += ["--wid="+str(self.window_id)] + + def _start_process(self): + """Start the mpv process. +diff --git a/anki/sound.py b/anki/sound.py +index aa3431b..a5fce44 100644 +--- a/anki/sound.py ++++ b/anki/sound.py +@@ -124,7 +124,6 @@ class MpvManager(MPV): + def setMpvConfigBase(base): + mpvConfPath = os.path.join(base, "mpv.conf") + MpvManager.default_argv += [ +- "--no-config", + "--include="+mpvConfPath, + ] + +-- +2.25.1 +