> On July 1, 2014, 12:29 p.m., David Faure wrote: > > src/kcmoduleproxy.cpp, line 95 > > <https://git.reviewboard.kde.org/r/119038/diff/1/?file=285705#file285705line95> > > > > technically this could lead to conflicts (if same filename in two > > different subdirs). > > > > Why not turn '/' into '_' ? > > > > Of course this raises the question of compatibility ... which makes me > > wonder who uses this dbus stuff in the first place. Any idea?
>From looking at the code and qdbusviewer it seems all that it exposes is a >method applicationName in internal/KSettingsWidget/kcm_clock which returns the >same thing as the property in MainApplication/ >org.qtproject.Qt.QGuiApplication applicationDisplayName, so I guess this could >actually be dropped? - Alexander ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119038/#review61363 ----------------------------------------------------------- On June 30, 2014, 5:18 p.m., Alexander Richardson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/119038/ > ----------------------------------------------------------- > > (Updated June 30, 2014, 5:18 p.m.) > > > 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 > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
