<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Aug 8, 2014 at 2:49 PM, Gustavo Zacarias <span dir="ltr"><<a href="mailto:gustavo@zacarias.com.ar" target="_blank">gustavo@zacarias.com.ar</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 08/08/2014 05:14 AM, Pieter De Gendt wrote:<br>
<br>
Hi.<br>
A couple of details...<br>
<div class=""><br>
> +config BR2_PACKAGE_MPD_JACK2<br>
> +     bool "jack2"<br>
> +     depends on BR2_LARGEFILE # libsndfile<br>
> +     depends on BR2_TOOLCHAIN_HAS_THREADS # alsa-lib<br>
> +     depends on BR2_USE_MMU # fork()<br>
> +     depends on BR2_INSTALL_LIBSTDCPP<br>
> +     depends on BR2_TOOLCHAIN_HAS_ATOMIC_INTRINSICS<br>
> +     select BR2_PACKAGE_JACK2<br>
> +     help<br>
> +       Enable jack output support.<br>
> +<br>
<br>
</div>The THREADS, MMU and LIBSTDCPP depends are unnecessary since mpd already<br>
requires them, so you can drop them like it's already done for all of<br>
the other options for mpd.<br>
Also you're missing a comment when LARGEFILE & ATOMIC_INTRINSICS aren't<br>
available telling that jack2 isn't available, similar to what's done for<br>
pulseaudio for example.<br>
Alpha-sorting the menu entry would be good too.<br>
Otherwise it looks good.<br>
Care to send a v2 patch with these fixes?<br>
Thanks.<br>
Regards.<br>
<br></blockquote><div><br></div><div>Done, thanks for comments.</div><div>Regards. </div></div><br></div></div>