Re: [Qemu-devel] [RFC PATCH 4/8] docker: Add packages required to build a Linux kernel

[Philippe Mathieu-Daudé]

On 06/29/2018 11:09 AM, Alex Bennée wrote:
> Philippe Mathieu-Daudé <address@hidden> writes:
> 
>> Being able to also cross build Linux kernels with our docker cross
>> images allow us to add more complex qemu-system acceptance tests.
>>
>> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
>> ---
>> This is not required to build QEMU, so I might resend it in the
>> acceptance-test series which actually requires this patch.
>>
>>  tests/docker/dockerfiles/debian-sid.docker | 6 +++++-
>>  1 file changed, 5 insertions(+), 1 deletion(-)
>>
>> diff --git a/tests/docker/dockerfiles/debian-sid.docker 
>> b/tests/docker/dockerfiles/debian-sid.docker
>> index 9a3d168705..846021b032 100644
>> --- a/tests/docker/dockerfiles/debian-sid.docker
>> +++ b/tests/docker/dockerfiles/debian-sid.docker
>> @@ -22,11 +22,15 @@ RUN DEBIAN_FRONTEND=noninteractive apt install -yy 
>> eatmydata
>>  RUN DEBIAN_FRONTEND=noninteractive eatmydata \
>>      apt install -y --no-install-recommends \
>>          bison \
>> +        binutils-multiarch \
>>          build-essential \
>>          ca-certificates \
>> +        curl \
>>          flex \
>> +        gettext \
>>          git \
>> +        gnupg \
>>          pkg-config \
>>          psmisc \
>> -        python \
>> +        python-minimal \
> 
> Hmm don't we need full python for QEMU though?

I'm not sure, but the goal of this image is to be useful, not tiny, so
I'm happy with the normal package (I cherry-picked this from an older
series and did not notice this particular change).

> 
>>          texinfo || { echo "Failed to build - see debian-sid.docker notes"; 
>> exit 1; }
> 
> 
> --
> Alex Bennée
>