[PATCH] test/py: Re-add setuptools to requirements.txt

[Javier Martinez Canillas]

Tom Rini <trini at konsulko.com> writes:

Hello Tom,

> On Mon, Jun 16, 2025 at 08:53:32PM +0200, Javier Martinez Canillas wrote:
>
>> The commit 859621b47f9e ("python: Recreate test/py and tools/buildman
>> requirements.txt files") recreated the python requirements but dropped
>> the setuptools module that is used by the test harness.
>> 
>> This leads to the following error when trying to run a test:
>> 
>>     pytest --build --bd sandbox test/py/tests/test_net.py
>>     +make O=/home/javier/devel/u-boot/build-sandbox -s sandbox_defconfig
>>     +make O=/home/javier/devel/u-boot/build-sandbox -s -j8
>>     Traceback (most recent call last):
>>       File "/home/javier/devel/u-boot/build-sandbox/../scripts/dtc/pylibfdt/setup.py", line 23, in <module>
>>         from setuptools import setup, Extension
>>     ModuleNotFoundError: No module named 'setuptools'
>>     make[3]: *** [../scripts/dtc/pylibfdt/Makefile:33: rebuild] Error 1
>>     make[2]: *** [../scripts/Makefile.build:398: scripts/dtc/pylibfdt] Error 2
>>     make[1]: *** [/home/javier/devel/u-boot/Makefile:2079: scripts_dtc] Error 2
>>     make: *** [Makefile:177: sub-make] Error 2
>>     Exit code: 2
>> 
>> Fixes: 859621b47f9e ("python: Recreate test/py and tools/buildman requirements.txt files")
>> Signed-off-by: Javier Martinez Canillas <javierm at redhat.com>
>
> It's not pytest that requires setup tools, it's pylibfdt. You're not
> able to build sandbox, is the problem. And that's a documented
> requirement for building U-Boot in general for a number of platforms.
> This doesn't belong in test/py/requirements.txt which is why the "make a
> requirements.txt" tooling drops it.
>

Got it. Sorry for the noise then.

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat