Re: [PATCH v4 07/15] perf tools report: Use less for scripts output

From: Feng Tang
Date: Sat Mar 09 2019 - 02:50:13 EST


On Fri, Mar 08, 2019 at 10:47:54AM -0300, Arnaldo Carvalho de Melo wrote:
> Em Tue, Mar 05, 2019 at 06:47:50AM -0800, Andi Kleen escreveu:
> > From: Andi Kleen <ak@xxxxxxxxxxxxxxx>
> >
> > The UI viewer for scripts output has a lot of limitations: limited size,
> > no search or save function, slow, and various other issues.
> >
> > Just use 'less' to display directly on the terminal instead.
>
> I'm ok with this, CCing Feng tho since he contributed this browser, to
> let him know.

Thanks for the note!

>
> - Arnaldo
>
> > This won't work in gtk mode, but gtk doesn't support these
> > context menus anyways. If that is ever done could use an terminal
> > for the output.
> >
> > Signed-off-by: Andi Kleen <ak@xxxxxxxxxxxxxxx>
> >

Looks good to me.

Acked-by: Feng Tang <feng.tang@xxxxxxxxx>

Thanks,
Feng