Re: [PATCH v2 05/11] tests/lcitool: add nettle to the minimal deps

["Alex Bennée" <alex.bennee@linaro.org>]

Thomas Huth <thuth@redhat.com> writes:

> On 04/12/2025 20.48, Alex Bennée wrote:
>> Technically we don't need a crypto library to do the base build but
>> I couldn't see an easy way to skip iotests which would otherwise fail.
>> Besides libnettle is a fairly small library even if its not the
>> fastest crypto implementation.
>
> I think we added checks to some of the iotests in the past already to
> skip if the crypto stuff is not available? So I think it might be
> better to fix the related iotests instead.
>
> Which test is failing for you?

I did have 049 qcow failing and two others but I can't remember exactly
which combo it was. The error messages where all about missing crypto
functions stopping the run.

Where are the crypto library tests done? I couldn't find them.

But I'm happy to drop this patch until we find out:

  https://gitlab.com/stsquad/qemu/-/pipelines/2197896632

>
>  Thomas
>
>
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>> ---
>>   tests/lcitool/projects/qemu-minimal.yml | 1 +
>>   1 file changed, 1 insertion(+)
>> diff --git a/tests/lcitool/projects/qemu-minimal.yml
>> b/tests/lcitool/projects/qemu-minimal.yml
>> index 2a1ac51d402..eebdc23e682 100644
>> --- a/tests/lcitool/projects/qemu-minimal.yml
>> +++ b/tests/lcitool/projects/qemu-minimal.yml
>> @@ -18,6 +18,7 @@ packages:
>>    - libffi
>>    - make
>>    - meson
>> + - nettle
>>    - ninja
>>    - pixman
>>    - pkg-config

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro