Bug 31454 - linking to libcanberra-gtkX fails because of missing -lX11
Summary: linking to libcanberra-gtkX fails because of missing -lX11
Status: RESOLVED FIXED
Alias: None
Product: libcanberra
Classification: Unclassified
Component: Unspecified (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Lennart Poettering
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-07 15:51 UTC by Diego Escalante Urrelo
Modified: 2012-09-25 00:11 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:

Attachments
Patch for explictly adding x11 to GTK* flags (1.03 KB, patch)
2011-02-28 09:36 UTC, Rodrigo Moya 		Details | Splinter Review
link-against-libx11 (717 bytes, patch)
2012-04-05 19:36 UTC, Jeremy Bicha 		Details | Splinter Review
Explicitly link against gdk and add missing gdk header (2.17 KB, patch)
2012-09-23 00:11 UTC, Laurent Bigonville 		Details | Splinter Review
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Escalante Urrelo 2010-11-07 15:51:05 UTC 
While trying to build the new gnome control center, the sound applet failed to link because libcanberra-gtk3 couldn't find XFree and XGetWindowProperty.

Adding -lX11 fixed the issue.
Comment 1 Lennart Poettering 2011-02-24 16:01:08 UTC 
Hmm, we need proper pkgconfig logic in configure.ac for that i guess.
Comment 2 Rodrigo Moya 2011-02-28 09:36:21 UTC 
Created attachment 43934 [details] [review]
Patch for explictly adding x11 to GTK* flags
Comment 3 Jeremy Bicha 2012-04-05 19:36:45 UTC 
Created attachment 59556 [details] [review]
link-against-libx11

I ran into this bug (I believe) when trying to build gnome-games. I've been running the attached patch for several weeks and it seemed to work ok. I don't know a lot about .pc files though.

https://bugzilla.gnome.org/show_bug.cgi?id=667905
Comment 4 Laurent Bigonville 2012-09-23 00:11:47 UTC 
Created attachment 67569 [details] [review]
Explicitly link against gdk and add missing gdk header

Hi,

I think that to be 100% correct the attached patch should also be added in addition of Rodrigo one
Comment 5 Lennart Poettering 2012-09-25 00:11:16 UTC 
Applied all patches.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct.