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

Bug #485889 reported by Jan on 2009-11-20
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openWNS SDK
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  Edit
Everyone can see this information.

Other bug subscribers