<div dir="ltr">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.<div><br></div><div style>-Yury</div>
<div style><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jan 3, 2013 at 3:09 PM, Yury Sobolev <span dir="ltr"><<a href="mailto:yury.sobolev@gmail.com" target="_blank">yury.sobolev@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p>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.</p>


<p>-Yury<br>
 </p>
<div class="gmail_quote"><div><div class="h5">On Jan 3, 2013 7:51 AM, "Debian Bug Tracking System" <<a href="mailto:owner@bugs.debian.org" target="_blank">owner@bugs.debian.org</a>> wrote:<br type="attribution">
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
This is an automatic notification regarding your Bug report<br>
which was filed against the acpi-support-base package:<br>
<br>
#697246: acpi-support-base: powerbtn-acpi-support.sh checks incorrect file for backwards compatibility<br>
<br>
It has been closed by Michael Meskes <<a href="mailto:meskes@debian.org" target="_blank">meskes@debian.org</a>>.<br>
<br>
Their explanation is attached below along with your original report.<br>
If this explanation is unsatisfactory and you have not received a<br>
better one in a separate message then please contact Michael Meskes <<a href="mailto:meskes@debian.org" target="_blank">meskes@debian.org</a>> by<br>
replying to this email.<br>
<br>
<br>
--<br>
697246: <a href="http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=697246" target="_blank">http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=697246</a><br>
Debian Bug Tracking System<br>
Contact <a href="mailto:owner@bugs.debian.org" target="_blank">owner@bugs.debian.org</a> with problems<br>
<br><br></div></div>---------- Forwarded message ----------<br>From: Michael Meskes <<a href="mailto:meskes@debian.org" target="_blank">meskes@debian.org</a>><br>To: Yury Arkady Sobolev <<a href="mailto:yury.sobolev@gmail.com" target="_blank">yury.sobolev@gmail.com</a>>, <a href="mailto:697246-done@bugs.debian.org" target="_blank">697246-done@bugs.debian.org</a><br>

Cc: <br>Date: Thu, 3 Jan 2013 13:47:25 +0100<br>Subject: Re: [Pkg-acpi-devel] Bug#697246: acpi-support-base: powerbtn-acpi-support.sh checks incorrect file for backwards compatibility<br>On Wed, Jan 02, 2013 at 07:41:22PM -0500, Yury Arkady Sobolev wrote:<br>


> The powerbtn-acpi-support.sh script in the acpi-support-base package<br>
> will look for previous configuration scripts. Namely, it looks for<br>
> powerbtn.sh and powerbtn.sh.dpkg-back. However, it does not check for<br>
> power.sh. As I understand it, this script has not been used in a long<br>
> time, but it is still present on some systems, while powerbtn.sh is not.<br>
<br>
No, your understanding is not correct. power.sh is something completeley<br>
different. Just look into the script, which btw is still present in current<br>
packages. powerbtn is about the power button press while power.sh is about<br>
powersave.<br>
<br>
> Normally this is not an issue, but it does sometimes cause problems such<br>
> as bug #694775. While power.sh calls CheckUPowerPolicy,<br>
> powerbtn-acpi-support.sh does not. As a result, upowerd may be running,<br>
> and certain desktop environments (such as MATE), may be expecting to<br>
> receive power events, but they will get them too late. By the time<br>
> upowerd sends the event, acpid will already be shutting the system down.<br>
<br>
Well that bug report ralks about mate-power-manager or something like that to<br>
be added to CheckPolicy, but I couldn't find a trace of this in Debian at all.<br>
So how does this relate to upowerd?<br>
<br>
Michael<br>
<br>
--<br>
Michael Meskes<br>
Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)<br>
Michael at BorussiaFan dot De, Meskes at (Debian|Postgresql) dot Org<br>
Jabber: michael.meskes at gmail dot com<br>
VfL Borussia! Força Barça! Go SF 49ers! Use Debian GNU/Linux, PostgreSQL<br><br>---------- Forwarded message ----------<br>From: Yury Arkady Sobolev <<a href="mailto:yury.sobolev@gmail.com" target="_blank">yury.sobolev@gmail.com</a>><br>

To: Debian Bug Tracking System <<a href="mailto:submit@bugs.debian.org" target="_blank">submit@bugs.debian.org</a>><br>Cc: <br>Date: Wed, 02 Jan 2013 19:41:22 -0500<br>Subject: acpi-support-base: powerbtn-acpi-support.sh checks incorrect file for backwards compatibility<br>

Package: acpi-support-base<br>
Version: 0.141-2<br>
Severity: normal<br>
Tags: patch<br>
<br>
Dear Maintainer,<br>
<br>
The powerbtn-acpi-support.sh script in the acpi-support-base package<br>
will look for previous configuration scripts. Namely, it looks for<br>
powerbtn.sh and powerbtn.sh.dpkg-back. However, it does not check for<br>
power.sh. As I understand it, this script has not been used in a long<br>
time, but it is still present on some systems, while powerbtn.sh is not.<br>
<br>
Normally this is not an issue, but it does sometimes cause problems such<br>
as bug #694775. While power.sh calls CheckUPowerPolicy,<br>
powerbtn-acpi-support.sh does not. As a result, upowerd may be running,<br>
and certain desktop environments (such as MATE), may be expecting to<br>
receive power events, but they will get them too late. By the time<br>
upowerd sends the event, acpid will already be shutting the system down.<br>
<br>
I am attaching a proposed patch. It adds CheckUPowerPolicy to<br>
powerbtn-acpi-support.sh and also checks for the existence of power.sh.<br>
<br>
-Yury<br>
<br>
<br>
-- System Information:<br>
Debian Release: 7.0<br>
  APT prefers unstable<br>
  APT policy: (990, 'unstable'), (1, 'experimental')<br>
Architecture: amd64 (x86_64)<br>
<br>
Kernel: Linux 3.2.0-4-amd64 (SMP w/2 CPU cores)<br>
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)<br>
Shell: /bin/sh linked to /bin/dash<br>
<br>
Versions of packages acpi-support-base depends on:<br>
ii  acpid  1:2.0.17-2<br>
<br>
acpi-support-base recommends no packages.<br>
<br>
Versions of packages acpi-support-base suggests:<br>
ii  acpi-support  0.141-2<br>
ii  consolekit    0.4.5-3.1<br>
<br>
-- no debconf information<br>
<br></blockquote></div>
</blockquote></div><br></div>