[MPlayer-dev-eng] [PATCH] correct false negative in configure check
Alexander Strange
astrange at ithinksw.com
Sat Jan 22 18:55:15 CET 2005
On Jan 22, 2005, at 9:18 AM, Diego Biurrun wrote:
> I'm a bit suspicious of this patch:
>
> elif hpux ; then
> _ld_pthread=''
> elif ( cc_check && $TMPO ) ; then # QNX
> - _ld_pthread=''
> + _ld_pthread=' '
>
> Maybe you can explain how adding a space as pthread option helps
> Darwin. Also if you look at the line above your change, this check is
> specific for QNX.. Are you sure you are not breaking QNX?
>
> Diego
I can't be completely sure that I'm breaking QNX, since I don't have
anything QNX to test on, but it almost certainly has the same problem
as Darwin there.
The problem is that setting _ld_pthread to '' turns off HAVE_PTHREADS
(line 2720). But if it's passed cc_check and the program runs, it
obviously has pthreads despite not needing any gcc flags for them.
Adding a space still indicates no options, but it doesn't make the test
on 2715/2720 fail. This doesn't make anything go badly wrong, but it
was disabling multithreaded encoding in libavcodec.
More information about the MPlayer-dev-eng
mailing list