From kde-core-devel Tue Jun 01 23:36:18 2010 From: "Aaron Seigo" Date: Tue, 01 Jun 2010 23:36:18 +0000 To: kde-core-devel Subject: Review Request: DBus spec compliant service names Message-Id: <20100601233618.13719.43129 () localhost> X-MARC-Message: https://marc.info/?l=kde-core-devel&m=127543544031048 ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/4203/ ----------------------------------------------------------- Review request for kdelibs. Summary ------- According to the DBus spec ( http://dbus.freedesktop.org/doc/dbus-specification.html ), services on the bus may only have [a-zA-Z0-9_] in them, must start with a letter and be no longer than 255 characters. For apps with illegal characters in them, such as plasma-desktop and plasma-netbook, this causes warnings to be printed that give concern to some users. It also means that apps that are strict about these things may not show these services. Right now dbus-daemon does not enforce this requirement, so it's a theoretical-in-practice problem at this point as far as I can tell. This patch makes the names compliant ... but will spark a lot of changes for us in workspace (all those calls from control panels, plasmoids, etc. to org.kde.plasma-desktop for instance) as well as patches to apps such as kquitapp (so that it would turn plasma-desktop into plasma_desktop internally). It would also break any scripts that assume names like "plasma-desktop". I'm not sure it's worth doing for this reason. It was an easy patch to do, however, so I figured I should at least float it past the rest of the kdelibs developers. Thoughts? This addresses bug 240358. https://bugs.kde.org/show_bug.cgi?id=240358 Diffs ----- /trunk/KDE/kdelibs/kdeui/kernel/kuniqueapplication.cpp 1130620 /trunk/KDE/kdelibs/kdeui/kernel/kapplication.cpp 1130620 Diff: http://reviewboard.kde.org/r/4203/diff Testing ------- Thanks, Aaron