build failure; does not include X11 library path in link

Bug #1392715 reported by Davin McCall on 2014-11-14
6
This bug affects 1 person
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/usr/X11R7/lib -lX11

Obviously this can be worked around be setting LIBRARY_PATH=/usr/X11R7/lib before running 'make', but this should not be necessary.

Related branches

Davin McCall (davmac) wrote :
Stephen M. Webb (bregma) wrote :

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).

Changed in compiz:
assignee: nobody → Stephen M. Webb (bregma)
importance: Undecided → Medium
milestone: none → 0.9.12.1
status: New → Triaged
Stephen M. Webb (bregma) on 2014-11-17
Changed in compiz:
status: Triaged → In Progress
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package compiz - 1:0.9.12.0+15.04.20141210.2-0ubuntu1

---------------
compiz (1:0.9.12.0+15.04.20141210.2-0ubuntu1) vivid; urgency=medium

  [ James Jones ]
  * Added support for GL_EXT_x11_sync_object OpenGL extension for Nvidia
    based GPUs to avoid screen refresh issues. (LP: #269904)

  [ Kyle Brenneman ]
  * Added support for GL_EXT_x11_sync_object OpenGL extension for Nvidia
    based GPUs to avoid screen refresh issues. (LP: #269904)

  [ Viktor A. Danilov ]
  * Added support for GL_EXT_x11_sync_object OpenGL extension for Nvidia
    based GPUs to avoid screen refresh issues. (LP: #269904)

  [ Chris Townsend ]
  * Added support for GL_EXT_x11_sync_object OpenGL extension for Nvidia
    based GPUs to avoid screen refresh issues. (LP: #269904)

  [ Stephen M. Webb ]
  * Added support for GL_EXT_x11_sync_object OpenGL extension for Nvidia
    based GPUs to avoid screen refresh issues. (LP: #269904)
  * libdecoration/CMakeLists.txt: use correct variable for link
    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
Stephen M. Webb (bregma) on 2014-12-18
Changed in compiz:
status: In Progress → Fix Committed
Stephen M. Webb (bregma) on 2015-02-03
Changed in compiz:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Bug attachments