Review Request 122589: [dataengine/mpris2] Drop workaround for VLC

Martin Gräßlin mgraesslin at kde.org
Wed Feb 18 08:14:41 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122589/
-----------------------------------------------------------

(Updated Feb. 18, 2015, 8:14 a.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma and Martin Klapetek.


Repository: plasma-workspace


Description
-------

If there was a vlc engine it got skipped due to a bug in VLC. This
bug seems to be fixed as of VLC 2.1.5 which contains the following
item in the changelog:
"dbus: Upgrade to an mpris2 compliant interface"

Given that the workaround breaks with the fixed VLC (no media control
at all), it's better to drop it and brake VLC installations which
haven't updated for more than half a year.


Diffs
-----

  dataengines/mpris2/mpris2engine.cpp 42ca565d06b240712fbc75cfdaf7b1cc03404796 

Diff: https://git.reviewboard.kde.org/r/122589/diff/


Testing
-------

media controller created if VLC is running when restarting plasmashell.


Thanks,

Martin Gräßlin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150218/79fac3d7/attachment.html>


More information about the Plasma-devel mailing list