Comment 39 for bug 1178373

Revision history for this message
In , Guido Berhoerster (gber) wrote :

(In reply to Eric Koegel from comment #16)
> (In reply to Guido Berhoerster from comment #15)
> > (In reply to Eric Koegel from comment #14)
> > > Created attachment 5498 [details]
> > > Suspend/hibernate upower 0.99 (no logind)
> > >
> > > Since everyone else is doing the logind piece, I wrote a small patch so
> > > xfce4-session will work with upower 0.99 but without logind.
> >
> > Although I think it would be a good thing not to rely on logind, please do
> > not extend this crude and ugly sudo hack that BSD and Solaris users have to
> > put up with, but replace xfsm-shutdown-helper with a small DBus service and
> > use PolicyKit to authorize access to shutdown/suspend methods.
> > xfsm-shutdown-helper should just go away, PolicyKit is nowadays available
> > even on the BSDs and Solaris.
>
> Indeed, I can look into doing that. I didn't know how long it would take to
> get a new project accepted by distros, especially something running as root.
> In the meantime, extending shutdown-helper will work until I (or someone)
> makes a daemon for that.

I don't think that it's big deal, it will not be used by most major Linux distros anyway since they are all systemd-based and there is already precedence with xfpm-backlight-helper which gets executed as root through polkit.
Speaking of, could you just execute xfpm-pm-helper through pkexec instead of sudo in the same way it is done with xfpm-backlight-helper?
xfpm-pm-helper should probably disabled by default on Linux since distros either carry upower < 0.99 or they have upower >= 0.99 and systemd.
Finally it'd be nice to install all of these helpers under $(libexecdir) instead of $(bindir) since they're not meant to be executed directly by the user and clutter $PATH. Thanks in advance.