Make dependency on GTK+ optional at compile time

Bug #431311 reported by Aurélien Gâteau
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Unity Foundations
Fix Released
Low
Ted Gould
libindicate
Fix Released
Wishlist
Ted Gould

Bug Description

Source-based distribution packagers would like to be able to provide libindicate to users without forcing them to install GTK+ to build it.

Related branches

Changed in libindicate:
importance: Undecided → Wishlist
Ted Gould (ted)
Changed in libindicate:
status: New → Confirmed
Revision history for this message
Dennis Schridde (devurandom) wrote :

As GObject Introspection is somewhat broken, and even more so is the test using a non-existant macro in configure.ac, the following patch was also necessary. I do not consider it a solution of any kind, so it's not attached.
--- configure.ac.orig 2010-03-31 12:42:16.497596552 +0200
+++ configure.ac 2010-03-31 12:42:47.001595062 +0200
@@ -87,6 +87,6 @@
 ###########################

-GOBJECT_INTROSPECTION_CHECK([0.6.3])
+AM_CONDITIONAL([HAVE_INTROSPECTION], [false])

 ###########################
 # GTK Doc

Revision history for this message
Dennis Schridde (devurandom) wrote :

See bug #552537 for the GObject Introspection issue.

Revision history for this message
Dennis Schridde (devurandom) wrote :

Attached patch makes it possible to disable compilation of libindicate-gtk, but it fails if you build the python bindings, because they try to link against it.

So this depends on bug #643921, or needs some code modifications to exclude the gtk dependent stuff from the python bindings.

Revision history for this message
Dennis Schridde (devurandom) wrote :

Attached patch improves upon the last, mainly preserving order of SUBDIRS.

Revision history for this message
Alex Launi (alexlauni) wrote :

Could you please add some output to the configure to show whether not gtk support will be built?

Changed in libindicate:
status: Confirmed → Triaged
Revision history for this message
Dennis Schridde (devurandom) wrote :

Improved patch with safer macro quoting and diagnostic output at the end of configure.

Ted Gould (ted)
Changed in libindicate:
status: Triaged → Fix Committed
status: Fix Committed → Confirmed
Ted Gould (ted)
Changed in libindicate:
status: Confirmed → Fix Committed
milestone: none → 0.5.92
assignee: nobody → Ted Gould (ted)
Changed in unity-foundations:
status: New → Fix Committed
importance: Undecided → Low
assignee: nobody → Ted Gould (ted)
milestone: none → oneiric-beta-2
Ted Gould (ted)
Changed in libindicate:
milestone: 0.5.92 → 0.6.0
Ted Gould (ted)
Changed in libindicate:
status: Fix Committed → Fix Released
Ted Gould (ted)
Changed in unity-foundations:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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