./playground.py install -help Usage outdated and missing debug symbols with callgrind

Bug #485889 reported by Jan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openWNS SDK
Fix Committed
Medium
Karsten Klagges

Bug Description

Since the last upgrade to SConstruct, playground.py install --help does not display the correct usage information how to build a binary for use with callgrind. It seems the correct syntax now is:

./playground.py install --flavour=opt --scons="callgrind=True"

Otherwise, the config/options.py file can be modified (the callgrind=True/False line has to be added) - is this documented anywhere?

Besides that, if callgrind=True is set, no debug symbols are included if the flavour is opt. As profiling with opt makes sense to me (it is the opt version that should run with optimal speed and profiling an unoptimized version makes profiling even slower), I would add the "-g" CXXFLAG for use with callgrind:

if env['callgrind']:
    env.Append(CXXFLAGS = ['-g'])

Cheers
Jan

Related branches

Changed in openwns-sdk:
assignee: nobody → Karsten Klagges (karsten.klagges)
status: New → Confirmed
Changed in openwns-sdk:
importance: Undecided → Medium
status: Confirmed → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.