Re: [PATCH v7 00/16] New perf ilist app

[Arnaldo Carvalho de Melo]

On Wed, Jul 23, 2025 at 04:08:55PM -0300, Arnaldo Carvalho de Melo wrote:
> On Wed, Jul 23, 2025 at 11:00:18AM -0700, Namhyung Kim wrote:
> > > I think there is some follow up for "make install" for scripts like
> > > these, but I'm keen for the python API to move forward.
> >  
> > I'll review the series today so that we can get some part of it, at
> > least.  Basically I think we need a wrapper script like perf-ilist to
> > run this easily (maybe with documentation).
> 
> I just tried, with the series applied:
> 
> root@number:~# perf ilist
> perf: 'ilist' is not a perf-command. See 'perf --help'.
> 
> Did you mean this?
> 	list
> root@number:~#
> 
> Now trying to figure out why it is not running.

So it is not wired up like 'perf archive', trying it directly:

root@number:~# ~acme/git/perf-tools-next/tools/perf/python/ilist.py 
Traceback (most recent call last):
  File "/home/acme/git/perf-tools-next/tools/perf/python/ilist.py", line 11, in <module>
    from textual import on
ModuleNotFoundError: No module named 'textual'
root@number:~#

I thought there was some discussion about catching this exception and
providing guidance, lemme try...

- Arnaldo