Ubuntu
clutter-1.0 package

all redepends unnecessary link directly against mesa-glx because of -lGL in .pc files

Bug #623376 reported by Alexander Sack
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Clutter
Fix Released
Medium
clutk (Ubuntu)
Fix Released
Undecided
Alexander Sack
clutter-1.0 (Ubuntu)
Fix Released
Undecided
Alexander Sack

Bug Description

this is a follow up bug of the eglx support to clutter.

In order to effectively drop-in replace the glx clutter with the eglx one, the rdepends must not directly depend on mesa-glx unless they really use symbols from there.

Current clutter .pc files however, always explicitly link against -lGL, which causes all rdepends to always depend on mesa glx.

One way to fix it is to not put all backend requires into the front API .pc files.

Tags: patch
Alexander Sack (asac)
Changed in clutter-1.0 (Ubuntu):
status: New → Triaged
assignee: nobody → Alexander Sack (asac)
Revision history for this message
Alexander Sack (asac) wrote :
Changed in clutk (Ubuntu):
status: New → Triaged
assignee: nobody → Alexander Sack (asac)
Revision history for this message
Alexander Sack (asac) wrote :

clutk is wrongly relying on clutter -dev and .pc bringing in the gl/gles2/egl dev files.

Changed in clutk (Ubuntu):
assignee: Alexander Sack (asac) → nobody
Revision history for this message
Alexander Sack (asac) wrote :

attaching patch that explicitly links against gl; clutk uses symbols directly so it wasnt right in the first place. Also adds the glx-dev to build depends. This fix is required for the clutter update of this bug.

Alexander Sack (asac)
Changed in clutk (Ubuntu):
assignee: nobody → Alexander Sack (asac)
Bug Watch Updater (bug-watch-updater)
Changed in clutter:
status: Unknown → Confirmed
Brian Murray (brian-murray)
tags: added: patch
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package clutter-1.0 - 1.2.12-0ubuntu10

---------------
clutter-1.0 (1.2.12-0ubuntu10) maverick; urgency=low

  * frontend api does not need BACKEND_PC_FILES; introduce a CLUTTER_FRONTEND_REQUIRES
    AC_SUBST and use that in .pc.in files (LP: #623376)
  * cogl-defines from gl breaks cross gles2 builds; needs to guard
    the GL/gl.h include if gl2.h from gles2 was already included (LP: #623376)
  * Provides: libclutter-1.0-0 does not work if there are versioned depends on it;
    fix debian/*.symbols to produce alternative dependencies on glx/es11/es20 for
    rdepends building against libclutter*-dev
  * in turn add make all variants alternative depends of gir1.0-clutter-1.0
    and gir1.0-clutter-eglx-1.0
 -- Alexander Sack <email address hidden> Mon, 23 Aug 2010 20:21:49 +0200

Changed in clutter-1.0 (Ubuntu):
status: Triaged → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package clutk - 0.3.50-0ubuntu3

---------------
clutk (0.3.50-0ubuntu3) maverick; urgency=low

  * debian/patches/02_gl_flags.patch:
    - add gl requirement in build system
    - add libgl-mesa-dev to build depends
    - fixes LP: #623376
  * debian/patches/99_autoreconf.patch
    - apply build-system changes from above
 -- Alexander Sack <email address hidden> Tue, 24 Aug 2010 16:20:42 +0200

Changed in clutk (Ubuntu):
status: Triaged → Fix Released
Bug Watch Updater (bug-watch-updater)
Changed in clutter:
importance: Unknown → Medium
Bug Watch Updater (bug-watch-updater)
Changed in clutter:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.