Bug 444256

Summary:

can_suspend does not affect suspend option in login screen

Product:

[Fedora] Fedora

Reporter:

Christopher Tubbs <ctubbsii>

Component:

gdm

Assignee:

jmccann

Status:

CLOSED WORKSFORME

QA Contact:

Fedora Extras Quality Assurance <extras-qa>

Severity:

low

Docs Contact:

Priority:

low

Version:

rawhide

CC:

cschalle, rstrode

Target Milestone:

---

Keywords:

Reopened

Target Release:

---

Hardware:

All

OS:

Linux

Whiteboard:

Fixed In Version:

Doc Type:

Bug Fix

Doc Text:

Story Points:

---

Clone Of:

Environment:

Last Closed:

2008-05-10 01:44:53 UTC

Type:

---

Regression:

---

Mount Type:

---

Documentation:

---

CRM:

Verified Versions:

Category:

---

oVirt Team:

---

RHEL 7.3 requirements from Atomic Host:

Cloudforms Team:

---

Target Upstream Version:

Embargoed:

Attachments:

Description	Flags
patch	none

Description Christopher Tubbs 2008-04-26 04:21:10 UTC

Description of problem:
gdm login window does not properly respect the global mandatory disabling of
suspend through gconf-editor

Version-Release number of selected component (if applicable):
Fedora 9 Pre-Release
gdm-2.21.10-0.2008.04.18.2.fc9.i386

How reproducible:
100%

Steps to Reproduce:
1. sudo gconf-editor
2. go to /apps/gnome-power-manager/general
3. un-check can_suspend
4. right-click can_suspend, select mandatory
  
Actual results:
suspend remains an option in the gdm login screen

Expected results:
suspend option should be removed in the login screen

Additional info:
none

Comment 1 Matthias Clasen 2008-04-26 23:39:47 UTC

Created attachment 303880 [details]
patch

Here is a patch

Comment 2 Christopher Tubbs 2008-04-28 00:36:58 UTC

How long before we can expect this patch to show up in the fedora updates? Also,
does can_hibernate have the same issue?

Comment 3 Matthias Clasen 2008-04-28 14:02:41 UTC

There is no hibernate button on the login window, so there is no can_hibernate
issue...

Comment 4 Matthias Clasen 2008-04-28 14:12:00 UTC

I have committed the patch upstream, so it will appear in Fedora with the next
upstream update we do.

Comment 5 Matthias Clasen 2008-04-29 00:01:07 UTC

* Mon Apr 28 2008 Ray Strode <rstrode> - 1:2.21.10-0.2008.04.28.1
- a11y improvements
- make "Suspend" desensitize properly when not-available
- make resize animation faster
- user switcher fixes

Comment 6 Christopher Tubbs 2008-05-09 03:37:21 UTC

The current fix disables the restart button when suspend is not available. That
shouldn't happen. The restart button should still be available, regardless.

Comment 7 Matthias Clasen 2008-05-09 17:07:04 UTC

I don't see how my patch can have that effect.

Comment 8 Christopher Tubbs 2008-05-10 01:44:53 UTC

My mistake. The patch works correctly.