[Pkg-acpi-devel] 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]

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.sobolev at 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" <
> owner at 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 <meskes at 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 <
>> meskes at debian.org> by
>> replying to this email.
>>
>>
>> --
>> 697246: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=697246
>> Debian Bug Tracking System
>> Contact owner at bugs.debian.org with problems
>>
>>
>> ---------- Forwarded message ----------
>> From: Michael Meskes <meskes at debian.org>
>> To: Yury Arkady Sobolev <yury.sobolev at gmail.com>,
>> 697246-done at 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.sobolev at gmail.com>
>> To: Debian Bug Tracking System <submit at 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
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/pkg-acpi-devel/attachments/20130114/eac8772b/attachment.html>