[Mesa-stable] [Mesa-dev] [PATCH] intel/aubinator_error_decode: link libm

[Jonathan Gray]

Yes, that looks like the way to go.

On Thu, Nov 01, 2018 at 09:22:35AM +0200, Sergii Romantsov wrote:
> Hello, it looks like not complete fix.
> In my opinion issue is that libmesautil requires -lm than any library that
> uses libmesautil will not have such issue.
> Patch: https://patchwork.freedesktop.org/patch/259176/
> 
> On Thu, Nov 1, 2018 at 6:12 AM Jonathan Gray <jsg at jsg.id.au> wrote:
> 
> > aubinator_error_decode needs to link libm to build on OpenBSD/i386
> > ../../src/util/.libs/libmesautil.a(libmesautil_la-half_float.o): In
> > function `_mesa_float_to_half':
> > half_float.c:(.text+0x91): undefined reference to `lrintf'
> > half_float.c:(.text+0xc1): undefined reference to `lrintf'
> >
> > Signed-off-by: Jonathan Gray <jsg at jsg.id.au>
> > Cc: mesa-stable at lists.freedesktop.org
> > ---
> >  src/intel/Makefile.tools.am | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/src/intel/Makefile.tools.am b/src/intel/Makefile.tools.am
> > index 4809962b188..da49d37a728 100644
> > --- a/src/intel/Makefile.tools.am
> > +++ b/src/intel/Makefile.tools.am
> > @@ -61,7 +61,8 @@ tools_aubinator_error_decode_LDADD = \
> >         isl/libisl.la \
> >         $(top_builddir)/src/util/libmesautil.la \
> >         $(PTHREAD_LIBS) \
> > -       $(ZLIB_LIBS)
> > +       $(ZLIB_LIBS) \
> > +       -lm
> >
> >  tools_aubinator_error_decode_CFLAGS = \
> >         $(AM_CFLAGS) \
> > --
> > 2.19.1
> >
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> >
> 
> 
> -- 
> Sergii Romantsov
> GlobalLogic Inc.
> www.globallogic.com