Messages in this thread Patch in this message | ![/](/images/icornerl.gif) | | From | Yunlong Song <> | Subject | [PATCH 11/13] perf tools: Add the bash completion for listing subsubcommands of perf timechart | Date | Wed, 18 Mar 2015 21:35:56 +0800 |
| |
The bash completion does not support listing subsubcommands for 'perf timechart <TAB>', so fix it.
Example:
Before this patch:
$ perf timechart <TAB> $
As shown above, the subsubcommands of perf timechart does not come out.
After this patch:
$ perf timechart <TAB> record
As shown above, the subsubcommands of perf timechart can come out now.
Signed-off-by: Yunlong Song <yunlong.song@huawei.com> --- tools/perf/builtin-timechart.c | 7 ++++--- tools/perf/perf-completion.sh | 2 +- 2 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/tools/perf/builtin-timechart.c b/tools/perf/builtin-timechart.c index 51440d1..494b3bb 100644 --- a/tools/perf/builtin-timechart.c +++ b/tools/perf/builtin-timechart.c @@ -1958,7 +1958,8 @@ int cmd_timechart(int argc, const char **argv, parse_time), OPT_END() }; - const char * const timechart_usage[] = { + const char * const timechart_subcommands[] = { "record", NULL }; + const char *timechart_usage[] = { "perf timechart [<options>] {record}", NULL }; @@ -1976,8 +1977,8 @@ int cmd_timechart(int argc, const char **argv, "perf timechart record [<options>]", NULL }; - argc = parse_options(argc, argv, timechart_options, timechart_usage, - PARSE_OPT_STOP_AT_NON_OPTION); + argc = parse_options_subcommand(argc, argv, timechart_options, timechart_subcommands, + timechart_usage, PARSE_OPT_STOP_AT_NON_OPTION); if (tchart.power_only && tchart.tasks_only) { pr_err("-P and -T options cannot be used at the same time.\n"); diff --git a/tools/perf/perf-completion.sh b/tools/perf/perf-completion.sh index 35f43fd..a33d2ef5 100644 --- a/tools/perf/perf-completion.sh +++ b/tools/perf/perf-completion.sh @@ -143,7 +143,7 @@ __perf_main () __perfcomp_colon "$evts" "$cur" else # List subcommands for perf commands - if [[ $prev_skip_opts == @(kvm|kmem|mem|lock|sched|data|help|script|test) ]]; then + if [[ $prev_skip_opts == @(kvm|kmem|mem|lock|sched|data|help|script|test|timechart) ]]; then subcmds=$($cmd $prev_skip_opts --list-cmds) __perfcomp_colon "$subcmds" "$cur" fi -- 1.8.5.2
| ![\](/images/icornerr.gif) |