On 10/08/13 13:04, Martin Storsjö wrote:
> On Sat, 10 Aug 2013, Luca Barbato wrote:
> 
>> Jack support requires a posix call (sem_timedwait) not supported on
>> some platforms that jack supports.
>> ---
>>
>> Something along those lines, probably.
>>
>> configure | 5 +++--
>> 1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/configure b/configure
>> index 2af4d2a..e6ff7d4 100755
>> --- a/configure
>> +++ b/configure
>> @@ -3836,8 +3836,9 @@ check_header soundcard.h
>> enabled_any alsa_indev alsa_outdev &&
>>     check_lib2 alsa/asoundlib.h snd_pcm_htimestamp -lasound
>>
>> -enabled jack_indev && check_lib2 jack/jack.h jack_client_open -ljack &&
>> -    check_func jack_port_get_latency_range -ljack
>> +enabled jack_indev && check_func sem_timedwait &&
>> +    check_func jack_port_get_latency_range -ljack &&
>> +    check_lib2 jack/jack.h jack_client_open -ljack
>>
>> enabled_any sndio_indev sndio_outdev && check_lib2 sndio.h sio_open
>> -lsndio
> 
> This changes the order for check_lib2 jack/jack.h and checking for
> jack_port_get_latency_range. Currently you can build it just fine if you
> don't have jack_port_get_latency_range (those parts are optional in the
> indev). But after this patch you won't be able to... Stefano's last
> patch kept it in the right order.
> 
> But other than this, I guess this is ok and works. Not sure if it's
> better stylisticly to keep it in _deps as well or not (we'd have to ask
> Diego).
> 
> So Stefano's last patch, minus the _deps part, should be exactly what
> you want?

Yep, I assumed jack_port_get_latency_range was needed.

in short

enabled jack_indev && check_func sem_timedwait &&

is all we need to not build on mac correctly.

lu

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to