audacious-media-player / audacious

A lightweight and versatile audio player
https://audacious-media-player.org
Other
871 stars 116 forks source link

gnomeshortcuts fix for bug 765 does not work #947

Closed Audacious-Bot closed 6 months ago

Audacious-Bot commented 6 months ago

Author Name: Michael Schwendt Original Redmine Issue: https://redmine.audacious-media-player.org/issues/808 Original Date: 2018-07-14


In reply to:

https://github.com/audacious-media-player/audacious-plugins/commit/2002c9b2f4fa7348ab66e4dff889da25f646c813#diff-b54d10e5a554e1f834b63a9f19d5cd43

A few months ago I've pointed out that the commit does not work with GNOME 3.26.2 (Fedora 27). It also doesn't work with GNOME 3.28.1 (Fedora 28).

As a workaround for Audacious 3.9 and GNOME >= 3.26 I went with a single-line fix as incorporated into the following patch. The top part of that patch is for something else, avoiding dbus activation of the g-s-d daemons on non-GNOME systems: https://src.fedoraproject.org/rpms/audacious-plugins/blob/master/f/audacious-plugins-3.9-gnomeshortcuts.patch

So, by default, Audacious 3.10-beta1 gnomeshortcuts plugin does not work yet:

(audacious:26570): WARNING : 01:34:59.140: Could not grab media player keys: The name org.gnome.SettingsDaemon was not provided by any .service files WARNING plugin-init.cc:102 [start_plugin]: GNOME Shortcuts failed to start.

Audacious-Bot commented 6 months ago

Original Redmine Comment Author Name: John Lindgren Original Date: 2018-07-15T04:34:53Z


I'm guessing this breaks compatibility with GNOME < 3.26; is that true? If so, that's okay with me, but we can strip out some of the fallback code in that case.

I don't use any recent version of GNOME (and have zero interest in doing so) which is why I haven't followed up on this until now. But I would be happy to merge a patch or pull request.

Audacious-Bot commented 6 months ago

Original Redmine Comment Author Name: John Lindgren Original Date: 2018-07-30T15:10:51Z


Fixed: https://github.com/audacious-media-player/audacious-plugins/commit/3dc8222f59bc8a2352fc0bac54e34a22adc6e122