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

[GNU bug Tracking System]

Your message dated Thu, 05 Sep 2024 22:24:15 +0200
with message-id <878qw5q26o.fsf@nicolasgoaziou.fr>
and subject line Re: [PATCH 0/2] gnu: mpv: Update to 0.38.0 and include sixel 
support.
has caused the debbugs.gnu.org bug report #72180,
regarding [PATCH 0/2] gnu: mpv: Update to 0.38.0 and include sixel support.
to be marked as done.

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


-- 
72180: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=72180
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> [PATCH 0/2] gnu: mpv: Update to 0.38.0 and include sixel support.
>
>
>
>
> Date: 
>
> Thu, 18 Jul 2024 19:55:22 +0200
>
>
>
>
> 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
>
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: [PATCH 0/2] gnu: mpv: Update to 0.38.0 and include sixel support.
>
>
>
>
> Date: 
>
> Thu, 05 Sep 2024 22:24:15 +0200
>
>
>
>
> 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
>
>
>
>
> --- End Message ---