[v1] tests/docker: start using libvirt-ci's "lcitool" for dockerfiles

[PATCH 09/23] tests/docker: fix mistakes in fedora package list

Posted by Daniel P. Berrangé 4 years, 11 months ago

llvm is not required by QEMU, just clang.

libblockdev-mpath-devel is not used by QEMU, rather it wants
device-mapper-multipath-devel.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 tests/docker/dockerfiles/fedora.docker | 2 --
 1 file changed, 2 deletions(-)

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 \
-- 
2.28.0

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

Posted by Philippe Mathieu-Daudé 4 years, 11 months ago

On 12/1/20 6:18 PM, 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.
> 
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
>  tests/docker/dockerfiles/fedora.docker | 2 --
>  1 file changed, 2 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>