[Buildroot] [PATCH v2 8/8] package/mpd: fix zeroconf support

Jörg Krause jkrause at posteo.de
Thu Oct 16 10:57:54 UTC 2014


Dear Thomas Petazzoni,

On 10/16/2014 11:50 AM, Thomas Petazzoni wrote:
> Dear Jörg Krause,
>
> On Wed, 15 Oct 2014 15:18:54 +0200, Jörg Krause wrote:
>
>>   # Some options need an explicit --disable or --enable
>> -ifeq ($(BR2_PACKAGE_AVAHI_DAEMON),y)
>> +
>> +# Zeroconf support depends on libdns_sd from avahi.
>> +ifeq ($(BR2_PACKAGE_AVAHI_LIBDNSSD_COMPATIBILITY),y)
>>   MPD_DEPENDENCIES += avahi
>>   else
>>   MPD_CONF_OPTS += --with-zeroconf=no
> It doesn't seem really obvious for the user that
> BR2_PACKAGE_AVAHI_LIBDNSSD_COMPATIBILITY should be enabled to get avahi
> support in mpd. So, in such situations, I would suggest to add a
> Config.in option to explicitly enable the Avahi support in mpd:
>
> config BR2_PACKAGE_MPD_AVAHI_SUPPORT
> 	bool "avahi support"
> 	depends on ... # all the dependencies of avahi
> 	select BR2_PACKAGE_AVAHI
> 	select BR2_PACKAGE_AVAHI_LIBDNSSD_COMPATIBILITY
> 	help
> 	  Some help text

Looks good!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20141016/a1d76809/attachment-0002.html>


More information about the buildroot mailing list