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 501590 - Gossip doesn't build if loudmouth isn't compiled with debug symbols
Gossip doesn't build if loudmouth isn't compiled with debug symbols
Status: RESOLVED NOTABUG
Product: gossip
Classification: Deprecated
Component: General
0.28
Other All
: Normal normal
: ---
Assigned To: Gossip Maintainers
Gossip Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-12-04 19:27 UTC by Gilles Dartiguelongue
Modified: 2008-02-28 01:53 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
configure-lmnodebug.patch (703 bytes, patch)
2007-12-04 19:31 UTC, Gilles Dartiguelongue
rejected Details | Review
Fixes the bug in Loudmouth (1.23 KB, patch)
2007-12-28 10:52 UTC, Martyn Russell
reviewed Details | Review

Description Gilles Dartiguelongue 2007-12-04 19:27:22 UTC
gossip-0.28 and svn head don't compile if loudmouth wasn't compiled with --enable-debug.

This is because there is no way (besides doing it by hand) to set the LM_NO_DEBUG variable to compile gossip. This is a problem for us at gentoo (see https://bugs.gentoo.org/show_bug.cgi?id=196218)

Will attach a patch shortly.
Comment 1 Gilles Dartiguelongue 2007-12-04 19:31:26 UTC
Created attachment 100198 [details] [review]
configure-lmnodebug.patch

This patch offers a configure time option to set LM_NO_DEBUG.
It allows us to check for consistency in configure options.
Thanks for considering.
Comment 2 Martyn Russell 2007-12-28 10:50:52 UTC
Thanks for the patch!

The fix for this is not to change Gossip.
I have sent a patch to Micke to fix Loudmouth.
I will attach the patch here and then close the bug.
Comment 3 Martyn Russell 2007-12-28 10:52:15 UTC
Created attachment 101723 [details] [review]
Fixes the bug in Loudmouth
Comment 4 Martyn Russell 2007-12-28 10:54:14 UTC
Micke, just CC'ing you for comments if you have any and for access to this patch from a central location. This is the same patch I emailed you recently.

Closing bug.
Comment 5 Gilles Dartiguelongue 2007-12-28 11:45:36 UTC
that still leaves no way to set LM_NO_DEBUG in gossip other than by hand right ?
Comment 6 Mikael Hallendal 2008-02-28 01:53:59 UTC
Committed this for the 1.3.4 milestone in Loudmouth.