[bug#54018] [PATCH] gnu: gallery-dl: Update to 1.20.5.

[bug#54018] [PATCH] gnu: gallery-dl: Update to 1.20.5.

[Jai Vetrivelan]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: 0001-gnu-gallery-dl-Update-to-1.20.5.patch --]
[-- Type: text/patch, Size: 1303 bytes --]

From 2d6d8be194e41bcc878d0a065520bf6fea737965 Mon Sep 17 00:00:00 2001
From: Jai Vetrivelan <jaivetrivelan@gmail.com>
Date: Wed, 16 Feb 2022 06:35:38 +0530
Subject: [PATCH] gnu: gallery-dl: Update to 1.20.5.

* gnu/packages/video.scm (gallery-dl): Update to 1.20.5.
---
 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 b0c5e5be1f..e430cf1545 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -2263,7 +2263,7 @@ (define-public smplayer
 (define-public gallery-dl
   (package
     (name "gallery-dl")
-    (version "1.20.3")
+    (version "1.20.5")
     (source (origin
               (method url-fetch)
               (uri (string-append "https://github.com/mikf/gallery-dl"
@@ -2271,7 +2271,7 @@ (define-public gallery-dl
                                   version ".tar.gz"))
               (sha256
                (base32
-                "0mh57fbq9xkkhqiy7cq5ahwjp464hgxmkrvq0pxxr85212yrf7bd"))))
+                "0149hd4nh7bk2yqjdnvrkvag5mwgsb6gxrgfcd88q4cv3k2ji42h"))))
     (build-system python-build-system)
     (inputs (list python-requests ffmpeg))
     (home-page "https://github.com/mikf/gallery-dl")

base-commit: 52269e27c628d34af896e95347870b1755cb3e5f
-- 
2.34.0

[bug#54018] video.scm? (was: [PATCH] gnu: gallery-dl: Update to 1.20.5.)

[Jai Vetrivelan]

Why is this package in video.scm?  As far as I understand, the primary
function of gallery-dl is not downloading videos.

Would it be better if the package definition existed somewhere else?
-- 
Jai Vetrivelan

bug#54018: [PATCH] gnu: gallery-dl: Update to 1.20.5.

[Leo Famulari]

On Wed, Feb 16, 2022 at 06:36:23AM +0530, Jai Vetrivelan wrote:
> From 2d6d8be194e41bcc878d0a065520bf6fea737965 Mon Sep 17 00:00:00 2001
> From: Jai Vetrivelan <jaivetrivelan@gmail.com>
> Date: Wed, 16 Feb 2022 06:35:38 +0530
> Subject: [PATCH] gnu: gallery-dl: Update to 1.20.5.
> 
> * gnu/packages/video.scm (gallery-dl): Update to 1.20.5.

Thanks! Pushed as e93c3d441453a72f9e6223e65c24779a117a90b1

[bug#54018] video.scm? (was: [PATCH] gnu: gallery-dl: Update to 1.20.5.)

[Leo Famulari]

On Wed, Feb 16, 2022 at 06:16:03PM +0530, Jai Vetrivelan wrote:
> Why is this package in video.scm?  As far as I understand, the primary
> function of gallery-dl is not downloading videos.

The description says that it can download video galleries, and the name
of the program seems to be modeled after 'youtube-dl'.

> Would it be better if the package definition existed somewhere else?

No, it wouldn't be better or worse if it was in another module.

However, moving packages between modules is something we avoid doing
without a concrete justification, because it makes the Git history of a
package harder to understand and breaks 3rd-party channels. We don't go
out of our way to avoid breaking 3rd-party channels, but we shouldn't do
it gratuitously either.