Review Request 119038: Allow loading KCMs from QT_PLUGIN_PATH subdirectories
Alexander Richardson
arichardson.kde at googlemail.com
Mon Jun 30 15:18:46 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119038/
-----------------------------------------------------------
Review request for KDE Frameworks.
Repository: kcmutils
Description
-------
Previously we would crash with an assertion inside libdbus-1 because of
additional / characters in the dbus path if X-KDE-Library in the
.desktop file pointed to a plugin inside a subdirectory of the plugin
path. We now just use the name of the library and strip all leading
directories from the name that is registered in DBus
This would allow moving all kcms into e.g. plugins/kcm5 which would allow
cleaning up the plugins/ directory
Diffs
-----
src/kcmoduleproxy.cpp 7596f36e1e26ce080634f3cc0f89e27280423f25
Diff: https://git.reviewboard.kde.org/r/119038/diff/
Testing
-------
- moved $KF5/lib64/plugins/kcm_clock.so to $KF5/lib64/plugins/kcm5/kcm_clock.so
- changed X-KDE-Library to "kcm5/kcm_clock" in $KF5/share/kservice5/clock.desktop
- ran kbuildsycoca5
"kcmshell5 clock" before this patch -> crash
"kcmshell5 clock" after this patch -> works as expected
Thanks,
Alexander Richardson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140630/1e3d36b4/attachment.html>
More information about the Kde-frameworks-devel
mailing list