On Sun, Jul 08, 2012 at 09:08:57PM +0100, Måns Rullgård wrote:
> Måns Rullgård <m...@mansr.com> writes:
> > Diego Biurrun <di...@biurrun.de> writes:
> >
> >> On some systems -lm is not required to link against libmath.
> >> ---
> >>  configure |    3 ++-
> >>  1 files changed, 2 insertions(+), 1 deletions(-)
> >>
> >> diff --git a/configure b/configure
> >> index ec68648..7e38c9c 100755
> >> --- a/configure
> >> +++ b/configure
> >> @@ -2931,9 +2931,10 @@ for thread in $THREADS_LIST; do
> >>      fi
> >>  done
> >>
> >> -check_lib math.h sin -lm && LIBM="-lm"
> >>  enabled vaapi && require vaapi va/va.h vaInitialize -lva
> >>
> >> +check_mathfunc sin || { check_mathfunc sin -lm && LIBM="-lm"; }
> >
> > This patch is unnecessary.  It does exactly the same thing as the
> > existing code in a more complicated way.
> 
> Sorry, it's worse than that.  It is plain wrong.

Indeed - fixed patchset coming up in a sec...

Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to