https://bugs.kde.org/show_bug.cgi?id=395925

--- Comment #13 from Henrik Fehlauer <rk...@lab12.net> ---
Thanks for the continued investigation, Duncan, that's really helpful. Awesome
solution, I'll create a Diff on Phabricator tonight.

Could you try one more thing, so we know a bit better whether the problem is a
real fix or more of a workaround for another problem? As you have plenty of
time on a slow/broken startup, please attach GDB to the process, interrupt
execution and get a backtrace:
$ gdb gwenview |& tee backtrace.log
$ r
- press Ctrl-C while Gwenview is stuck (but after loading debug symbols
finished)
$ thread apply all bt
- attach the backtrace to the bug

----

As for your patches, both are basically reverting the whole feature:
- First patch: You don't set HAVE_QTDBUS in CMakeLists.txt anymore, so the rest
of the MPRIS integration won't even get compiled.
- Second patch: You skip creating Mpris2Service, which would otherwise be the
main entry point for everything else of the MPRIS integration.

----

Regarding MPRIS: That's a DBus service provided by Gwenview
(org.mpris.MediaPlayer2.Gwenview), which other parts of the infrastructure can
call into because it uses a standardized interface (org.mpris.MediaPlayer2).
IOW it isn't a separate dependency to install. For example Plasma's mediaplayer
applet can list all services providing that interface and call the
forward/play/pause methods. Same thing for KDE Connect.

I think even with the patch we should investigate why your media keys are not
working:
- Can you see Gwenview's DBus service in "qdbusviewer"?
- Can you change to the next picture with "dbus-send --session
--dest=org.mpris.MediaPlayer2.Gwenview --type=method_call
/org/mpris/MediaPlayer2 org.mpris.MediaPlayer2.Player.Next"?
- Can you see other services in "qdbusviewer" implementing the interface, e.g.
when running VLC?
- Can you control VLC with Plasma's mediaplayer applet?
- What does your global shortcut configuration in Systemsettings (Media
Controller component) look like?

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to