[PATCH v1] ci: disable usage of libattr in Leap

[Peter Krempa]

On Mon, Jul 03, 2023 at 14:42:57 +0200, Olaf Hering wrote:
> Commit dffeef89ef ("meson: attr_dep switch to dependency()") made
> libattr.pc mandatory.

Could you explain why this is needed in the first place?

> Signed-off-by: Olaf Hering <olaf at aepfle.de>
> ---
>  ci/containers/opensuse-leap-15.Dockerfile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/ci/containers/opensuse-leap-15.Dockerfile b/ci/containers/opensuse-leap-15.Dockerfile
> index ca11edf808..4bfbe8057f 100644
> --- a/ci/containers/opensuse-leap-15.Dockerfile
> +++ b/ci/containers/opensuse-leap-15.Dockerfile
> @@ -37,7 +37,6 @@ RUN zypper update -y && \
>             kmod \
>             libacl-devel \
>             libapparmor-devel \
> -           libattr-devel \
>             libblkid-devel \
>             libcap-ng-devel \
>             libcurl-devel \

The top of this file says:

 # THIS FILE WAS AUTO-GENERATED
 #
 #  $ lcitool manifest ci/manifest.yml
 #
 # https://gitlab.com/libvirt/libvirt-ci


You need to patch lcitool data in the libvirt-ci project if that library
for some reason is not available. Otherwise next-regen of the files
would simply revert your change.

> @@ -95,3 +94,4 @@ ENV LANG "en_US.UTF-8"
>  ENV MAKE "/usr/bin/make"
>  ENV NINJA "/usr/bin/ninja"
>  ENV PYTHON "/usr/bin/python3"
> +ENV MESON_OPTS "-Dattr=disabled"
>