support/docker: use the distro-provided flake8

Currently, we install flake8 and its dependencies via pip. We
tried to be reproducible by pinning the version of those python
packages, but we did forget quite a few of them, and thus some
dependencies for flake8 are installed as uncontrolled versions.

Furthermore, before we install flake8 and its dependencies, we
forcibly update pip, setuptools, and wheels packages to their
latest versions. This explicitly breaks reproducibility.

While we could enforce a specific version of all those packages
and still grab them from PyPI, we can simply grab them from the
distribution-provided packages instead.

Since we're using a pinned version of stretch, this already
guarantees we'll reproducibly get the same versions over and
over again. Besides, we just need to list flake8 as a package to
install to automatically get all its dependencies (again, in a
reproducible way).

This has the slight unfortunate drawback of downgrading flake8
to version 3.2.1, from version 3.5.0, as well as downgrading a
few of flake8's dependencies, as noticed by Ricardo:
    http://lists.busybox.net/pipermail/buildroot/2018-May/222376.html

However, as Ricardo said, there isn't "any serious limitation of
this old version, the release notes for a version in the between
mentions 'Dramatically improve the performance' but we have a
limited number of scripts and running on Gitlab for all of them
still takes less than 5 minutes".

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Ricardo Martincoski <ricardo.martincoski@gmail.com>
Acked-by: Ricardo Martincoski <ricardo.martincoski@gmail.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
This commit is contained in:
Yann E. MORIN 2018-06-10 17:44:58 +02:00 committed by Thomas Petazzoni
parent afc1ec7073
commit 567a8476ad

View File

@ -36,9 +36,9 @@ RUN apt-get install -y --no-install-recommends \
libncurses5-dev \
locales \
mercurial \
python-flake8 \
python-nose2 \
python-pexpect \
python-pip \
qemu-system-arm \
qemu-system-x86 \
rsync \
@ -49,14 +49,6 @@ RUN apt-get install -y --no-install-recommends \
apt-get -y autoremove && \
apt-get -y clean
# For check-flake8
RUN python -m pip install --upgrade pip setuptools wheel && \
pip install -q \
flake8==3.5.0 \
mccabe==0.6.1 \
pycodestyle==2.3.1 \
pyflakes==1.6.0
# To be able to generate a toolchain with locales, enable one UTF-8 locale
RUN sed -i 's/# \(en_US.UTF-8\)/\1/' /etc/locale.gen && \
/usr/sbin/locale-gen