On Tuesday 09 December 2014 23:51:04 Albert Astals Cid wrote:
> El Dimarts, 9 de desembre de 2014, a les 02:15:58, Milian Wolff va escriure:
> > On Wednesday 03 December 2014 01:51:57 Aleix Pol wrote:
> > > On Tue, Dec 2, 2014 at 6:59 PM, Milian Wolff <m...@milianw.de> wrote:
> > > > Hey all,
> > > >
> > > > I have just finished writing a lengthy introduction to heaptrack, an
> > > > alternative to Massif, see:
> > > >
> > > > http://milianw.de/blog/heaptrack-a-heap-memory-profiler-for-linux
> > > >
> > > > I'd like to see more people starting to use it. Any feedback, or even
> > > > patches,
> > > > is welcome!
> > > >
> > > > Especially, I'd love to see more people use it on their pet project in
> > > > KDE.
> > > > Quite often, you'll find useless temporary allocations, overly large
> > > > memory
> > > > consumption or even significant memory leaks. C++/Qt/KDE code can be
> > > > extremely
> > > > efficient, but you have to code accordingly. If you have any questions
> > > > to
> > > > the
> > > > results you obtain from heaptrack, or how to improve the situation,
> > > > don't
> > > > hesitate to ask me. Please ask on a public mailing list, such that
> > > > others
> > > > can
> > > > benefit from the discussion as well.
> > > >
> > > > Furthermore, I hereby request an official code review. Heaptrack
> > > > currently
> > > > lives in a scratch repository:
> > > >
> > > > http://quickgit.kde.org/?p=scratch%2Fmwolff%2Fheaptrack.git
> > > >
> > > > I want to move this to extragear, skipping playground altogether, if
> > > > possible.
> > >
> > > Cool stuff! I'll definitely give it a try!
> > >
> > > Regarding the move to extragear, you can start requesting the repository
> > > to
> > > get it in playground and then request the move to kdereview, if you're
> > > confident it's pristine. ;)
> >
> > Heaptrack is now in kdereview, I'd like to invite everyone to review my
> > code at the new location:
> >
> > git clone kde:heaptrack
>
> Doesn't link here.
>
> Linking CXX executable heaptrack_print
> CMakeFiles/heaptrack_print.dir/heaptrack_print.cpp.o: In function `main':
> heaptrack_print.cpp:(.text+0x3f4f): undefined reference to
> `boost::program_options::options_description::m_default_line_length'
<snip>
Can you update and try again? I think I was missing the REQUIRED in the
find_package statements, which should have triggered an early-return. You
don't have Boost installed, do you?
Hm, but thinking about it - you seem to have the headers around, otherwise it
wouldn't even get that far. So something is fishy with your system, compared
to mine. Can you debug this a bit please?
make heaptrack_print VERBOSE=1 |& tee make_log.txt
-> esp. the linker invocation would be interesting, it should be something
like
/usr/lib/icecream/libexec/icecc/bin/g++ -std=c++11 -O2 -g -DNDEBUG
CMakeFiles/heaptrack_print.dir/heaptrack_print.cpp.o -o heaptrack_print -
rdynamic -lboost_iostreams -lboost_program_options
If this does not include the -lboost_* stuff, can you try to add
message(FATAL_ERROR "Boost libs: ${Boost_LIBRARIES}")
below the find_package in the CMakeLists.txt? What CMake do you use, btw.? And
maybe play with the following options, and toggle them on/off:
set(Boost_USE_STATIC_LIBS OFF)
set(Boost_USE_MULTITHREADED ON)
set(Boost_USE_STATIC_RUNTIME OFF)
(see
http://stackoverflow.com/questions/6646405/how-do-you-add-boost-libraries-in-cmakelists-txt)
Bye
--
Milian Wolff
m...@milianw.de
http://milianw.de
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
