Re: [Qemu-devel] [PATCH] docker/dockerfiles/debian-s390-cross: include clang

2017-03-07 Thread Fam Zheng
On Tue, 03/07 13:41, Alex Bennée wrote: > > Fam Zheng 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.docke

Re: [Qemu-devel] [PATCH] docker/dockerfiles/debian-s390-cross: include clang

2017-03-07 Thread Alex Bennée
Fam Zheng 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 w

Re: [Qemu-devel] [PATCH] docker/dockerfiles/debian-s390-cross: include clang

2017-03-07 Thread Alex Bennée
Philippe Mathieu-Daudé writes: > Hi Alex, Fam. > > I tested this way but got an error: > > qemu (docker_alex)$ make docker-test-clang@debian-s390x-cross > BUILD debian-s390x-cross > make[1]: Entering directory 'qemu' > ARCHIVE qemu.tgz > ARCHIVE dtc.tgz > ARCHIVE pixman.tgz > COPY

Re: [Qemu-devel] [PATCH] docker/dockerfiles/debian-s390-cross: include clang

2017-03-07 Thread Philippe Mathieu-Daudé
Hi Alex, Fam. I tested this way but got an error: qemu (docker_alex)$ make docker-test-clang@debian-s390x-cross BUILD debian-s390x-cross make[1]: Entering directory 'qemu' ARCHIVE qemu.tgz ARCHIVE dtc.tgz ARCHIVE pixman.tgz COPYRUNNER RUN test-clang in qemu:debian-s390x-cross

Re: [Qemu-devel] [PATCH] docker/dockerfiles/debian-s390-cross: include clang

2017-03-07 Thread Fam Zheng
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 coll

[Qemu-devel] [PATCH] docker/dockerfiles/debian-s390-cross: include clang

2017-03-06 Thread Alex Bennée
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