Re: [PATCH 0/2] perf tools: Basic bash completion support

[David Ahern]

On 8/7/12 7:22 AM, Frederic Weisbecker wrote:
> On Tue, Aug 07, 2012 at 03:19:44PM +0200, Frederic Weisbecker wrote:
> > Hey,
> >
> > Basic bash completion support. Only support perf subcommands and most -e basic
> > event descriptor (no grouping).
> >
> > I just have a small issue with tracepoints because of their ":" in the middle.
> > It auto completes as long as we haven't yet reached the semicolon. Otherwise
> > we need to add a double quote in the beginning of the expression. I'm quite
> > a newbie in bash completion though, so I might find a subtelty later to solve
> > this.
>
> Tips: for testing, you need to "make install" and update the bash completion
> scripts:
>
> 	# make install
> 	$ . /etc/bash_completion

ANd you need to make sure the PATH hits the updated binary and not the 
default other wise you end up with:

/tmp/pbuild/perf recUnknown option: --list-cmds

 Usage: perf [--version] [--help] COMMAND [ARGS]
Unknown option: --list-cmds

It's calling /usr/bin/perf with --list-cmds, versus the perf command I 
am running (/tmp/pbuild/perf). Any way to teach the completion to use 
the perf binary that the user is running?

David
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/