Can I get a reply on this? Adding a CheckUPolicy clause to powerbtn-acpi-support.sh seems like a logical thing to do. I have explained my reasoning in the last email. Thank you.
-Yury On Thu, Jan 3, 2013 at 3:09 PM, Yury Sobolev <yury.sobo...@gmail.com> 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. > > -Yury > On Jan 3, 2013 7:51 AM, "Debian Bug Tracking System" < > ow...@bugs.debian.org> wrote: > >> This is an automatic notification regarding your Bug report >> which was filed against the acpi-support-base package: >> >> #697246: acpi-support-base: powerbtn-acpi-support.sh checks incorrect >> file for backwards compatibility >> >> It has been closed by Michael Meskes <mes...@debian.org>. >> >> Their explanation is attached below along with your original report. >> If this explanation is unsatisfactory and you have not received a >> better one in a separate message then please contact Michael Meskes < >> mes...@debian.org> by >> replying to this email. >> >> >> -- >> 697246: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=697246 >> Debian Bug Tracking System >> Contact ow...@bugs.debian.org with problems >> >> >> ---------- Forwarded message ---------- >> From: Michael Meskes <mes...@debian.org> >> To: Yury Arkady Sobolev <yury.sobo...@gmail.com>, >> 697246-d...@bugs.debian.org >> Cc: >> Date: Thu, 3 Jan 2013 13:47:25 +0100 >> Subject: Re: [Pkg-acpi-devel] Bug#697246: acpi-support-base: >> powerbtn-acpi-support.sh checks incorrect file for backwards compatibility >> On Wed, Jan 02, 2013 at 07:41:22PM -0500, Yury Arkady Sobolev wrote: >> > The powerbtn-acpi-support.sh script in the acpi-support-base package >> > will look for previous configuration scripts. Namely, it looks for >> > powerbtn.sh and powerbtn.sh.dpkg-back. However, it does not check for >> > power.sh. As I understand it, this script has not been used in a long >> > time, but it is still present on some systems, while powerbtn.sh is not. >> >> No, your understanding is not correct. power.sh is something completeley >> different. Just look into the script, which btw is still present in >> current >> packages. powerbtn is about the power button press while power.sh is about >> powersave. >> >> > Normally this is not an issue, but it does sometimes cause problems such >> > as bug #694775. While power.sh calls CheckUPowerPolicy, >> > powerbtn-acpi-support.sh does not. As a result, upowerd may be running, >> > and certain desktop environments (such as MATE), may be expecting to >> > receive power events, but they will get them too late. By the time >> > upowerd sends the event, acpid will already be shutting the system down. >> >> Well that bug report ralks about mate-power-manager or something like >> that to >> be added to CheckPolicy, but I couldn't find a trace of this in Debian at >> all. >> So how does this relate to upowerd? >> >> 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 >> >> ---------- Forwarded message ---------- >> From: Yury Arkady Sobolev <yury.sobo...@gmail.com> >> To: Debian Bug Tracking System <sub...@bugs.debian.org> >> Cc: >> Date: Wed, 02 Jan 2013 19:41:22 -0500 >> Subject: acpi-support-base: powerbtn-acpi-support.sh checks incorrect >> file for backwards compatibility >> Package: acpi-support-base >> Version: 0.141-2 >> Severity: normal >> Tags: patch >> >> Dear Maintainer, >> >> The powerbtn-acpi-support.sh script in the acpi-support-base package >> will look for previous configuration scripts. Namely, it looks for >> powerbtn.sh and powerbtn.sh.dpkg-back. However, it does not check for >> power.sh. As I understand it, this script has not been used in a long >> time, but it is still present on some systems, while powerbtn.sh is not. >> >> Normally this is not an issue, but it does sometimes cause problems such >> as bug #694775. While power.sh calls CheckUPowerPolicy, >> powerbtn-acpi-support.sh does not. As a result, upowerd may be running, >> and certain desktop environments (such as MATE), may be expecting to >> receive power events, but they will get them too late. By the time >> upowerd sends the event, acpid will already be shutting the system down. >> >> I am attaching a proposed patch. It adds CheckUPowerPolicy to >> powerbtn-acpi-support.sh and also checks for the existence of power.sh. >> >> -Yury >> >> >> -- System Information: >> Debian Release: 7.0 >> APT prefers unstable >> APT policy: (990, 'unstable'), (1, 'experimental') >> Architecture: amd64 (x86_64) >> >> Kernel: Linux 3.2.0-4-amd64 (SMP w/2 CPU cores) >> Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8) >> Shell: /bin/sh linked to /bin/dash >> >> Versions of packages acpi-support-base depends on: >> ii acpid 1:2.0.17-2 >> >> acpi-support-base recommends no packages. >> >> Versions of packages acpi-support-base suggests: >> ii acpi-support 0.141-2 >> ii consolekit 0.4.5-3.1 >> >> -- no debconf information >> >>
