Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 353809 - Function Callgraph profile configuration does not support -vv and -vvv options
Summary: Function Callgraph profile configuration does not support -vv and -vvv options
Status: RESOLVED FIXED
Alias: None
Product: Linux Tools
Classification: Tools
Component: Call Graph (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 0.8.1   Edit
Assignee: Roland Grunberg CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-03 14:36 EDT by Otavio Pontes CLA
Modified: 2011-10-17 10:12 EDT (History)
2 users (show)

See Also:

Attachments
Patch to fix the reported problem (10.41 KB, patch)
2011-08-03 14:37 EDT, Otavio Pontes CLA 		no flags Details | Diff
Using a spinner instead of checkboxes (10.53 KB, patch)
2011-08-10 10:22 EDT, Otavio Pontes CLA 		rgrunber: iplog+
Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Otavio Pontes CLA 2011-08-03 14:36:08 EDT 
Build Identifier: M20110210-1200

There is a -v option to set the verbose mode for stap when running Function Callgraph profile. But there isn't a way to run stap with -vv or -vvv options. This options set a extra verbose mode.

Reproducible: Always

Steps to Reproduce:
1. Right-click a c/cpp project
2. Select 'Profile as' -> 'Profile Configurations'
3. Right click 'Profile with Systemtap' and click the 'New' option
4. Go to 'SystemTap options' tab
5. There is an option for -v but there isn't an option for -vv and -vvv
Comment 1 Otavio Pontes CLA 2011-08-03 14:37:50 EDT 
Created attachment 200844 [details]
Patch to fix the reported problem

This patch adds a -vv and a -vvv option to the profile configurations.
Comment 2 Roland Grunberg CLA 2011-08-08 09:47:11 EDT 
I have no issues with the idea, but I think a combo box or a spinner would be best to specify the verbosity level.
Comment 3 Otavio Pontes CLA 2011-08-10 10:22:00 EDT 
Created attachment 201244 [details]
Using a spinner instead of checkboxes

In this patch I use sppinners instead of checkboxes for selecting the verbosity level. It was really a better idea.
This patch is applying in master branch without the previous patch.
Comment 4 Roland Grunberg CLA 2011-08-11 11:20:01 EDT 
Commited as 40a1d66.
Comment 5 Andrew Overholt CLA 2011-08-11 12:13:02 EDT 
Roland, please be sure to set iplog+ to any patches from contributors that you apply.  Thanks :)
Comment 6 Roland Grunberg CLA 2011-08-12 09:12:28 EDT 
(Move to fixed)