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 70710 - Cannot communicate Logout/Shut Down/Reboot actions to session manager
Cannot communicate Logout/Shut Down/Reboot actions to session manager
Status: RESOLVED DUPLICATE of bug 92277
Product: gnome-session
Classification: Core
Component: gnome-session
1.5.x
Other All
: Normal enhancement
: future
Assigned To: Session Maintainers
Session Maintainers
: 61539 72602 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2002-02-06 16:04 UTC by keelin.boyle
Modified: 2005-01-06 10:33 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement


Attachments
Debian patch (2.28 KB, patch)
2004-09-09 06:12 UTC, Vincent Untz
none Details | Review

Description keelin.boyle 2002-02-06 16:04:14 UTC
To Reproduce:

1) Start panel , gnome-panel-2 &.
2) Choose Actions -> Logout.

Logout Dialog missing Actions menu box, containing radio button options
Logout, ShutDown, & Reboot.

On Linux Shutdown resides in /sbin and reboot in /usr/bin, On Solaris
reboot & shutdown reside in /usr/sbin, perhaps this they are not being
picked up on logout.
Comment 1 Roy-Magne Mo 2002-02-06 16:28:17 UTC
tagging as gnome2
Comment 2 Luis Villa 2002-02-06 22:53:25 UTC
Seth: wasn't it the decision of the usability team that these options
were confusing? Or am I the confused one? :) 
Comment 3 Seth Nickell 2002-02-11 11:26:27 UTC
These are supposed to be split into seperate menu items, but this
hasn't happened yet. We definitely need to fix this for GNOME2 but its
probably a seperate bug.
Comment 4 Seth Nickell 2002-02-21 21:32:58 UTC
We should use "Restart" rather than "Reboot". 

It would be worth considering "Turn Off" rather than "Shutdown".

If you go through with using menus, I would recommend:

Actions->
  Run...
  Lock Screen
  ____________
  Restart
  Shutdown
  ____________
  Logout
Comment 5 Arvind S N 2002-02-28 14:15:46 UTC
can we go ahead with the changes recommended by nickel ?
Comment 6 Luis Villa 2002-02-28 14:32:55 UTC
Arvind: you should go ahead with a patch to fix snickell's problem,
but you should also attach (as a separate patch!) something to fix the
problem found by keelin boyle. Really, these should be treated as two
separate bugs; I think I misunderstood the problem when it was first
reported.
Comment 7 Seth Nickell 2002-02-28 22:54:55 UTC
Louie, I think the problem is that in GNOME 2 we don't have an actions
box yet, its not a Solaris specific issue (at least, I see the same
thing as Kevin on x86 Debian). This is a case where a GNOME 1 feature
has not yet been implemented.

Thus a solution to Kevin's problem would be to follow my suggestion,
or we could simply duplicate what was in GNOME 1 (though I think this
would be a waste of effort, as putting multiple menu items in is
better, and is basically the same amount of work).
Comment 8 Luis Villa 2002-03-02 21:22:05 UTC
Yeah, should have marked this 'all'. Fixing that now.
Comment 9 Luis Villa 2002-03-14 17:53:02 UTC
Marking up to Urgent; this really has to be fixed, as it is extremely
basic to the entire system.
Comment 10 Seth Nickell 2002-03-15 04:27:27 UTC
Changing the bug title.
Comment 11 Seth Nickell 2002-03-15 04:28:28 UTC
*** Bug 61539 has been marked as a duplicate of this bug. ***
Comment 12 Seth Nickell 2002-03-15 04:29:05 UTC
*** Bug 72602 has been marked as a duplicate of this bug. ***
Comment 13 Gregory Merchan 2002-03-15 05:16:40 UTC
I think people have been giving Windows the three-finger salute long
enough for "Reboot" to have seeped into the culture at large.

Changing this to "Restart" would be confusing because you can restart
many different things, including the session. "Restart this computer"
would remove that ambiguity, but why bother. Everyone and his
grandmother knows "Reboot".

"Shutdown" is plainly wrong because it's a noun. The correct verb
phrase is "Shut Down" as is shown in Nils' panel menu proposal.
That is also a common phrase outside of computing.

"Logout" is also wrong because it is a noun. Again, Nils' proposal
gets this right.

http://www242.pair.com/nilsp/nils/panel/menus.html
(readable)

