[KCharSelect] Unbreak KRun powered startup
Review Request #123074 - Created March 19, 2015 and submitted
Due to how desktop file's set up, we need to register to DBus, otherwise klauncher isn't happy.
Adding Albert due to new dep
The bug is no more.
I have to admit that I am unable to review these changes, because I have no idea what the proposed changes do and I do not run Plasma 5 yet. The applications start fine on Plasma 4.
On the other hand, if this is needed to fix the bug, then the added dependency should not block the review. Actually, with the splitted frameworks, it would be a bit ugly if we could not use all of the old kdelibs4 features, just because they happen to be in a framework which wasn't used yet.
Additionally, https://bugs.kde.org/show_bug.cgi?id=344896#c4 mentions several other apps, and says "many games", so either all of them need to be fixed, or we have to check why Plasma 5 works differently than the old Plasma and fix it there.
No, it's a mismatch between the .desktop file and the application code.
The desktop file says "I will register to DBus", and then the application doesn't -> klauncher waits for a bit, then errors.
These apps indeed do not really need to register to DBus, so I would suggest to just remove the DBus-related keys from their desktop files.
This was the way to 'start an app then talk to dbus to it' in the old days (which can be useful for a few of them), but these days dbus-autostart covers that even better. This is why I think we should move away from this (kde-specific) X-DBus-StartupType stuff.