Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1497)

Unified Diff: blimp/engine/Dockerfile

Issue 2106843003: Blimp engine Dockerfile now depends on Dockerfile.base. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update documentation on docker build for engine to include building the base. Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « blimp/docs/container.md ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: blimp/engine/Dockerfile
diff --git a/blimp/engine/Dockerfile b/blimp/engine/Dockerfile
index 745c12de93b625b57ad1955df92bf37f9ebdaa20..12b4b61894afe2372d701bfe99e715201485fdd0 100644
--- a/blimp/engine/Dockerfile
+++ b/blimp/engine/Dockerfile
@@ -1,19 +1,4 @@
-FROM ubuntu:trusty
-
-# Run the command below to update the lib list.
-# ldd ./blimp_engine_app | grep usr/lib | awk '{print $3}' | xargs -n1 \
-# dpkg-query -S | awk -F: '{print $1}' | sort | uniq
-RUN apt-get update && \
- apt-get install -yq libdrm2 libfontconfig1 libfreetype6 libgraphite2-3 \
- libharfbuzz0b libnspr4 libnss3 libstdc++6
-
-# stunnel4 is needed for incoming TLS connections.
-# wget is needed for crash reporting.
-RUN apt-get update && apt-get install -yq stunnel4 wget
-
-# gdb and strace are not strictly required, but having them around makes
-# debugging easier.
-RUN apt-get update && apt-get install -yq gdb strace
+FROM base:latest
RUN mkdir /engine
« no previous file with comments | « blimp/docs/container.md ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698