From: Nicolas Goaziou via Guix-patches via <guix-patches@gnu.org>
To: 72180-done@debbugs.gnu.org
Cc: Z572 <873216071@qq.com>, Mathieu Laparie <matf@disr.it>
Subject: bug#72180: [PATCH 0/2] gnu: mpv: Update to 0.38.0 and include sixel support.
Date: Thu, 05 Sep 2024 22:24:15 +0200 [thread overview]
Message-ID: <878qw5q26o.fsf@nicolasgoaziou.fr> (raw)
In-Reply-To: <tencent_172059417F073A84E558255CB5419BB5A908@qq.com>
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
prev parent reply other threads:[~2024-09-05 20:25 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-18 17:55 [bug#72180] [PATCH 0/2] gnu: mpv: Update to 0.38.0 and include sixel support Mathieu Laparie
2024-07-18 17:57 ` [bug#72180] [PATCH 1/2] gnu: libplacebo: Update to 7.349.0 Mathieu Laparie
2024-07-18 17:58 ` [bug#72180] [PATCH 2/2] gnu: mpv: Update to 0.38.0, include sixel support Mathieu Laparie
[not found] ` <8734o58jh3.fsf@qq.com>
2024-07-19 15:55 ` [bug#72180] [PATCH 0/2] gnu: mpv: Update to 0.38.0 and " Z572 via Guix-patches via
2024-09-05 20:24 ` Nicolas Goaziou via Guix-patches via [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=878qw5q26o.fsf@nicolasgoaziou.fr \
--to=guix-patches@gnu.org \
--cc=72180-done@debbugs.gnu.org \
--cc=873216071@qq.com \
--cc=mail@nicolasgoaziou.fr \
--cc=matf@disr.it \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).