Re: [Xen-devel] [PATCH v2 2/4] automation: install some packages for CentOS from EPEL

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Mon, Jul 09, 2018 at 11:46:59AM -0500, Doug Goldstein wrote:
> On Mon, Jul 09, 2018 at 04:32:14PM +0100, Wei Liu wrote:
> > They are needed to build tools.
> > 
> > Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx>
> > ---
> >  automation/build/centos/7.2.dockerfile | 6 ++++++
> >  1 file changed, 6 insertions(+)
> > 
> > diff --git a/automation/build/centos/7.2.dockerfile 
> > b/automation/build/centos/7.2.dockerfile
> > index 34eac9abd4..f46a5e3f2b 100644
> > --- a/automation/build/centos/7.2.dockerfile
> > +++ b/automation/build/centos/7.2.dockerfile
> > @@ -6,6 +6,9 @@ LABEL maintainer.name="The Xen Project" \
> >  # the version we want
> >  COPY CentOS-7.2.repo /etc/yum.repos.d/CentOS-Base.repo
> >  
> > +# install EPEL for dev86, xz-devel and possibly other packages
> > +RUN yum -y install 
> > https://dl.fedoraproject.org/pub/epel/epel-release-latest-7.noarch.rpm
> 
> "&& yum clean all" to keep the layer size down
> 
> With that added in there (unless Acked-by is more appropriate):

Either is fine.

> 
> Reviewed-by: Doug Goldstein <cardoe@xxxxxxxxxx>

Thanks.

Wei.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel