Re: [PATCH v2 09/25] tests/docker: fix mistakes in fedora package list

[Wainer dos Santos Moschetta]

On 1/14/21 10:02 AM, Daniel P. Berrangé wrote:
> llvm is not required by QEMU, just clang.
>
> libblockdev-mpath-devel is not used by QEMU, rather it wants
> device-mapper-multipath-devel.
>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
>   tests/docker/dockerfiles/fedora.docker | 2 --
>   1 file changed, 2 deletions(-)


Reviewed-by: Wainer dos Santos Moschetta <wainersm@redhat.com>


> diff --git a/tests/docker/dockerfiles/fedora.docker 
> b/tests/docker/dockerfiles/fedora.docker
> index 03b88f1cfe..754b381e1e 100644
> --- a/tests/docker/dockerfiles/fedora.docker
> +++ b/tests/docker/dockerfiles/fedora.docker
> @@ -27,7 +27,6 @@ ENV PACKAGES \
>       libaio-devel \
>       libasan \
>       libattr-devel \
> -    libblockdev-mpath-devel \
>       libcacard-devel \
>       libcap-ng-devel \
>       libcurl-devel \
> @@ -45,7 +44,6 @@ ENV PACKAGES \
>       libusbx-devel \
>       libxml2-devel \
>       libzstd-devel \
> -    llvm \
>       lzo-devel \
>       make \
>       mingw32-bzip2 \