On Wed, 22 Oct 2025 10:19:58 -0600 Tom Rini <[email protected]> wrote:
> On Wed, Oct 22, 2025 at 06:13:00PM +0200, Kory Maincent wrote: > > > From: "Kory Maincent (TI.com)" <[email protected]> > > > > 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) <[email protected]> > > --- > > 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
