aboutsummaryrefslogtreecommitdiffhomepage
path: root/projects/brotli
diff options
context:
space:
mode:
authorGravatar Devin Jeanpierre <jeanpierreda@gmail.com>2017-05-01 23:31:02 -0700
committerGravatar Kostya Serebryany <konstantin.s.serebryany@gmail.com>2017-05-01 23:31:02 -0700
commit330386686d6e7b84e9fd927083d3d9cf681835b1 (patch)
tree04274d779612e66be7ecc51765b20a3ec96d0cf6 /projects/brotli
parentd5ac427bf877a68062f226f34f9258901153a494 (diff)
Use apt-get update && apt-get install, per best practices: (#561)
https://docs.docker.com/engine/userguide/eng-image/dockerfile_best-practices/ I ran into this because I was getting errors locally, like: E: Failed to fetch http://archive.ubuntu.com/ubuntu/pool/main/d/dpkg/libdpkg-perl_1.18.4ubuntu1.1_all.deb 404 Not Found [IP: 91.189.88.149 80] It turns out you get these if you don't update, and the official best practices are to `run apt-get update && apt-get install`. In fact, running _any_ apt-get install command without the apt-get update && before it can result in unfortunate caching artifacts -- see "cache busting". (P.S. thanks to Peng on Freenode for helping me, I'm bad at Ubuntu.) So: sed -re \ 's/RUN apt-get ((-y )?(install|build-dep))/RUN apt-get update \&\& apt-get \1/' -i \ projects/**/Dockerfile I also manually fixed the cases that already ran apt-get update in their Dockerfile: dlplibs/Dockerfile grpc/Dockerfile libreoffice/Dockerfile
Diffstat (limited to 'projects/brotli')
-rw-r--r--projects/brotli/Dockerfile2
1 files changed, 1 insertions, 1 deletions
diff --git a/projects/brotli/Dockerfile b/projects/brotli/Dockerfile
index a502f2cc..db253827 100644
--- a/projects/brotli/Dockerfile
+++ b/projects/brotli/Dockerfile
@@ -16,7 +16,7 @@
FROM gcr.io/oss-fuzz-base/base-builder
MAINTAINER eustas@chromium.org
-RUN apt-get install -y cmake libtool make
+RUN apt-get update && apt-get install -y cmake libtool make
RUN git clone --depth 1 https://github.com/google/brotli.git
WORKDIR brotli