Re: [Qemu-devel] [PATCH 2/2] docker: fedora: include more build dependencies

[Paolo Bonzini <pbonzini@redhat.com>]

On 16/03/2018 13:45, Daniel P. Berrangé wrote:
> On Fri, Mar 16, 2018 at 01:25:40PM +0100, Paolo Bonzini wrote:
>> Install optional dependencies of QEMU to get better coverage.
>>
>> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
>> ---
>>  tests/docker/dockerfiles/fedora.docker | 67 ++++++++++++++++++++++++++++------
>>  1 file changed, 55 insertions(+), 12 deletions(-)
>>
>> diff --git a/tests/docker/dockerfiles/fedora.docker b/tests/docker/dockerfiles/fedora.docker
>> index be3566325c..6d810f6a92 100644
>> --- a/tests/docker/dockerfiles/fedora.docker
>> +++ b/tests/docker/dockerfiles/fedora.docker
>> @@ -1,18 +1,61 @@
>>  FROM fedora:27
>>  ENV PACKAGES \
>>      ccache gettext git tar PyYAML sparse flex bison python3 bzip2 hostname \
>> -    glib2-devel pixman-devel zlib-devel SDL-devel libfdt-devel \
>> -    gcc gcc-c++ llvm clang make perl which bc findutils libaio-devel \
>> -    nettle-devel libasan libubsan \
>> -    mingw32-pixman mingw32-glib2 mingw32-gmp mingw32-SDL2 mingw32-pkg-config \
>> -    mingw32-gtk3 mingw32-gnutls mingw32-nettle mingw32-libtasn1 \
>> -    mingw32-libjpeg-turbo mingw32-libpng mingw32-curl mingw32-libssh2 \
>> -    mingw32-bzip2 \
>> -    mingw64-pixman mingw64-glib2 mingw64-gmp mingw64-SDL2 mingw64-pkg-config \
>> -    mingw64-gtk3 mingw64-gnutls mingw64-nettle mingw64-libtasn1 \
>> -    mingw64-libjpeg-turbo mingw64-libpng mingw64-curl mingw64-libssh2 \
>> -    mingw64-bzip2
>> -ENV QEMU_CONFIGURE_OPTS --python=/usr/bin/python3
>> +    gcc gcc-c++ llvm clang make perl which bc findutils libasan libubsan \
>> +    alsa-lib-devel \
>> +    bluez-libs-devel \
>> +    brlapi-devel \
>> +    bzip2-devel mingw32-bzip2 mingw64-bzip2 \
>> +    cyrus-sasl-devel \
>> +    device-mapper-multipath-devel \
>> +    glib2-devel mingw32-glib2 mingw64-glib2 \
> 
> No need for glib as that's pulled in automatically by gtk

We use it, so (unlike nettle-devel and nss-devel) I think it should be
added anyway.

>> +    nettle-devel mingw32-nettle mingw64-nettle \
> 
> nettle should be pulled in automatically by gnutls
> if it were needed - we automatically link against
> whatever gnutls pulls in.
> 
>> +    nss-devel \
> 
> nss-devel is not required by anything.

Yeah, libcacard uses it but we install its own devel package.

Paolo