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 316530 - gthread configure test breaks if building uninstalled
gthread configure test breaks if building uninstalled
Status: RESOLVED FIXED
Product: bonobo
Classification: Deprecated
Component: libbonobo
unspecified
Other All
: High normal
: ---
Assigned To: Michael Meeks
bonobo qa
Depends on:
Blocks:
 
 
Reported: 2005-09-16 22:51 UTC by Brian Cameron
Modified: 2006-08-10 13:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch to ensure that gthread configure test is only liked with gthread compiler/linker options (847 bytes, patch)
2005-09-16 22:52 UTC, Brian Cameron
none Details | Review
a better patch that uses GTHREAD instead of GOBJECT in variable names. (847 bytes, patch)
2005-09-16 22:54 UTC, Brian Cameron
none Details | Review

Description Brian Cameron 2005-09-16 22:51:52 UTC
If building libbonobo with an installed glib but an uninstalled ORBit2, then
the configure script breaks when checking for gthread support.  This is because
the uninstalled.pc -I and -L flags get included when trying to compile the 
conftest.c program.

It seems better to just compile the gthread test using the gthread includes and
libraries to avoid this problem.  Providing an attachment.
Comment 1 Brian Cameron 2005-09-16 22:52:29 UTC
Created attachment 52329 [details] [review]
patch to ensure that gthread configure test is only liked with gthread compiler/linker options
Comment 2 Brian Cameron 2005-09-16 22:54:09 UTC
Created attachment 52330 [details] [review]
a better patch that uses GTHREAD instead of GOBJECT in variable names.
Comment 3 Brent Smith (smitten) 2005-09-18 05:07:57 UTC
Marking NEW to get some attention for the patch.
Comment 4 Kjartan Maraas 2006-08-07 10:09:33 UTC
Marking the first patch as obsolete. Michael, is the new one ok to commit?
Comment 5 Michael Meeks 2006-08-07 10:59:48 UTC
looks fine to me - please do go for it.
Comment 6 Kjartan Maraas 2006-08-10 13:44:15 UTC
Commited.