On 09/07/2012 17:36, Eric Anholt wrote: >> We must either: >> (i) Not link libmesa with libglapi, and link anything that uses libmesa with >> libglapi as well, or >> (ii) Link libmesa with libglapi, and not link anything that uses libmesa with >> libglapi >> >> I choose (ii) just because it's least change, but I'm not sure it's right. >> >> Note that drivers/X11 makes libGL by linking with libmesa, but not libglapi, >> so it's a bit unclear to me how this was working correctly before in both >> cases >> (It might be some sort of accidental side-effect of what mklib's >> expand_archives() >> function does that this ever worked before?) >> --- >> src/mesa/drivers/osmesa/Makefile.am | 3 +-- >> 1 files changed, 1 insertions(+), 2 deletions(-) >> >> diff --git a/src/mesa/drivers/osmesa/Makefile.am >> b/src/mesa/drivers/osmesa/Makefile.am >> index a91d9e2..cfba658 100644 >> --- a/src/mesa/drivers/osmesa/Makefile.am >> +++ b/src/mesa/drivers/osmesa/Makefile.am >> @@ -40,8 +40,7 @@ lib@OSMESA_LIB@_la_SOURCES = osmesa.c >> lib@OSMESA_LIB@_la_LDFLAGS = -module -version-number @OSMESA_VERSION@ >> -shared >> lib@OSMESA_LIB@_la_LIBADD = \ >> $(top_srcdir)/src/mesa/libmesa.la \ >> - $(top_srcdir)/src/mapi/glapi/libglapi.la \ >> - $(top_srcdir)/src/glsl/libglsl.la >> + $(top_srcdir)/src/mapi/glapi/libglapi.la > > This commit actually looks like it drops linking with libglsl. What's > up?
Sorry, I had some kind of brain spasm when writing the comment for this commit, it should of course say libglsl everywhere it says libglapi. The libmesa convenience library is linked with the libglsl convenience library. libOsmesa is linked with libmesa, and also directly with libglsl. This gives rise to duplicate symbol errors. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
