From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp11.migadu.com ([2001:41d0:2:4a6f::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms5.migadu.com with LMTPS id CHyMJsfV7GIbQgAAbAwnHQ (envelope-from ) for ; Fri, 05 Aug 2022 10:33:11 +0200 Received: from aspmx1.migadu.com ([2001:41d0:2:4a6f::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp11.migadu.com with LMTPS id wI52JsfV7GIhSAEA9RJhRA (envelope-from ) for ; Fri, 05 Aug 2022 10:33:11 +0200 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 53A2112CB for ; Fri, 5 Aug 2022 10:33:11 +0200 (CEST) Received: from localhost ([::1]:59664 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1oJsla-0006oW-Em for larch@yhetil.org; Fri, 05 Aug 2022 04:33:10 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:33126) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oJsHS-0006ap-TO for guix-patches@gnu.org; Fri, 05 Aug 2022 04:02:02 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:38077) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1oJsHS-0002fD-Iv for guix-patches@gnu.org; Fri, 05 Aug 2022 04:02:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1oJsHS-0005K6-EP for guix-patches@gnu.org; Fri, 05 Aug 2022 04:02:02 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#56983] [PATCH] gnu: yt-dlp: Update to 2022.07.18, use gexps and new style inputs. Resent-From: Liliana Marie Prikler Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Fri, 05 Aug 2022 08:02:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 56983 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: kiasoc5 , 56983@debbugs.gnu.org Received: via spool by 56983-submit@debbugs.gnu.org id=B56983.165968651320442 (code B ref 56983); Fri, 05 Aug 2022 08:02:02 +0000 Received: (at 56983) by debbugs.gnu.org; 5 Aug 2022 08:01:53 +0000 Received: from localhost ([127.0.0.1]:56058 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1oJsHI-0005Jc-LL for submit@debbugs.gnu.org; Fri, 05 Aug 2022 04:01:53 -0400 Received: from mailrelay.tugraz.at ([129.27.2.202]:57425) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1oJsHG-0005JT-S2 for 56983@debbugs.gnu.org; Fri, 05 Aug 2022 04:01:51 -0400 Received: from lprikler-laptop.ist.intra (gw.ist.tugraz.at [129.27.202.101]) by mailrelay.tugraz.at (Postfix) with ESMTPSA id 4LzdN74h8tz1LZWB; Fri, 5 Aug 2022 10:01:43 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 mailrelay.tugraz.at 4LzdN74h8tz1LZWB DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tugraz.at; s=mailrelay; t=1659686503; bh=82R8c59+eOb+e5ei88ZpYMAUJM0U2HxMfgkUazm3tXE=; h=Subject:From:To:Date:In-Reply-To:References:From; b=rhHarH61sLeU14XpXiqk9T0bGelHas7Wn3ttymCFTlKleyOHAgumTOgDIPg9xc3Z6 PFkswhKGSLqPR6F13+xyuvs5c6WAaYoHIvVbQdg6ksZU+qpctDtFZAoSI9qSW8nC0i rsMyVlhVDLjWeMjkeNnxXg/AGoBZS+vXAPl5sfMo= Message-ID: <26f9b3120f00ca03839ac9a8a974a61759bfcff4.camel@ist.tugraz.at> From: Liliana Marie Prikler Date: Fri, 05 Aug 2022 10:01:42 +0200 In-Reply-To: <20220805053011.387828-1-kiasoc5@disroot.org> References: <20220805053011.387828-1-kiasoc5@disroot.org> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.42.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-TUG-Backscatter-control: waObeELIUl4ypBWmcn/8wQ X-Scanned-By: MIMEDefang 2.74 on 129.27.10.116 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" X-Migadu-Flow: FLOW_IN X-Migadu-To: larch@yhetil.org X-Migadu-Country: US ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1659688391; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: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:dkim-signature; bh=82R8c59+eOb+e5ei88ZpYMAUJM0U2HxMfgkUazm3tXE=; b=jecK4QHV113K3/L3p0UkhpVxQttv+d7x27RW8qmvEjPHh7lp3WTE/m4ORwRnYb9Lxw6RL7 /ffbzsuydQ76CmZVKcM4UjmfNrMveHyAoCWKS0B7RvEwp83MgwAqf7xJzMxH5xWQ0m9+pl yBakRC+6c60JA06Yg9Vpea2lEqJE7MtlBkDSnFoIK2aKiYul8Se0JT9GyhKGczpkUlcOho Yaw/keMCixeYS7AbavecUxo80gKvTvOhijKty+GfXqKJAlKodwMAo/SVfuQnjecbCidWf5 3OlvGmaqKXGtUgvuGaOwOGbJVscCUMkqlt9DenPJuGiWEs6Jg/5dn2O1CCoOHw== ARC-Seal: i=1; s=key1; d=yhetil.org; t=1659688391; a=rsa-sha256; cv=none; b=GCADIPVfgQDdEUVxYoaZC4BVzqAG9gxEsXNpxKhHEhUQf/xis16G76zeh6aksMm35GbSJW 1hKYAFRHXjXA9nNk7lbqv1mzRpEwzj9YQaFzWwGyo6h+mQ2TIxWVV025NYGq2TmWEGr6V+ 9Mq38WH7ZW31JX0IlBpFi3BOBez4BjkNt6eAAho0wA6HzZYtppEGWmr7TePMw7IUyxf42l wQLAKkAT4ANbrO8N3uHnPq1nmtCqFE41EM4H+ODlc04SuE8vKbMRODP40jFu0TcY5tpY2Q ZX5ujalsmj+sqKoCX9pvi5ZGOyLMAKStD2eXBEwNMLBqS32F09om2g09A+65kg== ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=tugraz.at header.s=mailrelay header.b=rhHarH61; dmarc=fail reason="SPF not aligned (relaxed)" header.from=tugraz.at (policy=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" X-Migadu-Spam-Score: 5.89 Authentication-Results: aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=tugraz.at header.s=mailrelay header.b=rhHarH61; dmarc=fail reason="SPF not aligned (relaxed)" header.from=tugraz.at (policy=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" X-Migadu-Queue-Id: 53A2112CB X-Spam-Score: 5.89 X-Migadu-Scanner: scn0.migadu.com X-TUID: BgNhjVZyHWOk Am Freitag, dem 05.08.2022 um 01:30 -0400 schrieb kiasoc5: > * gnu/packages/video.scm (yt-dlp): Update to 2022.07.18. -- Split -- > [arguments]: Use gexps. > [inputs]: Use new style. > [native-inputs]: Likewise. For the second patch, use "gnu: yt-dlp: Use new package style." as title. > --- >  gnu/packages/video.scm | 70 ++++++++++++++++++++---------------------- >  1 file changed, 34 insertions(+), 36 deletions(-) > > diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm > index 04049fd9c8..8cd6d03688 100644 > --- a/gnu/packages/video.scm > +++ b/gnu/packages/video.scm > @@ -2474,7 +2474,7 @@ (define-public youtube-dl >  (define-public yt-dlp >    (package/inherit youtube-dl >      (name "yt-dlp") > -    (version "2022.06.22.1") > +    (version "2022.07.18") >      (source (origin >                (method url-fetch) >                (uri (string-append "https://github.com/yt-dlp/yt-dlp/" > @@ -2482,7 +2482,7 @@ (define-public yt-dlp >                                    version "/yt-dlp.tar.gz")) >                (sha256 >                 (base32 > -                > "1nr6g3dhvjc10jzhyvgjrrxqhsgi3hiw8bswp8bi6bscimd9vhps")) > +                > "1wmzfqhysx1mqdba4ikvm6nbahasihi4xgqwqad20y3vs701slyj")) >                (snippet >                 '(begin >                    ;; Delete the pre-generated files, except for the > man page Haven't checked the hash, but on the premise that it's good, LGTM. -- Split -- > @@ -2497,40 +2497,38 @@ (define-public yt-dlp >       (substitute-keyword-arguments (package-arguments youtube-dl) >         ((#:tests? _) #t) >         ((#:phases phases) > -        `(modify-phases ,phases > -           ;; See the comment for the corresponding phase in > youtube-dl. > -           (replace 'default-to-the-ffmpeg-input > -             (lambda _ > -               (substitute* "yt_dlp/postprocessor/ffmpeg.py" > -                 (("\\.get_param\\('ffmpeg_location'\\)" match) > -                  (format #f "~a or '~a'" match (which > "ffmpeg")))))) > -           (replace 'build-generated-files > -             (lambda _ > -               ;; Avoid the yt-dlp.1 target, which requires pandoc. > -               (invoke "make" "PYTHON=python" "yt-dlp" > "completions"))) > -           (replace 'fix-the-data-directories > -             (lambda* (#:key outputs #:allow-other-keys) > -               (let ((prefix (assoc-ref outputs "out"))) > -                 (substitute* "setup.py" > -                   (("'etc/") > -                    (string-append "'" prefix "/etc/")) > -                   (("'share/") > -                    (string-append "'" prefix "/share/")))))) > -           (delete 'install-completion) > -           (replace 'check > -             (lambda* (#:key tests? #:allow-other-keys) > -               (when tests? > -                 (invoke "pytest" "-k" "not download")))))))) > -    (inputs > -     `(("python-brotli" ,python-brotli) > -       ("python-certifi" ,python-certifi) > -       ("python-mutagen" ,python-mutagen) > -       ("python-pycryptodomex" ,python-pycryptodomex) > -       ("python-websockets" ,python-websockets) > -       ,@(package-inputs youtube-dl))) > -    (native-inputs > -     `(("python-pytest" ,python-pytest) > -       ,@(package-native-inputs youtube-dl))) > +         #~(modify-phases #$phases > +            ;; See the comment for the corresponding phase in > youtube-dl. > +            (replace 'default-to-the-ffmpeg-input > +              (lambda _ > +                (substitute* "yt_dlp/postprocessor/ffmpeg.py" > +                  (("\\.get_param\\('ffmpeg_location'\\)" match) > +                   (format #f "~a or '~a'" match (which > "ffmpeg")))))) > +            (replace 'build-generated-files > +              (lambda _ > +                ;; Avoid the yt-dlp.1 target, which requires pandoc. > +                (invoke "make" "PYTHON=python" "yt-dlp" > "completions"))) > +            (replace 'fix-the-data-directories > +              (lambda* (#:key outputs #:allow-other-keys) > +                (let ((prefix (assoc-ref outputs "out"))) > +                  (substitute* "setup.py" > +                    (("'etc/") > +                     (string-append "'" prefix "/etc/")) > +                    (("'share/") > +                     (string-append "'" prefix "/share/")))))) > +            (delete 'install-completion) > +            (replace 'check > +              (lambda* (#:key tests? #:allow-other-keys) > +                (when tests? > +                  (invoke "pytest" "-k" "not download")))))))) The indentation here seems a little off. Did you TAB in Emacs? > +    (inputs (modify-inputs (package-inputs youtube-dl) > +              (append python-brotli > +                      python-certifi > +                      python-mutagen > +                      python-pycryptodomex > +                      python-websockets))) > +    (native-inputs (modify-inputs (package-native-inputs youtube-dl) > +                     (append python-pytest))) LGTM. Cheers