http://developer.gnome.org/projects/gup/proposals/menus.html
(all scrunched up)
Comment 14 Seth Nickell 2002-03-15 11:24:20 UTC
I'm not sure I agree that we should use reboot. Both MacOS and Windows
use restart, it feels like a less archane word, and I don't really see
where its ambiguous. I don't think anyone is going to confuse restart
with "restart your session". Esp. if sessions get buried like I hope
they do ;-)

I agree with "Shut Down" and "Log Out"
Comment 15 jacob berkman 2002-03-15 18:06:25 UTC
btw the reason they show up in the linux logout dialog and not solaris
is that it's a (red hat) linux specific hack which allows non-root
users to shutdown the machine if they are logged into the console.

Comment 16 Owen Taylor 2002-03-15 21:11:20 UTC
It's actually not a hack at all, but a clean solution to 
allow users at the console to shut down the machine.
Comment 17 Arvind S N 2002-03-17 05:26:09 UTC
Currently there is no way of communicating to the session 
manager from the panel which action (logout,shutdown,restart) 
was selected. Because the call made from the client,
'SmcRequestSaveYourself' does not allow this additional
information to be passed. This is due to the limitations of XSMP.
Hence we may have to retain the existing functionality.
As Owen suggested this probably will have to be looked 
in the future as UI improvement.

Please also refer to mail archives for thoughts exchanged on this bug.

http://mail.gnome.org/archives/desktop-devel-list/2002-March/msg00445.html
Comment 18 Mark McLoughlin 2002-03-19 09:42:42 UTC
Okay, so this now looks like its a feature request for the session
manager. Adjusting everything accordingly
Comment 19 Mark McLoughlin 2002-03-19 09:45:59 UTC
Okay, let me clarify that :-)

This requires extending the session manager to allow the action
required for the SaveYourself to be communicated ....

This is definetly something for the future .....
Comment 20 Gregory Merchan 2002-03-19 11:37:59 UTC
Does this really need a change to the session management protocol?

xsm responds to SIGTERM with a fast shutdown, and to SIGUSR1
as a checkpoint request.

The gnome session manager could do this and in addition process
SIGUSR2 as a requested shutdown or reboot.
Comment 21 Brian Nitz 2002-12-16 17:20:19 UTC
This works as it should in the Solaris GNOME build fcs_07.  Logout
presents these options only if the user is logged in as root.  
This bug can be closed.
Comment 22 Mark McLoughlin 2002-12-16 18:09:04 UTC
Brian: this bug somehow got from "broken on Solaris" to "we need some
way of putting these options in the menus". Weird.

Good to hear that problem is fixed in any case.
Comment 23 Murray Cumming 2003-07-09 08:28:22 UTC
So is there still a bug here?
Comment 24 Mark Finlay 2003-09-08 18:24:31 UTC
I think that there is still a usability issue here:
"Shut Down" "Restart" and "Log out" should all be menu items
in the Actions menu instead of being in one dialog. And if it
isn't possible for the user to shutdown/reboot then only the
"Log Out" option should be there.
Comment 25 Calum Benson 2003-11-27 13:33:27 UTC
The remaining usability issue looks like a dup of #92277 to me, I'd
suggest closing this one as a dup of that since the
originally-reported problem here seems to be resolved.  

(If there are any other outstanding issues here I've missed, they
should probably be filed as new bugs...)
Comment 26 Mark Finlay 2003-11-27 15:56:39 UTC
I wasn't aware of #92277 and since I'm the one who has been harping on
about this, I'm going to close this.

Thanks Calum

*** This bug has been marked as a duplicate of 92277 ***
Comment 27 Mark McLoughlin 2003-11-27 16:08:02 UTC
This bug is not fixed.

You ask the session manager to log out by sending a
SaveYourselfRequest to the session manager with shutdown == TRUE.

There is no way of telling the session manager that you want to reboot
or shutdown the computer after logging out.

XSMP sucks in other words.
Comment 28 Vincent Untz 2004-09-09 06:12:07 UTC
Created attachment 31431 [details] [review]
Debian patch

Debian has a patch to use some fancy new stuff in GDM 2.6. Could this be used?
Comment 29 Mark McLoughlin 2004-09-09 08:57:08 UTC
Looks like it might be worth considering, I think it even probably makes more
sense to have GDM do the shutting down rather than using consolehelper directly.
Comment 30 Mark McLoughlin 2005-01-06 10:33:00 UTC
Okay, I'm going to re-mark this as a dup of #92277 because it looks like we're
going to go with a solution which involves asking GDM to shutdown/reboot. There
will be some gnome-session changes involved, but lets figure those out in bug
#92277.

*** This bug has been marked as a duplicate of 92277 ***