qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] tests/docker: Use lower case for centos8 powertools


From: Alex Bennée
Subject: Re: [PATCH] tests/docker: Use lower case for centos8 powertools
Date: Fri, 18 Dec 2020 10:08:47 +0000
User-agent: mu4e 1.5.7; emacs 28.0.50

Jiaxun Yang <jiaxun.yang@flygoat.com> writes:

> Our gitlab amd64-centos8-container pipeline constantly fail at:
>
> 15.36 Error: No matching repo to modify: PowerTools.
>
> Fix it by convert it to lower case.
>
> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>

Thanks for you patch, we already have a similar fix in flight this PR:

  Subject: [PULL v2 00/11] testing and configure updates
  Date: Thu, 17 Dec 2020 09:43:30 +0000
  Message-Id: <20201217094330.17400-1-alex.bennee@linaro.org>

> ---
>  tests/docker/dockerfiles/centos8.docker | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/docker/dockerfiles/centos8.docker 
> b/tests/docker/dockerfiles/centos8.docker
> index 54bc6d54cd..06b67962fd 100644
> --- a/tests/docker/dockerfiles/centos8.docker
> +++ b/tests/docker/dockerfiles/centos8.docker
> @@ -31,6 +31,6 @@ ENV PACKAGES \
>      zlib-devel
>  
>  RUN dnf install -y dnf-plugins-core && \
> -  dnf config-manager --set-enabled PowerTools && \
> +  dnf config-manager --set-enabled powertools && \
>    dnf install -y $PACKAGES
>  RUN rpm -q $PACKAGES | sort > /packages.txt


-- 
Alex Bennée



reply via email to

[Prev in Thread] Current Thread [Next in Thread]