Re: [PATCH 1/5] docker: Remove 32-bit MIPS toolchain from debian-all-test image

["Philippe Mathieu-Daudé" <philmd@linaro.org>]

On 22/8/25 16:49, Philippe Mathieu-Daudé wrote:
> On 22/8/25 13:23, Alex Bennée wrote:
>> Philippe Mathieu-Daudé <philmd@linaro.org> writes:
>>
>>> In commit d3322023bfe ("configure: unify again the case arms in
>>> probe_target_compiler") we lost coverage of 32-bit MIPS with the
>>> debian-all-test image. No need to keep installing the toolchain.
>>>
>>> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
>>> ---
>>>   tests/docker/dockerfiles/debian-all-test-cross.docker | 4 ----
>>>   1 file changed, 4 deletions(-)
>>>
>>> diff --git a/tests/docker/dockerfiles/debian-all-test-cross.docker b/ 
>>> tests/docker/dockerfiles/debian-all-test-cross.docker
>>> index 420a4e33e60..bc74d65a634 100644
>>> --- a/tests/docker/dockerfiles/debian-all-test-cross.docker
>>> +++ b/tests/docker/dockerfiles/debian-all-test-cross.docker
>>> @@ -40,14 +40,10 @@ ENV AVAILABLE_COMPILERS gcc-aarch64-linux-gnu \
>>>           libc6-dev-arm64-cross \
>>>           gcc-arm-linux-gnueabihf \
>>>           libc6-dev-armhf-cross \
>>> -        gcc-mips-linux-gnu \
>>> -        libc6-dev-mips-cross \
>>>           gcc-mips64-linux-gnuabi64 \
>>>           libc6-dev-mips64-cross \
>>>           gcc-mips64el-linux-gnuabi64 \
>>>           libc6-dev-mips64el-cross \
>>> -        gcc-mipsel-linux-gnu \
>>> -        libc6-dev-mipsel-cross \
>>>           gcc-powerpc64le-linux-gnu \
>>>           libc6-dev-ppc64el-cross \
>>>           gcc-riscv64-linux-gnu \
>>
>> You also need to remove the references in configure:
>>

Expanding:

     case $target_arch in

>>        hppa|m68k|mips|riscv64|sparc64)
>>          container_image=debian-all-test-cross
>>          ;;
> 
> OK.

So actual patch is correct here.

> 
>>        mips64)
>>          container_image=debian-all-test-cross
>>          container_cross_prefix=mips64-linux-gnuabi64-
>>          ;;
> 
> We aren't removing mips64 yet. We will but since the reason is
> different, it will be in another series.