[PATCH] test: py: Add pyelftools dependency for test_event_dump

[Kory Maincent]

On Wed, 22 Oct 2025 10:19:58 -0600
Tom Rini <trini at konsulko.com> wrote:

> On Wed, Oct 22, 2025 at 06:13:00PM +0200, Kory Maincent wrote:
> 
> > From: "Kory Maincent (TI.com)" <kory.maincent at bootlin.com>
> > 
> > The test_event_dump test requires pyelftools to run.
> > Add this missing dependency to requirements.txt to ensure the test can run
> > successfully.
> > 
> > Signed-off-by: Kory Maincent (TI.com) <kory.maincent at bootlin.com>
> > ---
> >  test/py/requirements.txt | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/test/py/requirements.txt b/test/py/requirements.txt
> > index 804a427b351..6d7909cb919 100644
> > --- a/test/py/requirements.txt
> > +++ b/test/py/requirements.txt
> > @@ -3,3 +3,4 @@ pycryptodomex==3.21.0
> >  pytest==6.2.5
> >  pytest-xdist==2.5.0
> >  FATtools==1.0.42
> > +pyelftools==0.32  
> 
> So with commit 859621b47f9e ("python: Recreate test/py and
> tools/buildman requirements.txt files") I re-created this
> file and pyelftools was removed. I think what's wrong is that it's not
> documented that you need tools/binman/requirements.txt installed because
> test_event_dump uses scripts/event_dump.py and that imports elf from
> binman and that in turn needs pyelftools.
 
Ok, so it is rather a documentation change needed here.

Regards,
-- 
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com