By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 1614

Summary: Build Issue: cmake doesn't defines HAVE_BACKTRACE
Product: TDE Reporter: Alexander Golubev (Fat-Zer) <fatzer2>
Component: tdelibsAssignee: Timothy Pearson <kb9vqf>
Status: RESOLVED FIXED    
Severity: normal CC: bugwatch, darrella, fatzer2, slavek.banko
Priority: P5    
Version: R14.0.0 [Trinity]   
Hardware: All   
OS: Linux   
Compiler Version: TDE Version String:
Application Version: Application Name:
Attachments: tdelibs-cmake-define-backtrace.patch

Description Alexander Golubev (Fat-Zer) 2013-08-08 17:41:44 CDT
Created attachment 1452 [details]
tdelibs-cmake-define-backtrace.patch

subj.
This is a regression comparing to autotools and impede thurther development/debug.
Comment 1 Darrell 2013-08-09 13:00:12 CDT
Alexander, how do I test this patch?
Comment 2 Alexander Golubev (Fat-Zer) 2013-08-09 13:39:26 CDT
(In reply to comment #1)
> Alexander, how do I test this patch?

It's a debugging feature, so testing will be kinda tricky (if you really want it)
1. Add to the function you are debugging something like:
   kdDebug() << kdBacktrace() << endl;
2. Enable the debug output:
   build without -DNDEBUG definition.
   Edit kdebugrc and enable debug InfoOutput for 0-area.
3. Without the patch you won't see the backtrace you supposed to. The patch fixes that.

Note: if you are running uncommon libc, then it may not provide backtrace(). So check CMakeCache or config.h if HAVE_BACKTRACE is set if you are.


On the other hand, you can just check that it doesn't break build and take my words, that it works ;)
Comment 3 Darrell 2013-08-09 16:14:02 CDT
I'll just accept your word. :-)

I already tested compiling. I see the following in my build logs:

...
-- Looking for backtrace
-- Looking for backtrace - found
...
//Have symbol backtrace
HAVE_BACKTRACE:INTERNAL=1


Tim, does the patch look sane and make sense? If yes, I'll push to git.
Comment 4 Darrell 2013-08-10 22:37:54 CDT
Patch pushed to git in commit 8abbad9d. Or perhaps 12976397. I don't know. Looks like I screwed up the comment and screwed up trying to amend the comment, but git was made for geeks and not normal people. Don't bother trying to explain to me. I just don't care!

Thanks for reporting and the patch!