build failure; does not include X11 library path in link
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | Compiz |
Medium
|
Stephen M. Webb | ||
| | compiz (Ubuntu) |
Undecided
|
Unassigned | ||
Bug Description
I have just tried to build compiz-0.9.12.0. I encountered several problems, but the first one (which is at least easily worked around) is that the correct library paths for certain X11 libraries are not used when linking. I tried the most straightforward build possible:
mkdir build
cd build
cmake ..
make VERBOSE=1 2>&1 | tee build.log
I have attached the resulting build.log file. As you can see there is a link failure since the build tries to link against libraries libXrender and libX11, but it does not include the library path (via -L switch). On my system these libraries are in /usr/X11R7/lib, and this is correctly identified by pkg-config:
$ pkg-config --libs x11
-L/
Obviously this can be worked around be setting LIBRARY_
Related branches
- Christopher Townsend: Approve on 2014-12-02
- PS Jenkins bot: Approve (continuous-integration) on 2014-11-19
-
Diff: 31 lines (+2/-4)2 files modifiedCMakeLists.txt (+1/-1)
libdecoration/CMakeLists.txt (+1/-3)
| Davin McCall (davmac) wrote : | #1 |
| Stephen M. Webb (bregma) wrote : | #2 |
| Changed in compiz: | |
| assignee: | nobody → Stephen M. Webb (bregma) |
| importance: | Undecided → Medium |
| milestone: | none → 0.9.12.1 |
| status: | New → Triaged |
| Changed in compiz: | |
| status: | Triaged → In Progress |
| Launchpad Janitor (janitor) wrote : | #3 |
This bug was fixed in the package compiz - 1:0.9.12.
---------------
compiz (1:0.9.
[ James Jones ]
* Added support for GL_EXT_
based GPUs to avoid screen refresh issues. (LP: #269904)
[ Kyle Brenneman ]
* Added support for GL_EXT_
based GPUs to avoid screen refresh issues. (LP: #269904)
[ Viktor A. Danilov ]
* Added support for GL_EXT_
based GPUs to avoid screen refresh issues. (LP: #269904)
[ Chris Townsend ]
* Added support for GL_EXT_
based GPUs to avoid screen refresh issues. (LP: #269904)
[ Stephen M. Webb ]
* Added support for GL_EXT_
based GPUs to avoid screen refresh issues. (LP: #269904)
* libdecoration/
directories (LP: #1392715)
* Switcher Plugin: renamed a local variable so it does not hide a
function parameter. (LP: #1101585)
* added support for multi-arch installations (LP: #1395105, #959282)
* libcompizconfig: reorder stat() and open() calls to eliminate a race
condition (LP: #1101608)
* dodge plugin: avoid potential null pointer dereferences (LP:
#1101554)
* PrivateScreen unit tests: added additional assertions to verify the
tests are run in an expected state (LP: #1101413)
[ Ubuntu daily release ]
* New rebuild forced
[ Brandon Schaefer ]
* Rev 3891 change caused 2 known regressions. (Could be more).
Reverting. (LP: #1398512, #1393020)
-- Ubuntu daily release <email address hidden> Wed, 10 Dec 2014 19:52:06 +0000
| Changed in compiz (Ubuntu): | |
| status: | New → Fix Released |
| Changed in compiz: | |
| status: | In Progress → Fix Committed |
| Changed in compiz: | |
| status: | Fix Committed → Fix Released |


This particular problem is die to a typo introduced in 2008 and only shows up when someone tries to build Compiz against Xrender development files build into a non-FHS-standard custom location (which explains why it's only being reported now).