[Buildroot] [PATCH] acpid: Make bundled event files optional
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Sun Nov 11 20:50:16 UTC 2018
Hello Alexander,
Adding in Cc: Carlos Santos, who recently tweakedthe powerbtn script
logic.
On Sun, 11 Nov 2018 16:06:51 +0100, Alexander Sverdlin wrote:
> On one hand the preinstalled /etc/acpi/events/powerbtn is not part of
> acpid, one other hand others might want to implement more complicated logic
> on power button. Therefore, make it optional.
>
> Signed-off-by: Alexander Sverdlin <alexander.sverdlin at gmail.com>
On the other hand, it makes quite a bit of sense to have some kind of
default/minimal behavior for this, which can easily be overridden in a
post-build script / root filesystem overlay.
To me, this is a little bit like "let's make the installation of the
OpenSSH server configuration file optional, because the default doesn't
suite me".
> +if BR2_PACKAGE_ACPID
> +
> +config BR2_PACKAGE_ACPID_SUPPLEMENTARY
If anything, the name of the option should be revisited, because
"supplementary" is quite vague.
But overall, I'm not really convinced that we need to change anything
here, I believe the current behavior is OK.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the buildroot
mailing list