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 421912 - Integrate session management in the process list
Integrate session management in the process list
Status: RESOLVED OBSOLETE
Product: system-monitor
Classification: Core
Component: process list
unspecified
Other Linux
: Normal enhancement
: ---
Assigned To: System-monitor maintainers
System-monitor maintainers
: 102585 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2007-03-23 15:07 UTC by Vincent Untz
Modified: 2013-02-13 21:46 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
a patch (28.68 KB, patch)
2008-08-07 18:45 UTC, Matthias Clasen
none Details | Review
Updated process session info patch (32.06 KB, patch)
2013-02-13 02:46 UTC, Robert Roth
none Details | Review

Description Vincent Untz 2007-03-23 15:07:00 UTC
I thought I already opened such a bug, but maybe this was only an IRC discussion.

Right now, we have a list of processes that can talk with the session manager in gnome-session-properties, and we a have a list of processes in the system monitor.

I believe it makes sense to get rid of the list of processes in gnome-session-properties, and move the useful features to the system monitor.
Comment 1 Benoît Dejean 2007-03-27 14:05:51 UTC
And what are the useful features ?
Comment 2 Vincent Untz 2007-06-21 13:29:00 UTC
I'd say changing the "style" can be useful (for example, to stop an application being respawned by gnome-session), and removing an application from the session can be useful too. It doesn't kill the process, but sends it a "Die" session message (I believe).

Order is probably useless.

It might also be interesting to see which programs are session-aware. Because those programs are the ones which are likely to automatically save their state.
Comment 3 Matthias Clasen 2008-08-07 18:45:15 UTC
Created attachment 116086 [details] [review]
a patch

Here is a patch that adds a "Session" column, which displays up to two icons in the same style as the session capplet used to do. The first icon indicates the 
status of the session client, the second icon indicates the restart style.

Currently there is no way to modify the restart style, since gnome-session doesn't export a dbus interface for doing so.

The patch isn't quite perfect yet, it may be necessary to connect a signal handler to GsmSessionManager:client-{added,removed} and update the session
information in the tree model from the handler.
Comment 4 Lucas Rocha 2008-08-13 20:01:15 UTC
The patch looks pretty for a start. Honestly, from user perspective, I don't see much value on being able to modidy restart style. 

It would be nice to hear from Benoît about it.
Comment 5 Benoît Dejean 2008-08-14 19:48:30 UTC
I'm a bit concerned about how fast it is to retrieve info from DBUS.
Comment 6 Matthias Clasen 2008-08-14 19:58:21 UTC
The way my patch does it is to only get the full list of clients once, and then listen for add/remove signals from the session manager. Refreshing the process list is done off the locally cached information.
Comment 7 Teppo Turtiainen 2009-03-20 13:35:34 UTC
I don't think session management belongs in a system monitor. The purpose of a system monitor is to display information about about the system, not to handle settings. The similarities between session management settings and the system monitor are cosmetic, not conceptual. Not in a million years would I think to look for session managements settings in the system monitor if I didn't know that they are there.
Comment 8 Vincent Untz 2009-03-20 14:59:54 UTC
(In reply to comment #7)
> I don't think session management belongs in a system monitor. The purpose of a
> system monitor is to display information about about the system, not to handle
> settings. The similarities between session management settings and the system
> monitor are cosmetic, not conceptual.

Except that the status of an application in the current session is not a setting.
Comment 9 Jeremy Nickurak 2009-03-20 15:13:34 UTC
It is when you _set_ whether the application is a persistent part of the session
Comment 10 Vincent Untz 2009-03-20 15:21:47 UTC
Really, it's just changing the interaction with the session manager for the current session (and it won't survive on next login). So it's not really a setting.

I see this as a similar feature as "kill this application". 
Comment 11 Vincent Untz 2009-06-21 12:42:31 UTC
*** Bug 102585 has been marked as a duplicate of this bug. ***
Comment 12 André Klapper 2012-02-26 10:46:35 UTC
[Adding missing "QA Contact" entry so system monitor bug report changes can still be watched via the "Users to watch" list on https://bugzilla.gnome.org/userprefs.cgi?tab=email when the assignee is changed to an individual.]
Comment 13 Robert Roth 2013-02-13 02:46:23 UTC
Created attachment 235860 [details] [review]
Updated process session info patch
Comment 14 Robert Roth 2013-02-13 02:50:22 UTC
Matthias, Vincent: is this still relevant? I have updated the patch to apply and work on the latest trunk, however I am getting some strange results, e.g. gedit and firefox appearing as registered gsm clients. Is there any other way to find the clients registered in gnome-session-manager and their restart styles? I have tried checking gnome-session-properties, but I only have there 3 items, and none of them appear registered in this patched system monitor.
Comment 15 Vincent Untz 2013-02-13 17:38:18 UTC
I'm not totally sure we still want this. Nearly no app should play with the restart style anymore...
Comment 16 Robert Roth 2013-02-13 19:24:44 UTC
OK, based on the reporter's comment 15 I guess this will not be useful anymore, will not add real value, so I am marking this as Obsolete. Please let me know if you disagree and see some real use-case when process session property management in system monitor can be useful.
Comment 17 Matthias Clasen 2013-02-13 19:55:58 UTC
I agree with Vincent: not needed and not wanted
Comment 18 Jeremy Nickurak 2013-02-13 21:46:58 UTC
Would the opposite be better? Integrating the process list into the session manager, ie, in gnome-session-properties? Being able to take something which is currently running, and add it to the saved session...