[Buildroot] [PATCH 4/5] package/shairport-sync: add dbus option

Thomas Petazzoni thomas.petazzoni at bootlin.com
Sun Dec 8 22:24:23 UTC 2019


On Tue,  3 Dec 2019 11:53:40 +0100
Jörg Krause <joerg.krause at embedded.rocks> wrote:

> Shairport Sync added DBus support in version 3.2.
> 
> Signed-off-by: Jörg Krause <joerg.krause at embedded.rocks>
> ---
>  package/shairport-sync/Config.in         | 11 +++++++++++
>  package/shairport-sync/shairport-sync.mk |  5 +++++
>  2 files changed, 16 insertions(+)
> 
> diff --git a/package/shairport-sync/Config.in b/package/shairport-sync/Config.in
> index 87f38a940d..6325e9d30d 100644
> --- a/package/shairport-sync/Config.in
> +++ b/package/shairport-sync/Config.in
> @@ -26,6 +26,17 @@ config BR2_PACKAGE_SHAIRPORT_SYNC_CONVOLUTION
>  	help
>  	  Enable audio DSP convolution support.
>  
> +config BR2_PACKAGE_SHAIRPORT_SYNC_DBUS
> +	bool "dbus support"
> +	depends on BR2_USE_WCHAR # libglib2 -> gettext

No need to repeat the full comment, just "# libglib2" is sufficient.

However, you forgot the other dependencies from libglib2: yes the
thread dependency was already implied by the dependency of
shairport-sync on NPTL, but the BR2_USE_MMU dependency was missing. So
I've replicated both, to keep things simple.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


More information about the buildroot mailing list