All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: "Philippe Mathieu-Daudé" <f4bug@amsat.org>
Cc: cota@braap.org, famz@redhat.com, berrange@redhat.com,
	richard.henderson@linaro.org, balrogg@gmail.com,
	aurelien@aurel32.net, agraf@suse.de, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH for 3.0 02/10] docker: debian-tricore add git tools
Date: Thu, 12 Jul 2018 11:44:12 +0100	[thread overview]
Message-ID: <87pnzsd98j.fsf@linaro.org> (raw)
In-Reply-To: <1de61649-ac83-234a-0d97-3f9f4b51767e@amsat.org>


Philippe Mathieu-Daudé <f4bug@amsat.org> writes:

> On 07/09/2018 12:21 PM, Alex Bennée wrote:
>> As we check out we need git installed before we start. We would have
>> had this based on the heavier qemu:debian9 image but we only use plain
>> debian:9 here.
>
> I first thought "weird, I checked this already" then noticed you rebased
> on debian:9.
> 'stretch-slim' is an alias for '9-slim' and should be tinier (the
> previous RFC image is 323MB).

Hmm why not just switch to qemu:debian9 as we need the compiler as well?
It's not like there will be many users only carrying this image in their
docker setup.

>
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>> ---
>>  tests/docker/dockerfiles/debian-tricore-cross.docker | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/tests/docker/dockerfiles/debian-tricore-cross.docker b/tests/docker/dockerfiles/debian-tricore-cross.docker
>> index 898b8dd511..5ba3ca88a7 100644
>> --- a/tests/docker/dockerfiles/debian-tricore-cross.docker
>> +++ b/tests/docker/dockerfiles/debian-tricore-cross.docker
>> @@ -11,6 +11,9 @@ FROM debian:9
>>
>>  MAINTAINER Philippe Mathieu-Daudé <f4bug@amsat.org>
>>
>> +RUN apt-get update && \
>> +    DEBIAN_FRONTEND=noninteractive apt-get install -yy git
>> +
>>  RUN git clone --single-branch \
>>          https://github.com/bkoppelmann/tricore-binutils.git \
>>          /usr/src/binutils && \
>>


--
Alex Bennée

  parent reply	other threads:[~2018-07-12 10:44 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-09 15:21 [Qemu-devel] [PATCH for 3.0 00/10] various docker fixes Alex Bennée
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 01/10] tests/.gitignore: don't ignore docker tests Alex Bennée
2018-07-10  6:57   ` Fam Zheng
2018-07-10  7:54     ` Alex Bennée
2018-07-10  8:36       ` Fam Zheng
2018-07-10 20:50   ` Philippe Mathieu-Daudé
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 02/10] docker: debian-tricore add git tools Alex Bennée
2018-07-10 20:44   ` Philippe Mathieu-Daudé
2018-07-11  0:07     ` Philippe Mathieu-Daudé
2018-07-12 10:44     ` Alex Bennée [this message]
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 03/10] docker: fail more gracefully on docker.py check Alex Bennée
2018-07-10 20:44   ` Philippe Mathieu-Daudé
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 04/10] docker: split configure_qemu from build_qemu Alex Bennée
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 05/10] docker: move make check into check_qemu helper Alex Bennée
2018-07-10  6:59   ` Fam Zheng
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 06/10] docker: gracefully skip check_qemu Alex Bennée
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 07/10] docker: Makefile.include don't include partial images Alex Bennée
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 08/10] docker: disable debian-powerpc-user-cross Alex Bennée
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 09/10] docker: add test-unit runner Alex Bennée
2018-07-10  7:04   ` Fam Zheng
2018-07-12 10:20     ` Alex Bennée
2018-07-09 15:21 ` [Qemu-devel] [PATCH for 3.0 10/10] docker: add expansion for docker-test-FOO to Makefile.include Alex Bennée
2018-07-10 20:49   ` Philippe Mathieu-Daudé
2018-07-10 21:04     ` Alex Bennée
2018-07-11  1:31       ` Fam Zheng
2018-07-12  8:41         ` Alex Bennée
2018-07-10  7:06 ` [Qemu-devel] [PATCH for 3.0 00/10] various docker fixes Fam Zheng

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87pnzsd98j.fsf@linaro.org \
    --to=alex.bennee@linaro.org \
    --cc=agraf@suse.de \
    --cc=aurelien@aurel32.net \
    --cc=balrogg@gmail.com \
    --cc=berrange@redhat.com \
    --cc=cota@braap.org \
    --cc=f4bug@amsat.org \
    --cc=famz@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.