[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Review Request: DBus spec compliant service names
From:       "Aaron Seigo" <aseigo () kde ! org>
Date:       2010-06-01 23:36:18
Message-ID: 20100601233618.13719.43129 () localhost
[Download RAW message or body]


-----------------------------------------------------------
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


[prev in list] [next in list] [prev in thread] [next in thread]