GNOME Bugzilla – Bug 328228
upon startup gpm launches hibernate
Last modified: 2006-02-06 10:33:51 UTC
with the latest changes as soon as i login into gnome and gnome-power-manager is loaded it triggers hibernation.
Diego, can you try the patch from: http://bugzilla.gnome.org/attachment.cgi?id=57880&action=view specifically the change in src/gpm-manager.c: - /* less than critical, do action */ - if (percentage < critical_threshold) { + /* Just after unplugging remaining_time can be 0 because it could + * not be determined yet + */ + if (primary && discharging && remaining_time != 0 && + remaining_time < BATTERY_CRITICAL_REMAINING_TIME) { Thanks, Richard.
it seems to work.
I think Diego was seeing the bug I told you about. We're hibernating whenever any of the battery powered devices is critical
it might be, but my laptop, when that happend did have the battery in and was running on the AC power cable. I have also observed that since some time ago GPM doesn't hibernate the computer after the time specified in the gpm-preferences expires. Don't know if they are related in any way, though.
>I have also observed that since some time ago GPM doesn't hibernate the >computer after the time specified in the gpm-preferences expires. Don't know if >they are related in any way, though. Is this with a really recent gnome-screensaver? I'm having issues with the signals from g-s, I think I need to have a closer look. It works okay for Jon, so this might just be a config issue -- I'll have a play today. Richard.
yeap, gnome-screensaver from CVS
Diego, does the current g-p-m cvs fix the issue of hibernation on startup? If so, can you please close the bug, many thanks.
it is still there. Sorry.
Okay, I've committed another fix. Could you wait for anon-cvs to sync please, and then try again. You'll know it has synced when: 2006-01-25 Richard Hughes <richard@hughsie.com> * src/gpm-manager.c (power_battery_power_changed_cb): Only do action when discharging, trying to fix bug (#328228) is the top entry in the ChangeLog. Many thanks, Richard.
Created attachment 58124 [details] [review] test patch Also, if that doesn't work, could you try this patch please.
Diego, has this been fixed in CVS?
sorry. Yes it is fixed