! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
upower is no longer doing suspend/hibernate, switch to logind
Status:
RESOLVED: FIXED
Product:
Xfce4-power-manager
Component:
General

Comments

Comment 1 Mikhail Efremov 2013-04-09 17:14:42 CEST
Created attachment 4994 
Add-systemd-logind-support-for-suspend-hibernate.patch

Patch for systemd-logind support is attached.
Comment 2 Kevin Fenzi 2013-04-13 20:12:19 CEST
Works fine here. Thanks!
Comment 3 Guido Berhoerster 2013-10-23 17:41:26 CEST
Created attachment 5196 
systemd suspend/hibernate support

xfce4-session treats "challenge" responses as true, the attached version of the patch does the same here for consistency
Comment 5 Guido Berhoerster 2013-10-24 21:46:20 CEST
(In reply to Yves-Alexis Perez from comment #4)
> What's the status on those patches, especially compared to:
> 
> https://build.opensuse.org/package/view_file/X11:xfce/xfce4-power-manager/
> xfce4-power-manager-systemd-inhibit.patch?expand=1
> https://build.opensuse.org/package/view_file/X11:xfce/xfce4-power-manager/
> xfce4-power-manager-systemd-shutdown-reboot-support.patch?expand=1
> https://build.opensuse.org/package/view_file/X11:xfce/xfce4-power-manager/
> xfce4-power-manager-systemd-suspend-hibernate-suport.patch?expand=1
> 
> ?

What do you mean by status, the three patches do different things and they are attached to their respective bugs here on bugzilla.xfce.org. https://build.opensuse.org/package/view_file/X11:xfce/xfce4-power-manager/xfce4-power-manager-systemd-suspend-hibernate-suport.patch?expand=1 is the patch that I attached to comment #3.
Comment 6 Yves-Alexis Perez editbugs 2013-10-24 22:04:46 CEST
(In reply to Guido Berhoerster from comment #5)
> (In reply to Yves-Alexis Perez from comment #4)
> > What's the status on those patches, especially compared to:
> > 
> > https://build.opensuse.org/package/view_file/X11:xfce/xfce4-power-manager/
> > xfce4-power-manager-systemd-inhibit.patch?expand=1
> > https://build.opensuse.org/package/view_file/X11:xfce/xfce4-power-manager/
> > xfce4-power-manager-systemd-shutdown-reboot-support.patch?expand=1
> > https://build.opensuse.org/package/view_file/X11:xfce/xfce4-power-manager/
> > xfce4-power-manager-systemd-suspend-hibernate-suport.patch?expand=1
> > 
> > ?
> 
> What do you mean by status, the three patches do different things and they
> are attached to their respective bugs here on bugzilla.xfce.org.
> https://build.opensuse.org/package/view_file/X11:xfce/xfce4-power-manager/
> xfce4-power-manager-systemd-suspend-hibernate-suport.patch?expand=1 is the
> patch that I attached to comment #3.

I found bug #9335 for the inhibit patch and bug #10167 for the shutdown/reboot. Is that correct?

By status, I mean I'm not sure what's the latest version I should use (for the Debian packages). It seems the one on this bug was updated yesterday, not sure if the other need an update etc?

Also, I'm adding Nick to CC: since Aliov is completely MIA these days, so maybe he can comment on the patches and commit them if they look sane.
Comment 7 Guido Berhoerster 2013-10-24 23:47:03 CEST
(In reply to Yves-Alexis Perez from comment #6)
> I found bug #9335 for the inhibit patch and bug #10167 for the
> shutdown/reboot. Is that correct?

Correct, see also bug #9952 for the corresponding xfce4-session patch.

> By status, I mean I'm not sure what's the latest version I should use (for
> the Debian packages). It seems the one on this bug was updated yesterday,
> not sure if the other need an update etc?

The patch on bug #9952 should be fine.

> Also, I'm adding Nick to CC: since Aliov is completely MIA these days, so
> maybe he can comment on the patches and commit them if they look sane.

IMO systemd support should be detected at runtime like in xfce4-session rather than a compile-time option. What will Debian/Ubuntu do when upower drops suspend/hibernate?
Comment 8 Yves-Alexis Perez editbugs 2013-10-25 07:22:28 CEST
(In reply to Guido Berhoerster from comment #7)
> (In reply to Yves-Alexis Perez from comment #6)
> > I found bug #9335 for the inhibit patch and bug #10167 for the
> > shutdown/reboot. Is that correct?
> 
> Correct, see also bug #9952 for the corresponding xfce4-session patch.
> 
> > By status, I mean I'm not sure what's the latest version I should use (for
> > the Debian packages). It seems the one on this bug was updated yesterday,
> > not sure if the other need an update etc?
> 
> The patch on bug #9952 should be fine.

Thanks.
> 
> > Also, I'm adding Nick to CC: since Aliov is completely MIA these days, so
> > maybe he can comment on the patches and commit them if they look sane.
> 
> IMO systemd support should be detected at runtime like in xfce4-session
> rather than a compile-time option. What will Debian/Ubuntu do when upower
> drops suspend/hibernate?

Well, right now, it seems we'll lose suspend/hibernate support in Xfce, unfortunately…
Comment 9 Eric Koegel editbugs 2014-04-12 17:40:57 CEST
Created attachment 5428 
Add support for logind suspend/hibernate (runtime checks)

This patch does runtime checks to see if we can use logind for
suspend/resume same as poweroff/reboot. It will continue to work
fine without it.

Bug #9963

Reported by:
Kevin Fenzi
Reported on: 2013-03-29
Last modified on: 2014-04-13

People

Assignee:
Ali Abdallah
CC List:
11 users

Version

Attachments

Additional information