From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp10.migadu.com ([2001:41d0:8:6d80::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms5.migadu.com with LMTPS id UIbzMJVxBGQ+EgEAbAwnHQ (envelope-from ) for ; Sun, 05 Mar 2023 11:40:21 +0100 Received: from aspmx1.migadu.com ([2001:41d0:8:6d80::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp10.migadu.com with LMTPS id yBYyMJVxBGTEMAAAG6o9tA (envelope-from ) for ; Sun, 05 Mar 2023 11:40:21 +0100 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id 793FF2CB8A for ; Sun, 5 Mar 2023 11:40:21 +0100 (CET) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1pYlme-00022j-7z; Sun, 05 Mar 2023 05:40:04 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1pYlmc-00022P-W2 for guix-patches@gnu.org; Sun, 05 Mar 2023 05:40:03 -0500 Received: from debbugs.gnu.org ([209.51.188.43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1pYlmc-0004a3-Ms for guix-patches@gnu.org; Sun, 05 Mar 2023 05:40:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1pYlmc-0001qz-6z for guix-patches@gnu.org; Sun, 05 Mar 2023 05:40:02 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#61117] Update svtplay-dl to 4.18 Resent-From: Jessica Tallon Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Sun, 05 Mar 2023 10:40:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 61117 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: moreinfo To: Tobias Geerinckx-Rice Cc: 61117@debbugs.gnu.org X-Debbugs-Original-Cc: 61117@debbugs.gnu.org, guix-patches@gnu.org Received: via spool by submit@debbugs.gnu.org id=B.16780127657070 (code B ref -1); Sun, 05 Mar 2023 10:40:02 +0000 Received: (at submit) by debbugs.gnu.org; 5 Mar 2023 10:39:25 +0000 Received: from localhost ([127.0.0.1]:38557 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1pYlm0-0001pw-Fj for submit@debbugs.gnu.org; Sun, 05 Mar 2023 05:39:24 -0500 Received: from lists.gnu.org ([209.51.188.17]:34796) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1pYllx-0001pi-R7 for submit@debbugs.gnu.org; Sun, 05 Mar 2023 05:39:23 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1pYllx-0001yW-IA for guix-patches@gnu.org; Sun, 05 Mar 2023 05:39:21 -0500 Received: from mout-p-102.mailbox.org ([2001:67c:2050:0:465::102]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_CHACHA20_POLY1305:256) (Exim 4.90_1) (envelope-from ) id 1pYllv-0004Va-Ay for guix-patches@gnu.org; Sun, 05 Mar 2023 05:39:21 -0500 Received: from smtp2.mailbox.org (smtp2.mailbox.org [10.196.197.2]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-384) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mout-p-102.mailbox.org (Postfix) with ESMTPS id 4PTyr21BTdz9sdQ; Sun, 5 Mar 2023 11:39:14 +0100 (CET) Date: Sun, 5 Mar 2023 11:39:10 +0100 From: Jessica Tallon Message-ID: <20230305112617.3babd216@tsyesika.se> In-Reply-To: <87357ult1k.fsf@nckx> References: <5646122.DvuYhMxLoT@localhost.localdomain> <87357ult1k.fsf@nckx> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="MP_/hKS+q8zEHoSCgTjYOovsjLs" Received-SPF: pass client-ip=2001:67c:2050:0:465::102; envelope-from=tsyesika@tsyesika.se; helo=mout-p-102.mailbox.org X-Spam_score_int: -25 X-Spam_score: -2.6 X-Spam_bar: -- X-Spam_report: (-2.6 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_DNSWL_LOW=-0.7, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+larch=yhetil.org@gnu.org Sender: guix-patches-bounces+larch=yhetil.org@gnu.org X-Migadu-Flow: FLOW_IN X-Migadu-Country: US ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1678012821; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:resent-cc:resent-from:resent-sender: resent-message-id:in-reply-to:in-reply-to:references:references: list-id:list-help:list-unsubscribe:list-subscribe:list-post; bh=e+eqpwPkhbUZ0G0hCkgNJVcGcts0F37Efko/L3SNBjo=; b=hjrmN/GL6N36jQVanGE138Y4U4WvZiNSRL4+3dnvj3EqyCcF34LUTSBRkzlJTBB+M77hMY Eo92ot9OHX78+pIGcQ6f+qGISho2C5lWihdI/yZ7CroXYGJ8EGmQUu/IDxmBTq2JkTBbib qkOYhycBhMKdYmo120vVxiFFwtbwwRlz2tIwqmS3755MhO7IytCDXVsio+oJgVu/KtCZbA uTjdz/MKyBmtPBHi4pnCdbWxMpTSmUdDzKGfUWkvSyMzKJxHDLwvDGpfuVyJa5G05rY2jT d9iJQ/8jL+wZcNVomtLsYOxwEqkbrEx+PETRWPErUYOPitlXfjOfhzTOdW+zdA== ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=none; spf=pass (aspmx1.migadu.com: domain of "guix-patches-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="guix-patches-bounces+larch=yhetil.org@gnu.org"; dmarc=none ARC-Seal: i=1; s=key1; d=yhetil.org; t=1678012821; a=rsa-sha256; cv=none; b=fntXlhnILHGh0U6whxw/FvmgFzqZyAX2D6xpZ5DjQ0l/ZMPEClwFNpDGNRRfaZWhAQOzin mbCA1F9JWhURu/8hmb0l/5kB6j98lxAMCqg5qX5BAAVk8h7m4uXsmdUWNaDpzic1S6hpU4 0T4+Q2oOFXVImuxQt5jSy3RfswGwNvT+0yjGPd97gQXe48HryCSKy5Py0F0oYDwavgyb1e qHXBCiCMRD0YbjLcOgqclRgxzLSERCMGl1KZhJVPfgWcGSwHFN0j8Pr1Gb68JjUels2Q6X 2Dq5OO6T+BQk6kx8l6+449yUcPTrt2/esQ1+i9oRGFF37vkIDbgHZo+4Mmcofg== X-Migadu-Scanner: scn1.migadu.com X-Migadu-Spam-Score: -2.01 X-Spam-Score: -2.01 X-Migadu-Queue-Id: 793FF2CB8A Authentication-Results: aspmx1.migadu.com; dkim=none; spf=pass (aspmx1.migadu.com: domain of "guix-patches-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="guix-patches-bounces+larch=yhetil.org@gnu.org"; dmarc=none X-TUID: zy6D/Cb5K0ob --MP_/hKS+q8zEHoSCgTjYOovsjLs Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Content-Disposition: inline Hello, Sorry for the long delay on addressing this feedback. Tobias Geerinckx-Rice writes: > Jessica, > > Please avoid propagation whenever possible; it breaks all kinds of > nice things. > > Propagating A from B means that Guix will effectively =E2=80=98install=E2= =80=99 A > alongside B when the user installs only B. Or how classical package > managers bluntly handle =E2=80=98dependencies=E2=80=99. Not good! > Thanks for the tip, I've included a patch which does it this way. > >> it's needed while the program is running to merge two files, not >> while building. > > So 4.17 did not need or use ffmpeg this way? > > Otherwise, put each unrelated changes into a separate commit: one to > improve the ffmpeg situation, and one to update svtplay-dl. You can > send multiple related commits as one patch series to one bug number, > though. 4.17 did indeed need ffmpeg, I've included two patches one which resolves the ffmpeg issue and the other which updates it to 4.18. Thanks, Jessica. --MP_/hKS+q8zEHoSCgTjYOovsjLs Content-Type: text/x-patch Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=0000-cover-letter.patch >From 1b81f1786ba0cb038bf76ab74cbc90847d06206b Mon Sep 17 00:00:00 2001 Message-Id: From: Jessica Tallon Date: Sun, 5 Mar 2023 11:02:49 +0100 Subject: [PATCH 0/2] Upgrade svtplay-dl to 4.18 & fix wrapping with ffmpeg The first patch fixes svtplay-dl which needs ffmpeg to create a unified file with both sound and video combined. Before it didn't have access to ffmpeg so it would display an error and leave the two files uncombined. The second upgrades svtplay-dl to the latest version, 4.18. Jessica Tallon (2): gnu: fix svtplay-dl wrapping to use ffmpeg gnu: svtplay-dl update to 4.18 gnu/packages/video.scm | 18 +++++++++++++++--- 1 file changed, 15 insertions(+), 3 deletions(-) -- 2.39.1 --MP_/hKS+q8zEHoSCgTjYOovsjLs Content-Type: text/x-patch Content-Transfer-Encoding: quoted-printable Content-Disposition: attachment; filename=0001-gnu-fix-svtplay-dl-wrapping-to-use-ffmpeg.patch =46rom e9d073dbf37323d11bddae32a1c54d65864959be Mon Sep 17 00:00:00 2001 Message-Id: In-Reply-To: References: From: Jessica Tallon Date: Sun, 5 Mar 2023 10:43:58 +0100 Subject: [PATCH 1/2] gnu: fix svtplay-dl wrapping to use ffmpeg * gnu/packages/video.scm (svtplay-dl): Wrap program to give access to ffmpe= g. --- gnu/packages/video.scm | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm index 13299fdf06..721f7c4d4e 100644 --- a/gnu/packages/video.scm +++ b/gnu/packages/video.scm @@ -64,6 +64,7 @@ ;;; Copyright =C2=A9 2022 Chadwain Holness ;;; Copyright =C2=A9 2022 Andy Tai ;;; Copyright =C2=A9 2023 Ott Joon +;;; Copyright =C2=A9 2023 Jessica Tallon ;;; ;;; This file is part of GNU Guix. ;;; @@ -5690,8 +5691,19 @@ (define-public svtplay-dl (base32 "0yjxmvldskw4pji3lg69pbx05izvxahz9my7z5p31mkiz6v33dmx")))) (build-system python-build-system) + (arguments + (list + #:phases + #~(modify-phases %standard-phases + (add-after 'install 'wrap-executable + (lambda* (#:key inputs outputs #:allow-other-keys) + (let ((out (assoc-ref outputs "out")) + (ffmpeg (assoc-ref inputs "ffmpeg"))) + (wrap-program (string-append out "/bin/svtplay-dl") + `("PATH" ":" prefix + ,(list (string-append ffmpeg "/bin")))))))))) (inputs (list ffmpeg python-pyaml python-requests python-pysocks - python-cryptography)) + python-cryptography bash-minimal)) (home-page "https://svtplay-dl.se/") (synopsis "Download or stream SVT Play's (and others) TV programmes") (description --=20 2.39.1 --MP_/hKS+q8zEHoSCgTjYOovsjLs Content-Type: text/x-patch Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=0002-gnu-svtplay-dl-update-to-4.18.patch >From 1b81f1786ba0cb038bf76ab74cbc90847d06206b Mon Sep 17 00:00:00 2001 Message-Id: <1b81f1786ba0cb038bf76ab74cbc90847d06206b.1678010569.git.tsyesika@tsyesika.se> In-Reply-To: References: From: Jessica Tallon Date: Sun, 5 Mar 2023 10:56:54 +0100 Subject: [PATCH 2/2] gnu: svtplay-dl update to 4.18 * gnu/packages/video.scm (svtplay-dl) update to 4.18 --- gnu/packages/video.scm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm index 721f7c4d4e..fbebaaff28 100644 --- a/gnu/packages/video.scm +++ b/gnu/packages/video.scm @@ -5680,7 +5680,7 @@ (define-public video-contact-sheet (define-public svtplay-dl (package (name "svtplay-dl") - (version "4.17") + (version "4.18") (source (origin (method git-fetch) (uri (git-reference @@ -5689,7 +5689,7 @@ (define-public svtplay-dl (file-name (git-file-name name version)) (sha256 (base32 - "0yjxmvldskw4pji3lg69pbx05izvxahz9my7z5p31mkiz6v33dmx")))) + "1xb2n3bwjddr86mjynbfd8m2g98gzqbj5mmsfc735q9xp14l30pf")))) (build-system python-build-system) (arguments (list -- 2.39.1 --MP_/hKS+q8zEHoSCgTjYOovsjLs--