[bug#72180] [PATCH 0/2] gnu: mpv: Update to 0.38.0 and include sixel support.

[bug#72180] [PATCH 0/2] gnu: mpv: Update to 0.38.0 and include sixel support.

[Mathieu Laparie]

I needed sixel support in mpv and noticed our package did not include it, and updated to 0.38.0 while I was at it. This inmplied upgrading libplacebo to 6.338.2, but 7.349.0 is available and works too, so I jumped to that version. mpv 0.38.0 comes with a number of improvements on top of the sixel support included here, among which better playlist display in terminal (was previously garbled with broken character sequences hardcoded from the OSD), support for --input-commands to run commands at startup, zooming on Ctrl+wheel, etc.

The pkg-config name of libsixel seems to be gdk-pixbuf, and it appeared that adding just libsixel in inputs was not enough for mpv to build with sixel support.

Mathieu Laparie (2):
  gnu: libplacebo: Update to 7.349.0.
  gnu: mpv: Update to 0.38.0, include sixel support.

 gnu/packages/video.scm | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

-- 
2.45.2

[bug#72180] [PATCH 1/2] gnu: libplacebo: Update to 7.349.0.

[Mathieu Laparie]

From: Mathieu Laparie <mathieu.laparie@inrae.fr>

    * gnu/pacakges/video.scm (libplacebo): Update to 7.349.0.
---
 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 8728da1775..c7470284dc 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -1549,7 +1549,7 @@ (define-public libmatroska
 (define-public libplacebo
   (package
     (name "libplacebo")
-    (version "6.338.1")
+    (version "7.349.0")
     (source
      (origin
        (method git-fetch)
@@ -1558,7 +1558,7 @@ (define-public libplacebo
              (commit (string-append "v" version))))
        (file-name (git-file-name name version))
        (sha256
-        (base32 "1miqk3gfwah01xkf4a6grwq29im0lfh94gp92y7js855gx3v169m"))))
+        (base32 "1q12yf04z570a2l3vkig2iqm7bgqr90hdxmn78sk2injrsyx124q"))))
     (build-system meson-build-system)
     (arguments
      (list #:configure-flags
-- 
2.45.2

[bug#72180] [PATCH 2/2] gnu: mpv: Update to 0.38.0, include sixel support.

[Mathieu Laparie]

From: Mathieu Laparie <mathieu.laparie@inrae.fr>

    * gnu/packages/video.scm (mpv): Update to 0.38.0, include sixel support.
    [inputs] Add gdk-pixbuf, libsixel.
---
 gnu/packages/video.scm | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index c7470284dc..3a715cce6a 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -2618,7 +2618,7 @@ (define-public mplayer
 (define-public mpv
   (package
     (name "mpv")
-    (version "0.37.0")
+    (version "0.38.0")
     (source (origin
               (method git-fetch)
               (uri (git-reference
@@ -2626,7 +2626,7 @@ (define-public mpv
                     (commit (string-append "v" version))))
               (file-name (git-file-name name version))
               (sha256
-               (base32 "1xcyfpd543lbmg587wi0mahrz8vhyrlr4432054vp6wsi3s36c4b"))))
+               (base32 "11l8b9cka81xwrcc148g6avj7jcz8khz3h3xpyadm5265afa6mkl"))))
     (build-system meson-build-system)
     (arguments
      (list
@@ -2658,9 +2658,11 @@ (define-public mpv
     ;; Missing features: libguess, V4L2.
     (inputs
      (list enca
+           gdk-pixbuf
            ladspa
            lcms
            libbs2b
+           libsixel
            mpg123
            rsound
            vulkan-headers
-- 
2.45.2

[bug#72180] [PATCH 0/2] gnu: mpv: Update to 0.38.0 and include sixel support.

[Z572 via Guix-patches via]

[-- Attachment #1: Type: text/plain, Size: 1177 bytes --]

Mathieu Laparie <matf@disr.it> writes:

> I needed sixel support in mpv and noticed our package did not include it, and updated to 0.38.0 while I was at it. This inmplied upgrading libplacebo to 6.338.2, but 7.349.0 is available and works too, so I jumped to that version. mpv 0.38.0 comes with a number of improvements on top of the sixel support included here, among which better playlist display in terminal (was previously garbled with broken character sequences hardcoded from the OSD), support for --input-commands to run commands at startup, zooming on Ctrl+wheel, etc.
>
> The pkg-config name of libsixel seems to be gdk-pixbuf, and it appeared that adding just libsixel in inputs was not enough for mpv to build with sixel support.

I think should move gdk-pixbuf to libsixel's propagated-inputs, and add
a comment say gdk-pixbuf pkg-config's “Requires.private” need
gdk-pixbuf, and add a TODO for "remove it when we use pkgconf."

>
> Mathieu Laparie (2):
>   gnu: libplacebo: Update to 7.349.0.
>   gnu: mpv: Update to 0.38.0, include sixel support.
>
>  gnu/packages/video.scm | 10 ++++++----
>  1 file changed, 6 insertions(+), 4 deletions(-)

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

bug#72180: [PATCH 0/2] gnu: mpv: Update to 0.38.0 and include sixel support.

[Nicolas Goaziou via Guix-patches via]

Hello,

> Mathieu Laparie <matf@disr.it> writes:
> 
> > I needed sixel support in mpv and noticed our package did not include it, and updated to 0.38.0 while I was at it. This inmplied upgrading libplacebo to 6.338.2, but 7.349.0 is available and works too, so I jumped to that version. mpv 0.38.0 comes with a number of improvements on top of the sixel support included here, among which better playlist display in terminal (was previously garbled with broken character sequences hardcoded from the OSD), support for --input-commands to run commands at startup, zooming on Ctrl+wheel, etc.
> >
> > The pkg-config name of libsixel seems to be gdk-pixbuf, and it appeared that adding just libsixel in inputs was not enough for mpv to build with sixel support.
> 
> I think should move gdk-pixbuf to libsixel's propagated-inputs, and add
> a comment say gdk-pixbuf pkg-config's “Requires.private” need
> gdk-pixbuf, and add a TODO for "remove it when we use pkgconf."

Thank you. I applied the patch with the suggestions above. MPV had
already been updated, tho.

Regards,
-- 
Nicolas Goaziou