FS#18017 - [acpid] Add /etc/conf.d/acpid

Attached to Project: Arch Linux
Opened by Sebastian (sepo) - Tuesday, 26 January 2010, 15:38 GMT
Last edited by Thayer Williams (thayer) - Thursday, 28 January 2010, 19:02 GMT
Task Type Feature Request
Category Packages: Extra
Status Closed
Assigned To Thayer Williams (thayer)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Description:

I would like to pass some command line options to the acpid daemon, but it dont like to edit the /etc/rc.d/acpid daemon script directly. It would be nice to have a file /etc/conf.d/acpid which let the user specify some command line options.

Additional info:
acpid 1.0.10-1
This task depends upon

Closed by  Thayer Williams (thayer)
Thursday, 28 January 2010, 19:02 GMT
Reason for closing:  Implemented
Comment by Thayer Williams (thayer) - Tuesday, 26 January 2010, 19:04 GMT
I'm not sure what you had in mind, but are you aware of /etc/acpi/handler.sh
The daemon script itself should only be used to start/stop the acpid service.
Comment by Sebastian (sepo) - Wednesday, 27 January 2010, 07:06 GMT
As far as i can see /etc/acpi/handler.sh has nothing to do with the command line switches of acpid.
I mean a small file named /etc/conf.d/acpid which holds a variable ACPID_ARGS were i can place some command line switches of acpid.
e.g. ACPID_ARGS="--logevents --confdir /etc/apicd/my_events".
At the moment i have to add these command line switches directly to /etc/rc.d/acpid which is in my opinion not the preferred way.

I've added a patch to visualize what i mean.
   acpid.conf.d.diff (1.8 KiB)
Comment by Sebastian (sepo) - Thursday, 28 January 2010, 16:30 GMT
Oh i forgot to add "/etc/conf.d/acpid" to the "backup" array of the PKGBUILD.

backup=('etc/acpi/handler.sh' 'etc/acpi/events/anything' 'etc/conf.d/acpid')
Comment by Thayer Williams (thayer) - Thursday, 28 January 2010, 19:02 GMT
Thanks for the clarification and patch. I've updated it locally and will push it sometime today. Cheers!

Loading...