get rid of acpid and provide /var/run/acpid.socket by the hald-acpi-addon
Danny Kukawka
danny.kukawka at web.de
Thu Aug 9 04:15:43 PDT 2007
On Mittwoch, 8. August 2007, Kay Sievers wrote:
> With the upcoming ACPI module autoloading, the ACPI sysfs representatin
> and events, the ACPI events passed to the input layer, and the HAL
> support for the events, it's time to get rid of the acpid daemon.
>
> The attached patch implements the multiplexer logic of acpid in the hal
> ACPI addon. That way, distros who care, can optionally get rid of the
> acpid daemon, and still provide the events for tools who still depend on
> it, as long as needed.
>
> If HAL is compiled with the support to listen directly to the kernel
> events and acpid is not running during HAL startup, HAL will open the
> kernel events interface and multiplex the events to listeners at the
> /var.run/acpid.socket, just like acpid was doing it.
Without any test two comments:
[...]
+#include <sys/socket.h>
+#include <sys/poll.h>
+#include <glib-2.0/glib.h>
IMO it should be simply '#include <glib.h>'
---------------------------------------------------
[...]
dbus_error_init (&error);
if (!libhal_device_addon_is_ready (ctx, getenv ("UDI"), &error)) {
+ HAL_ERROR (("libhal_device_addon_is_ready == FALSE"));
return 1;
}
we can remove the (second) 'dbus_error_init (&error);' in main() because we
never reach this code if the error was already set before.
Danny
More information about the hal
mailing list