[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 08/12] docker: expand debian-amd64 image to include tag to
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v2 08/12] docker: expand debian-amd64 image to include tag tools |
Date: |
Thu, 14 Jan 2021 17:55:47 +0000 |
User-agent: |
Mutt/1.14.6 (2020-07-11) |
On Thu, Jan 14, 2021 at 06:53:36PM +0100, Philippe Mathieu-Daudé wrote:
> On 1/14/21 6:52 PM, Daniel P. Berrangé wrote:
> > On Thu, Jan 14, 2021 at 06:50:36PM +0100, Philippe Mathieu-Daudé wrote:
> >> Hi Alex,
> >>
> >> On 1/14/21 5:57 PM, Alex Bennée wrote:
> >>> This is going to be helpful when we want to both test the tool
> >>> integration and in the case of global generate a xref doc build.
> >>>
> >>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> >>> ---
> >>> tests/docker/dockerfiles/debian-amd64.docker | 5 ++++-
> >>> 1 file changed, 4 insertions(+), 1 deletion(-)
> >>>
> >>> diff --git a/tests/docker/dockerfiles/debian-amd64.docker
> >>> b/tests/docker/dockerfiles/debian-amd64.docker
> >>> index 55075d9fce..a98314757d 100644
> >>> --- a/tests/docker/dockerfiles/debian-amd64.docker
> >>> +++ b/tests/docker/dockerfiles/debian-amd64.docker
> >>> @@ -1,7 +1,7 @@
> >>> #
> >>> # Docker x86_64 target
> >>> #
> >>> -# This docker target builds on the debian Stretch base image. Further
> >>> +# This docker target builds on the Debian Buster base image. Further
> >>> # libraries which are not widely available are installed by hand.
> >>> #
> >>> FROM qemu/debian10
> >>> @@ -14,7 +14,10 @@ RUN apt update && \
> >>> RUN apt update && \
> >>> DEBIAN_FRONTEND=noninteractive eatmydata \
> >>> apt install -y --no-install-recommends \
> >>> + cscope \
> >>> genisoimage \
> >>> + exuberant-ctags \
> >>> + global \
> >>
> >> This clash with Daniel's series which seems ready (and harder
> >> to rebase):
> >>
> >> 'start using libvirt-ci's "lcitool" for dockerfiles'
> >> https://www.mail-archive.com/qemu-devel@nongnu.org/msg772839.html
> >>
> >> Do you mind getting it merged before your patch?
> >
> > I don't mind either way. I'll have to get mappings for those three
> > packages added to libvirt-ci, so that we pull them in on the other
> > non-debian distros too. Aside from that it is no big deal.
>
> Thanks for the quick feedback :)
Actually my answer was a tangent. My series doesn't touch the
debian dockerfile(s) at all, only ubuntu, fedora, centos & suse.
I've left debian untouched because QEMU's handling that with layering
so it needs further consideration wrt libvirt-ci auto-generation.
IOW, there is no clash between these series
> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
[PATCH v2 04/12] Add newline when generating Dockerfile, Alex Bennée, 2021/01/14
[PATCH v2 05/12] Makefile: wrap ctags in quiet-command calls, Alex Bennée, 2021/01/14
[PATCH v2 07/12] Makefile: wrap cscope in quiet-command calls, Alex Bennée, 2021/01/14