Diego Biurrun <di...@biurrun.de> writes:

> On Sat, Jul 07, 2012 at 01:58:38PM -0700, Ronald S. Bultje wrote:
>> --- a/configure
>> +++ b/configure
>> @@ -2943,7 +2943,9 @@ for thread in $THREADS_LIST; do
>>  
>> -check_lib math.h sin -lm && LIBM="-lm"
>> +if ! check_func_headers math.h sin; then
>> +    check_lib math.h sin -lm && LIBM="-lm"
>> +fi
>
> I fear this has the same problem as Mans noted on my alternative patch:
>
> The math functions may be macros or inline functions and not only
> regular functions.  Thus check_func_headers can succeed even if -lm
> is required for some of the other functions.

check_func_headers checks for an actual linker symbol, so it doesn't
have this particular problem.  However, the change is still unnecessary
for reasons I have already explained.

Any patch trying to do the equivalent of this is automatically rejected.
If you want results, stop wasting my time with more of these.

-- 
Måns Rullgård
m...@mansr.com
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to