After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 643077 - Header collision between clutter-gtk-1.0 and clutter-gtk-0.10 due to include path order
Header collision between clutter-gtk-1.0 and clutter-gtk-0.10 due to include ...
Status: RESOLVED FIXED
Product: libchamplain
Classification: Core
Component: General
0.9.x
Other Linux
: Normal normal
: ---
Assigned To: libchamplain-maint
libchamplain-maint
Depends on:
Blocks:
 
 
Reported: 2011-02-23 15:21 UTC by Nirbheek Chauhan
Modified: 2011-02-23 23:34 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Change depend order in pkg-config file, and CFLAG order in champlain-gtk/Makefile.am (1.65 KB, patch)
2011-02-23 15:23 UTC, Nirbheek Chauhan 		none Details | Review

Description Nirbheek Chauhan 2011-02-23 15:21:04 UTC 
clutter-1.0 installs its headers in: /usr/include/clutter-1.0/clutter

clutter-gtk-0.10 installs its headers in: /usr/include/clutter-1.0/clutter-gtk

clutter-gtk-1.0 installs its headers in: /usr/include/clutter-gtk-1.0/clutter-gtk

If while building, `-I/usr/include/clutter-1.0 -I/usr/include/clutter-gtk-1.0` is passed, gcc will pick up clutter-gtk-0.10's headers, and the build will fail.

I talked to clutter upstream, and the only solution we could come up with was to add clutter-gtk-1.0's path in the build order *first*. This means that the order of dependencies in champlain-gtk-0.10.pc is also important so that other packages don't have problems.

Attached below is a patch to change the order (Makefile + pkg-config file) so that clutter-gtk-1.0 is always added first, for complete safety.

Thanks!
Comment 1 Nirbheek Chauhan 2011-02-23 15:23:41 UTC 
Created attachment 181704 [details] [review]
Change depend order in pkg-config file, and CFLAG order in champlain-gtk/Makefile.am
Comment 2 Jiri Techet 2011-02-23 23:34:29 UTC 
Applied, thanks.