Re: [Mesa-dev] [PATCH 2/2] egl-static: resolve library linking

2014-05-29 Thread Jose Fonseca
- Original Message - > On 29/05/14 16:45, Jose Fonseca wrote: > > I'm happy however you guys want to address this, as long as it builds. > > > > The only reason I added $(top_builddir)/src/loader/libloader.la was because > > the autotools build was broken for almost a week. I don't care

Re: [Mesa-dev] [PATCH 2/2] egl-static: resolve library linking

2014-05-29 Thread Emil Velikov
On 29/05/14 16:45, Jose Fonseca wrote: > I'm happy however you guys want to address this, as long as it builds. > > The only reason I added $(top_builddir)/src/loader/libloader.la was because > the autotools build was broken for almost a week. I don't care as much for > autotools build breakage

Re: [Mesa-dev] [PATCH 2/2] egl-static: resolve library linking

2014-05-29 Thread Jose Fonseca
I'm happy however you guys want to address this, as long as it builds. The only reason I added $(top_builddir)/src/loader/libloader.la was because the autotools build was broken for almost a week. I don't care as much for autotools build breakage as scons build breakage, but I still don't like

Re: [Mesa-dev] [PATCH 2/2] egl-static: resolve library linking

2014-05-28 Thread Chia-I Wu
On Wed, May 28, 2014 at 10:58 PM, Emil Velikov wrote: > With DRM_PLATFORM libloader.la in linked into libEGL pleading > to multiple defition of the loader* symbols. For every other > platform we need to explicitly link against it. I would prefer to see libloader.la removed from st/egl, and added t

[Mesa-dev] [PATCH 2/2] egl-static: resolve library linking

2014-05-28 Thread Emil Velikov
With DRM_PLATFORM libloader.la in linked into libEGL pleading to multiple defition of the loader* symbols. For every other platform we need to explicitly link against it. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=79263 Cc: José Fonseca Cc: Chia-I Wu Signed-off-by: Emil Velikov ---