[PATCH] command line options for hald-addon-acpi

Richard Hughes hughsient at gmail.com
Tue Aug 2 02:59:20 PDT 2005


On Tue, 2005-08-02 at 11:45 +0200, Danny Kukawka wrote:
> Hi,
> this patch adds command line options to hald-addon-acpi to set source socket 
> for ACPI events. Added acpid_only=yes|no and proc_only=yes|no. With no 
> arguments hald-addon-acpi will works so far.

Why? What's wrong with the fallback options that we have now? That gives
us the legacy support of acpid and the newest non-acpid support. 

I've removed acpid now, as I have no use for it so I use the proc option
- most people will use the socket option.

I thought DavidZ comment:

/* TODO: make /var/run/acpid.socket a configure option */

meant to make the location a -- configure'able option, rather than a
argument type option.

Richard.

_______________________________________________
hal mailing list
hal at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/hal



More information about the Hal mailing list