After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 686546 - make the session-name property official
make the session-name property official
Status: RESOLVED FIXED
Product: gnome-session
Classification: Core
Component: gnome-session
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Session Maintainers
Session Maintainers
Depends on:
Blocks: 686556
 
 
Reported: 2012-10-20 21:42 UTC by Matthias Clasen
Modified: 2012-10-22 14:25 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch (556 bytes, patch)
2012-10-20 21:42 UTC, Matthias Clasen 		committed Details | Review

Description Matthias Clasen 2012-10-20 21:42:50 UTC 
Created attachment 226909 [details] [review]
patch

gnome-settings-daemon nowadays calls org.freedesktop.Properties.Get for the "session-name" property - and it works, despite the property not being officially in the interface !

The other thing is that I am seeing lots of GetAll calls for org.gnome.SessionManager in a bustle log - basically for every app which creates a proxy using gdbus. These calls all return an error currently, because the interface doesn't have any properties in it.

Here is a patch which avoids both of these issues by making the session-name property official.
Comment 1 Matthias Clasen 2012-10-20 22:03:49 UTC 
The property name in the xml should be SessionName, actually
Comment 2 Ray Strode [halfline] 2012-10-22 14:25:40 UTC 
Thanks.