From: Jose G Perez Taveras <josegpt27@gmail.com>
To: Christopher Baines <mail@cbaines.net>
Cc: 56111@debbugs.gnu.org
Subject: [bug#56111] [PATCH] gnu: playerctl: update to 2.4.1.
Date: Tue, 28 Jun 2022 09:19:41 -0400 [thread overview]
Message-ID: <878rphsz6a.fsf@gmail.com> (raw)
In-Reply-To: <87h74ep06p.fsf@gmail.com>
Hello,
I would like to know if there are any updates in regards to this patch?
--
Best,
josegpt
Jose G Perez Taveras <josegpt27@gmail.com> writes:
> Christopher Baines <mail@cbaines.net> writes:
>
>> [[PGP Signed Part:Undecided]]
>>
>> Jose G Perez Taveras <josegpt27@gmail.com> writes:
>>
>>> Hello,
>>>
>>> Here is an update to playerctl to 2.4.1 and fixed dependencies.
>>> It uses `dbus-launch' to start a new daemon.
>>>
>>> This is my first time sending a patch, I would be very happy if you
>>> could point me out to anything I should know.
>>
>> Hi Jose,
>>
>> Thanks for the patch.
>>
>>> + (propagated-inputs (list dbus))
>>
>> The addition of dbus as a propagated input was the only thing that stood
>> out to me.
>>
>> If playerctl does call out to dbus-launch, then patching the source to
>> refer to the specific /gnu/store/...-dbus/.../dbus-launch binary would
>> be the nicest approach, since that would avoid propagating dbus.
>>
>> Is it necessary to propagate the dbus package, and if so, what doesn't
>> work if you don't?
>>
>> Thanks,
>>
>> Chris
>>
>> [[End of PGP Signed Part]]
>
> Hello Christopher,
>
> I kept on testing the patch. As you mentioned, propagating dbus is not
> necessary for the package to work correctly.
>
> Thank you for the feedback.
>
> Best,
> josegpt
>
> Here's an updated version of the patch.
>
> ---
> gnu/packages/music.scm | 11 ++++++-----
> 1 file changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
> index 60cb176320..2d0d582e6c 100644
> --- a/gnu/packages/music.scm
> +++ b/gnu/packages/music.scm
> @@ -48,6 +48,7 @@
> ;;; Copyright © 2022 Remco van 't Veer <remco@remworks.net>
> ;;; Copyright © 2022 Maxim Cournoyer <maxim.cournoyer@gmail.com>
> ;;; Copyright © 2022 Wamm K. D. <jaft.r@outlook.com>
> +;;; Copyright © 2022 Jose G Perez Taveras <josegpt27@gmail.com>
> ;;;
> ;;; This file is part of GNU Guix.
> ;;;
> @@ -5562,7 +5563,7 @@ (define-public pragha
> (define-public playerctl
> (package
> (name "playerctl")
> - (version "2.2.1")
> + (version "2.4.1")
> (source (origin
> (method git-fetch)
> (uri (git-reference
> @@ -5571,14 +5572,14 @@ (define-public playerctl
> (file-name (git-file-name name version))
> (sha256
> (base32
> - "17hi33sw3663qz5v54bqqil31sgkrlxkb2l5bgqk87pac6x2wnbz"))))
> + "0ij065blj3h5v6iivvpmgh1095vicj1nc7hp1nhlhpqagd98l89s"))))
> (build-system meson-build-system)
> (arguments
> - `(#:configure-flags '("-Dintrospection=false" "-Dgtk-doc=false")))
> - (inputs (list python-pygobject))
> + `(#:configure-flags '("-Dgtk-doc=false")))
> (native-inputs
> `(("glib:bin" ,glib "bin")
> - ("pkg-config" ,pkg-config)))
> + ("pkg-config" ,pkg-config)
> + ("gobject-introspection" ,gobject-introspection)))
> (synopsis "Control MPRIS-supporting media player applications")
> (description
> "Playerctl is a command-line utility and library for controlling media
>
> base-commit: d13ffd1580ec2253b6e05fb6300c3629aee893e3
> --
> 2.36.1
next prev parent reply other threads:[~2022-06-28 13:33 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-06-20 17:05 [bug#56111] [PATCH] gnu: playerctl: update to 2.4.1 Jose G Perez Taveras
2022-06-21 11:29 ` Christopher Baines
2022-06-21 14:02 ` Jose G Perez Taveras
2022-06-28 13:19 ` Jose G Perez Taveras [this message]
2022-06-28 22:03 ` bug#56111: " Christopher Baines
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=878rphsz6a.fsf@gmail.com \
--to=josegpt27@gmail.com \
--cc=56111@debbugs.gnu.org \
--cc=mail@cbaines.net \
/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).