<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Aug 25, 2017 at 1:20 PM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Emil Velikov <<a href="mailto:emil.velikov@collabora.com">emil.velikov@collabora.com</a>><br>
<br>
Originally required for the u_vector implementation, which was inlined<br>
in u_vector.h with previous commit.<br>
<br>
Using libmesautil pulled the C++ runtime (string_to_uint_map.cpp),<br>
which is something don't want to impose in our libEGL.<br>
<br>
We could consider rewriting string_to_uint_map in C, but that's too<br>
invasive for a stable fix.<br></blockquote><div><br></div><div>A quick grep and it looks like the only users of string_to_uint_map are src/compiler/glsl and src/mesa which depends on src/compiler/glsl.  Why not just move string_to_uint_map into src/compiler/glsl or src/compiler until such a time as we actually have another user.  Don't get me wrong, I think string_to_uint_map is useful but I think the better option here is to disallow C++ in src/util.</div><div><br></div><div>--Jason<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Cc: Daniel Stone <<a href="mailto:daniels@collabora.com">daniels@collabora.com</a>><br>
Cc: "17.2" <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.<wbr>freedesktop.org</a>><br>
Fixes: 02cc35937277 ("egl/wayland: Use linux-dmabuf interface for buffers")<br>
Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=101851" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/<wbr>show_bug.cgi?id=101851</a><br>
Signed-off-by: Emil Velikov <<a href="mailto:emil.velikov@collabora.com">emil.velikov@collabora.com</a>><br>
---<br>
 src/egl/Makefile.am | 1 -<br>
 1 file changed, 1 deletion(-)<br>
<br>
diff --git a/src/egl/Makefile.am b/src/egl/Makefile.am<br>
index bb8ec9745dd..7331b616a8d 100644<br>
--- a/src/egl/Makefile.am<br>
+++ b/src/egl/Makefile.am<br>
@@ -82,7 +82,6 @@ AM_CFLAGS += $(WAYLAND_CFLAGS)<br>
 libEGL_common_la_LIBADD += $(WAYLAND_LIBS)<br>
 libEGL_common_la_LIBADD += $(LIBDRM_LIBS)<br>
 libEGL_common_la_LIBADD += $(top_builddir)/src/egl/<wbr>wayland/wayland-drm/<a href="http://libwayland-drm.la" rel="noreferrer" target="_blank">libwayland<wbr>-drm.la</a><br>
-libEGL_common_la_LIBADD += $(top_builddir)/src/util/<a href="http://libmesautil.la" rel="noreferrer" target="_blank">libme<wbr>sautil.la</a><br>
 dri2_backend_FILES += \<br>
        drivers/dri2/platform_wayland.<wbr>c \<br>
        drivers/dri2/linux-dmabuf-<wbr>unstable-v1-protocol.c \<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.13.3<br>
<br>
______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div></div>