[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] docker/dockerfiles/debian-s390-cross: include c
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH] docker/dockerfiles/debian-s390-cross: include clang |
Date: |
Wed, 8 Mar 2017 14:15:31 +0800 |
User-agent: |
Mutt/1.7.1 (2016-10-04) |
On Tue, 03/07 13:41, Alex Bennée wrote:
>
> Fam Zheng <address@hidden> writes:
>
> > On Mon, 03/06 11:28, Alex Bennée wrote:
> >> It's a silly little limitation on Shippable that is looks for clang
> >> in the container even though we won't use it. The arm/aarch64 cross
> >> builds inherit this from debian.docker but as we needed to use
> >> debian-testing for this we add it here. We also collapse the update
> >> step into one RUN line to remove and intermediate layer of the docker
> >> build.
> >>
> >> Signed-off-by: Alex Bennée <address@hidden>
> >> ---
> >> tests/docker/dockerfiles/debian-s390x-cross.docker | 4 ++--
> >> 1 file changed, 2 insertions(+), 2 deletions(-)
> >>
> >> diff --git a/tests/docker/dockerfiles/debian-s390x-cross.docker
> >> b/tests/docker/dockerfiles/debian-s390x-cross.docker
> >> index bbb21ed088..3a687feda0 100644
> >> --- a/tests/docker/dockerfiles/debian-s390x-cross.docker
> >> +++ b/tests/docker/dockerfiles/debian-s390x-cross.docker
> >> @@ -13,8 +13,8 @@ RUN cat /etc/apt/sources.list | sed "s/deb/deb-src/" >>
> >> /etc/apt/sources.list
> >> RUN dpkg --add-architecture s390x
> >>
> >> # Grab the updated list of packages
> >> -RUN apt update
> >> -RUN apt dist-upgrade -yy
> >> +RUN apt update && apt dist-upgrade -yy
> >> +RUN apt install -yy build-essential clang
> >> RUN apt-get build-dep -yy -a s390x qemu || apt-get -f install
> >> RUN apt install -yy gcc-multilib-s390x-linux-gnu binutils-multiarch
> >>
> >> --
> >> 2.11.0
> >>
> >
> > Looks sane. Is this for 2.9?
>
> Yes please. The s390x cross build on shippable is currently broken
> without it although we can use the container locally.
OK, I'll send a pull request soon.
Fam