build: Don't overlink gallium xlib target
authorDylan Baker <dylan@pnwbakers.com>
Mon, 17 Sep 2018 17:17:48 +0000 (10:17 -0700)
committerDylan Baker <dylan@pnwbakers.com>
Mon, 17 Sep 2018 20:21:01 +0000 (13:21 -0700)
commit7f08bcb73f7f7dbe675b8c4cc87ef98ea4412b72
treee16002241e5e7f79ecd99cff60e1146a02dc7bac
parent3acc18fcf7c38e7060e43e2de59f09392aa8fdea
build: Don't overlink gallium xlib target

Currently gallium's xlib target will fail to link due to multiple
definitions of all the symbols in libmesautil, this only shows up in
autotools, and not in meson due to differences in the way that meson and
autotools handle linking static archives into static archives. Autotools
uses -Wl,--whole-archive implicitly, meson requires this behavior to be
opted-into. The solution is just to remove libmesautils from the
libgl-xlib target, since it will get all of those symbols form
libmesagallium.

I've dropped the link from meson as well, it doesn't seem to hurt
anything and should make linking just a little faster.

Fixes: 8396043f304bb2a752130230055605c5c966e89f
       ("Replace uses of _mesa_bitcount with util_bitcount")
bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107923
Tested-by: Brian Paul <brianp@vmware.com>
Tested-by: Vinson Lee <vlee@freedesktop.org>
Cc: Sergii Romantsov<sergii.romantsov@globallogic.com>
src/gallium/targets/libgl-xlib/Makefile.am
src/gallium/targets/libgl-xlib/meson.build