[Buildroot] [PATCH] package/usbmount: select util-linux logger

[Arnout Vandecappelle]

On 28/11/2022 10:41, Michael Nosthoff via buildroot wrote:
> usbmount has a runtime dependency on "logger" from util-linux.
> Else it will just fail when it tries to log.

  If it is just /bin/logger, that is usually provided by busybox (if 
CONFIG_LOGGER=y). We don't normally add things for stuff provided by busybox, at 
least if it is enabled by our default configuration.

  So, do you maybe have a custom busybox config?

  I've marked the patch as Rejected, feel free to revert if you think this is 
needed after all.

> 
> Signed-off-by: Michael Nosthoff <buildroot at heine.tech>
> 
> ---
> 
> I wasn't sure if it is neccessary to also add
> select BR2_PACKAGE_UTIL_LINUX
> or if this is done automatically.

  No it's not done automatically, you indeed have to add it.

  Regards,
  Arnout

> 
> So please adjust if neccessary.
> ---
>   package/usbmount/Config.in | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/package/usbmount/Config.in b/package/usbmount/Config.in
> index 7760356b7c..5145587b7c 100644
> --- a/package/usbmount/Config.in
> +++ b/package/usbmount/Config.in
> @@ -3,6 +3,7 @@ config BR2_PACKAGE_USBMOUNT
>   	depends on BR2_USE_MMU # lockfile-progs -> liblockfile
>   	depends on BR2_PACKAGE_HAS_UDEV
>   	select BR2_PACKAGE_LOCKFILE_PROGS
> +	select BR2_PACKAGE_UTIL_LINUX_LOGGER
>   	help
>   	  The usbmount package automatically mounts USB mass storage
>   	  devices when they are plugged in, and unmounts them when