Summary: | Linking error in egl_gallium.la when compiling 32 bit on multiarch | ||
---|---|---|---|
Product: | Mesa | Reporter: | Knut Andre Tidemann <knut.tidemann> |
Component: | EGL | Assignee: | mesa-dev |
Status: | RESOLVED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | medium | CC: | emil.l.velikov, olvaffe |
Version: | git | ||
Hardware: | x86-64 (AMD64) | ||
OS: | Linux (All) | ||
Whiteboard: | |||
i915 platform: | i915 features: |
Description
Knut Andre Tidemann
2014-05-26 14:15:34 UTC
The issue was really introduced by commit 39ae284a6941df5942602c027d4a8863ca35e654 Author: Emil Velikov <emil.l.velikov@gmail.com> Date: Sat May 10 15:59:03 2014 +0100 egl-static: include libradeonwinsys.la only once which removed libloader.la from LIBADD. CC Emil. Handling of the egl_platform is slightly move convoluted than expected. platform_drm - things will build nicely as the loader* symbols are provided via gbm/libgbm. Everything else - the symbols are missing. Might be better off re-adding the libloader + multiple-declarations for now. Jose has pushed a patch that adds the symbol back, although it causes a multiple definition of loader* as expected - bug 79382. Proper fix is available http://patchwork.freedesktop.org/patch/26746/ Fixed with commit febec731475a5b288cc50660b2d32780e395b950 Author: Emil Velikov <emil.l.velikov@gmail.com> Date: Wed May 28 15:43:35 2014 +0100 st/egl: do not link against libloader Move the link to the final targets, like any other place in mesa/gallium. This allows better visibilty and will prevent us from including the library archive twice. Resolves multiple definition of `loader_get_pci_id_for_fd' multiple definition of `loader_get_pci_id_for_fd' |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.