GNU bug report logs - #57173
[PATCH] gnu: yt-dlp: Update to 2022.08.08.

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Vagrant Cascadian <vagrant <at> debian.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#57173: closed ([PATCH] gnu: yt-dlp: Update to 2022.08.08.)
Date: Sat, 02 Sep 2023 03:59:01 +0000

[Message part 1 (text/plain, inline)]

Your message dated Fri, 01 Sep 2023 20:58:26 -0700
with message-id <87il8tp6ot.fsf <at> wireframe>
and subject line Re: [bug#57173] [PATCH] gnu: yt-dlp: Update to 2022.08.08.
has caused the debbugs.gnu.org bug report #57173,
regarding [PATCH] gnu: yt-dlp: Update to 2022.08.08.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
57173: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=57173
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: raingloom <raingloom <at> riseup.net>
To: guix-patches <at> gnu.org
Cc: raingloom <raingloom <at> riseup.net>
Subject: [PATCH] gnu: yt-dlp: Update to 2022.08.08.
Date: Sat, 13 Aug 2022 15:53:01 +0200

* gnu/packages/video.scm (yt-dlp): Update to 2022.08.08.
  [source]: remove snippet, switch to git-fetch
---
 gnu/packages/video.scm | 23 ++++++-----------------
 1 file changed, 6 insertions(+), 17 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 6d0b5ccb58..94ef3f0468 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -2486,26 +2486,15 @@ (define-public youtube-dl
 (define-public yt-dlp
   (package/inherit youtube-dl
     (name "yt-dlp")
-    (version "2022.07.18")
+    (version "2022.08.08")
     (source (origin
-              (method url-fetch)
-              (uri (string-append "https://github.com/yt-dlp/yt-dlp/"
-                                  "releases/download/"
-                                  version "/yt-dlp.tar.gz"))
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/yt-dlp/yt-dlp/")
+                    (commit version)))
               (sha256
                (base32
-                "1wmzfqhysx1mqdba4ikvm6nbahasihi4xgqwqad20y3vs701slyj"))
-              (snippet
-               #~(begin
-                   ;; Delete the pre-generated files, except for the man page
-                   ;; which requires 'pandoc' to build.
-                   (for-each delete-file
-                             (list "yt-dlp"
-                                   ;;pandoc is needed to generate
-                                   ;;"yt-dlp.1"
-                                   "completions/bash/yt-dlp"
-                                   "completions/fish/yt-dlp.fish"
-                                   "completions/zsh/_yt-dlp"))))))
+                "07qz1zdndlpki0asw35zk5hdjcwpl3n1g54nxg4yb1iykbyv7rll"))))
     (arguments
      (substitute-keyword-arguments (package-arguments youtube-dl)
        ((#:tests? _) #t)
-- 
2.37.1

[Message part 3 (message/rfc822, inline)]

From: Vagrant Cascadian <vagrant <at> debian.org>
To: Csepp <raingloom <at> riseup.net>, Tobias Geerinckx-Rice <me <at> tobias.gr>
Cc: 57173-done <at> debbugs.gnu.org
Subject: Re: [bug#57173] [PATCH] gnu: yt-dlp: Update to 2022.08.08.
Date: Fri, 01 Sep 2023 20:58:26 -0700

[Message part 4 (text/plain, inline)]

On 2022-08-14, Csepp wrote:
> Tobias Geerinckx-Rice <me <at> tobias.gr> writes:
>> raingloom 写道：
>>> * gnu/packages/video.scm (yt-dlp): Update to 2022.08.08.
...
>> When using GIT-FETCH, always set the file-name to something
>> meaningful:
>>
>>  (file-name (git-file-name name version))
>>
>>> -                   ;; Delete the pre-generated files, except for
>>> the man page
>>> -                   ;; which requires 'pandoc' to build.
>>
>> OK, but then we do need to build it ourselves, not remove it.
>>
>> Done in c2481b1bea56f2885110d5af0a6cff36a3d95933.
>
> I lazily assumed it was being generated already, I guess the comment was
> misleading and I should have verified it.
>
>> The new package works fine, but differs slightly:
>> extractor/lazy_extractors.py is no longer installed.  What's up with
>> that?
>
> Not sure what that is or where it's used.  I'll try to take a look soon-ish.
>
>> Leaving open for now,

If there are other outstanding issues, might be best to open a new bug
for them at this point?

yt-dlp has been updated several times since, the latest being:

88d107b2b9bf72a628065a1475ecce7b49852c35 gnu: yt-dlp: Update to 2023.07.06.

Marking as done.

live well,
  vagrant

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.