[MPlayer-dev-eng] [PATCH] correct false negative in configure check
Diego Biurrun
diego at biurrun.de
Sat Jan 22 19:26:29 CET 2005
Alexander Strange writes:
>
> 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?
>
> 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.
OK, you have convinced me now, committed.
Diego
More information about the MPlayer-dev-eng
mailing list