FS#8026 - default handler script for acpid conflicts with acpi philosophy

Attached to Project: Arch Linux
Opened by Pete (tam1138) - Sunday, 16 September 2007, 06:19 GMT
Last edited by Thayer Williams (thayer) - Sunday, 07 December 2008, 02:32 GMT
Task Type Feature Request
Category Packages: Extra
Status Closed
Assigned To Tobias Powalowski (tpowa)
Thayer Williams (thayer)
Architecture All
Severity Low
Priority Normal
Reported Version 2007.08 Don't Panic
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 2
Private No

Details

Description:

As shipped with acpid, /etc/acpi/handler.sh is a "default acpi script that takes an entry for all actions". There are three problems with this: first, it does not, in fact, contain an entry for "all" actions. The hotkeys on my ThinkPad are not supported. Secondly, the design of the acpi event delivery mechanism encourages separate scripts for every action, which is a cleaner, more understandable, and easier-to-maintain approach. The third problem is a consequence of the second: handler.sh is fugly and ponderous.

I would volunteer to produce a set of replacement default scripts for acpid, providing the Powers-That-Be approve. I like the (as I recall it) Gentoo method whereby the default script merely prints the received event to the system logger: this allows people writing new handlers to easily look up the events they want to handle, while remaining a nop.

Additional info:
* package version(s)

$ pacman -Qs acpid
local/acpid 1.0.6-1

* config and/or log files etc.

/etc/acpi/handler.sh

Steps to reproduce:

# cat /etc/acpi/handler.sh
# /usr/bin/gag
This task depends upon

Closed by  Thayer Williams (thayer)
Sunday, 07 December 2008, 02:32 GMT
Reason for closing:  Works for me
Comment by Riri (chicha) - Friday, 09 May 2008, 12:48 GMT

Hello,

This is not a bug but a feature request.

Please open a feature request, and as long as you volunteer for providing a new solution, please attach the scripts and patches to the report.
Also you may (if you wish to, of course) post something to AUR and the forums and see if the community like it or not.

There is no doubt that if your solution is better than the current one it will be adopted.
Thank you very much !
Comment by Aaron Griffin (phrakture) - Friday, 09 May 2008, 17:22 GMT
Changed the report to a feature request
Comment by Glenn Matthys (RedShift) - Friday, 18 July 2008, 15:27 GMT
If it doesn't come from upstream this way I don't see why we have to do it. If the reporter really wants this "feature", they can push for it upstream @ https://sourceforge.net/tracker/?group_id=33140&atid=407341
Comment by Thayer Williams (thayer) - Sunday, 07 December 2008, 02:32 GMT
Closing this ticket. I only recently adopted acpid, but from what I can see the current handler.sh is largely what the OP desired--it doesn't really do anything except provide a shell to build from--what an administrator decides to do with it beyond that is entirely up to them. If the current handler is a result of the OP, then thanks for your contribution!

Cheers and all that =)

Loading...