[Pkg-acpi-devel] Bug#697246: Bug#697246: closed by Michael Meskes <meskes at debian.org> (Re: Bug#697246: acpi-support-base: powerbtn-acpi-support.sh checks incorrect file for backwards compatibility)

[Yury Sobolev]

The script should "do the right thing." The bug may not be an issue on a
default install, but only by sheer luck. Upowerd should be given the chance
to catch and process the power event. At the moment upowerd works correctly
only in the presence of a display manager. If the display manager is
removed, and the user chooses to run a startx session from the terminal,
upower will not function correctly. There is no dependency listed on the
upower package for a display manager.

Would you like me to open this as a bug against upower? It is clearly
something that should be fixed here, but it affects the upower package.

The change required is also very small. Only a check for upower needs to be
made. This would not adversely affect behavior on any "standard" Debian
system, but it would handle more corner cases correctly.

-Yury
 On Jan 16, 2013 9:21 AM, "Michael Meskes" <meskes at debian.org> wrote:

> > > CheckUPolicy checks if upowerd is running. If upowerd is running, this
> > > means there is probably some freedesktop  compliant desktop environment
> > > running. It should be given the option to respond to the power button
> > > event. For this reason I believe CheckUPolicy should be added to the
> script.
>
> The keyword here is "probably". Why should we check for upowerd and just
> guess
> that there may be a power manager running for the desktop when we can
> check for
> that power manager instead and know or sure?
>
> Michael
> --
> Michael Meskes
> Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
> Michael at BorussiaFan dot De, Meskes at (Debian|Postgresql) dot Org
> Jabber: michael.meskes at gmail dot com
> VfL Borussia! Força Barça! Go SF 49ers! Use Debian GNU/Linux, PostgreSQL
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/pkg-acpi-devel/attachments/20130116/0f318648/attachment.html>