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 <j...@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 <j...@jsg.id.au> > > Cc: mesa-sta...@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@lists.freedesktop.org > > https://lists.freedesktop.org/mailman/listinfo/mesa-dev > > > > > -- > Sergii Romantsov > GlobalLogic Inc. > www.globallogic.com _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
