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.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to