[PATCH RFC next 02/18] CI: pass clang HOSTCC and CC to pytest

[Quentin Schulz]

Hi Tom,

On 5/4/26 8:05 PM, Tom Rini wrote:
> On Mon, May 04, 2026 at 03:20:29PM +0200, Quentin Schulz wrote:
>> From: Quentin Schulz <quentin.schulz at cherry.de>
>>
>> test_env_initial_env_file test calls make on the build directory passed
>> with --build-dir parameter which points at UBOOT_TRAVIS_BUILD_DIR which
>> is populated with buildman before pytest is run. For "sandbox with clang
>> test.py" and "sandbox64 with clang test.py" jobs, buildman uses clang-20
>> toolchain, but pytest isn't configured to use that as well, resulting in
>> gcc being used with a configuration made with clang-20 toolchain.
>>
>> Specify the HOSTCC and CC clang make arguments so pytest uses the proper
>> toolchain, this will be required in a few commits when we'll upgrade to
>> a newer version of scripts/Makefile.extrawarn which gets rid of checking
>> whether a clang flag exists (because it's supported by the min clang
>> version the kernel advertises as being supported) which will be an issue
>> since GCC for sure doesn't support all clang flags.
>>
>> Signed-off-by: Quentin Schulz <quentin.schulz at cherry.de>
>> ---
>>   .gitlab-ci.yml | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
>> index 245e422d72f..d0ebd9f123b 100644
>> --- a/.gitlab-ci.yml
>> +++ b/.gitlab-ci.yml
>> @@ -288,6 +288,7 @@ sandbox with clang test.py:
>>       - ${HOST}
>>     variables:
>>       TEST_PY_BD: "sandbox"
>> +    TEST_PY_EXTRA: "--make-args HOSTCC=clang-20 --make-args CC=clang-20"
> 
> Is ""--make-args 'HOSTCC=clang-20 CC=clang-20'" not parsed correctly ?
> 

Well, this will make 'HOSTCC=clang-20 CC=clang-20' a single argument no?

Try:
make 'HOSTCC=clang CC=clang'

Makefile doesn't like it.

If you're complaining about the need for duplicating --make-args, it's a 
choice I made when selecting the "append" action for that parameter, 
c.f. https://docs.python.org/3/library/argparse.html#action.

We could use the "extend" action with nargs=* or +, but I personally 
hate those. It'd look like something like that:
--make-args HOSTCC=clang-20 CC=clang-20

It requires you to be aware of the special "--" argument to stop parsing 
of command options and start parsing positional arguments.
Cheers,
Quentin