diff mbox series

[v2,4/8] docker: do not use Trusty APT source in Xenial

Message ID 20180115143500.22631-5-f4bug@amsat.org
State New
Headers show
Series docker: update Ubuntu and Fedora images, deprecate old ones | expand

Commit Message

Philippe Mathieu-Daudé Jan. 15, 2018, 2:34 p.m. UTC
probably missed in 7fc581c29518

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
 tests/docker/dockerfiles/ubuntu.docker | 2 --
 1 file changed, 2 deletions(-)

Comments

Alex Bennée Jan. 17, 2018, 11:04 a.m. UTC | #1
Philippe Mathieu-Daudé <f4bug@amsat.org> writes:

> probably missed in 7fc581c29518
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Reviewed-by: Alex Bennée <alex.bennee@linaro.org>

> ---
>  tests/docker/dockerfiles/ubuntu.docker | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/tests/docker/dockerfiles/ubuntu.docker b/tests/docker/dockerfiles/ubuntu.docker
> index dabbf2a8a4..686d90ae8c 100644
> --- a/tests/docker/dockerfiles/ubuntu.docker
> +++ b/tests/docker/dockerfiles/ubuntu.docker
> @@ -1,6 +1,4 @@
>  FROM ubuntu:16.04
> -RUN echo "deb http://archive.ubuntu.com/ubuntu/ trusty universe multiverse" >> \
> -    /etc/apt/sources.list
>  RUN apt-get update
>  ENV PACKAGES flex bison \
>      libusb-1.0-0-dev libiscsi-dev librados-dev libncurses5-dev libncursesw5-dev \


--
Alex Bennée
Alex Bennée Jan. 17, 2018, 11:54 a.m. UTC | #2
Philippe Mathieu-Daudé <f4bug@amsat.org> writes:

> probably missed in 7fc581c29518
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
>  tests/docker/dockerfiles/ubuntu.docker | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/tests/docker/dockerfiles/ubuntu.docker b/tests/docker/dockerfiles/ubuntu.docker
> index dabbf2a8a4..686d90ae8c 100644
> --- a/tests/docker/dockerfiles/ubuntu.docker
> +++ b/tests/docker/dockerfiles/ubuntu.docker
> @@ -1,6 +1,4 @@
>  FROM ubuntu:16.04
> -RUN echo "deb http://archive.ubuntu.com/ubuntu/ trusty universe multiverse" >> \
> -    /etc/apt/sources.list

do we not need anything in universe or multiverse?

>  RUN apt-get update
>  ENV PACKAGES flex bison \
>      libusb-1.0-0-dev libiscsi-dev librados-dev libncurses5-dev libncursesw5-dev \


--
Alex Bennée
Philippe Mathieu-Daudé Jan. 17, 2018, 12:25 p.m. UTC | #3
On 01/17/2018 08:54 AM, Alex Bennée wrote:
> 
> Philippe Mathieu-Daudé <f4bug@amsat.org> writes:
> 
>> probably missed in 7fc581c29518
>>
>> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>> ---
>>  tests/docker/dockerfiles/ubuntu.docker | 2 --
>>  1 file changed, 2 deletions(-)
>>
>> diff --git a/tests/docker/dockerfiles/ubuntu.docker b/tests/docker/dockerfiles/ubuntu.docker
>> index dabbf2a8a4..686d90ae8c 100644
>> --- a/tests/docker/dockerfiles/ubuntu.docker
>> +++ b/tests/docker/dockerfiles/ubuntu.docker
>> @@ -1,6 +1,4 @@
>>  FROM ubuntu:16.04
>> -RUN echo "deb http://archive.ubuntu.com/ubuntu/ trusty universe multiverse" >> \
>> -    /etc/apt/sources.list
> 
> do we not need anything in universe or multiverse?

Since this image is Xenial based, surely not from Trusty :)

This line is now included in the base image, I'll update the commit
description.

> 
>>  RUN apt-get update
>>  ENV PACKAGES flex bison \
>>      libusb-1.0-0-dev libiscsi-dev librados-dev libncurses5-dev libncursesw5-dev \
> 
> 
> --
> Alex Bennée
>
diff mbox series

Patch

diff --git a/tests/docker/dockerfiles/ubuntu.docker b/tests/docker/dockerfiles/ubuntu.docker
index dabbf2a8a4..686d90ae8c 100644
--- a/tests/docker/dockerfiles/ubuntu.docker
+++ b/tests/docker/dockerfiles/ubuntu.docker
@@ -1,6 +1,4 @@ 
 FROM ubuntu:16.04
-RUN echo "deb http://archive.ubuntu.com/ubuntu/ trusty universe multiverse" >> \
-    /etc/apt/sources.list
 RUN apt-get update
 ENV PACKAGES flex bison \
     libusb-1.0-0-dev libiscsi-dev librados-dev libncurses5-dev libncursesw5-dev \