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

Philippe Mathieu-Daudé posted 8 patches 7 years, 4 months ago
[Qemu-devel] [RFC PATCH 4/8] docker: Add packages required to build a Linux kernel
Posted by Philippe Mathieu-Daudé 7 years, 4 months ago
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é <f4bug@amsat.org>
---
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 \
         texinfo || { echo "Failed to build - see debian-sid.docker notes"; exit 1; }
-- 
2.18.0


Re: [Qemu-devel] [RFC PATCH 4/8] docker: Add packages required to build a Linux kernel
Posted by Alex Bennée 7 years, 4 months ago
Philippe Mathieu-Daudé <f4bug@amsat.org> 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é <f4bug@amsat.org>
> ---
> 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?

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


--
Alex Bennée

Re: [Qemu-devel] [RFC PATCH 4/8] docker: Add packages required to build a Linux kernel
Posted by Philippe Mathieu-Daudé 7 years, 4 months ago
On 06/29/2018 11:09 AM, Alex Bennée wrote:
> Philippe Mathieu-Daudé <f4bug@amsat.org> 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é <f4bug@amsat.org>
>> ---
>> 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
>