Merge lp:~cjwatson/isitdeployable/docker-dependencies-steps into lp:isitdeployable

Proposed by Colin Watson
Status: Merged
Approved by: Colin Watson
Approved revision: 274
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: lp:~cjwatson/isitdeployable/docker-dependencies-steps
Merge into: lp:isitdeployable
Diff against target: 35 lines (+6/-5)
2 files modified
Dockerfile (+4/-3)
setup-container (+2/-2)
To merge this branch: bzr merge lp:~cjwatson/isitdeployable/docker-dependencies-steps
Reviewer Review Type Date Requested Status
Adam Collard (community) Approve
Review via email: mp+336768@code.launchpad.net

Commit message

Split Dockerfile installation of system dependencies into multiple steps.

Description of the change

This speeds things up when iterating on changes to dependencies.txt.docker.

To post a comment you must log in.
Revision history for this message
Adam Collard (adam-collard) :
review: Approve
Revision history for this message
Otto Co-Pilot (otto-copilot) wrote :

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'Dockerfile'
--- Dockerfile 2018-01-30 15:33:52 +0000
+++ Dockerfile 2018-02-07 14:18:42 +0000
@@ -8,11 +8,12 @@
8WORKDIR /usr/src/app8WORKDIR /usr/src/app
99
10# install system dependencies (debs)10# install system dependencies (debs)
11RUN apt-get update \
12 && apt-get install -y software-properties-common \
13 && add-apt-repository --yes --update ppa:ubuntuone/isitdeployable-dependencies \
14 && apt-get upgrade -y
11COPY dependencies.txt.docker ./15COPY dependencies.txt.docker ./
12RUN apt-get update \16RUN apt-get update \
13 && apt-get install -y software-properties-common \
14 && apt-add-repository --yes ppa:ubuntuone/isitdeployable-dependencies \
15 && apt-get update && apt-get upgrade -y \
16 && cat dependencies.txt.docker | xargs apt-get install -y --no-install-recommends \17 && cat dependencies.txt.docker | xargs apt-get install -y --no-install-recommends \
17 && apt-get autoremove -y \18 && apt-get autoremove -y \
18 && apt-get clean \19 && apt-get clean \
1920
=== modified file 'setup-container'
--- setup-container 2016-09-27 14:31:46 +0000
+++ setup-container 2018-02-07 14:18:42 +0000
@@ -26,9 +26,9 @@
26# Install software-properties-common for easy addition of PPA26# Install software-properties-common for easy addition of PPA
27apt-get install -y software-properties-common27apt-get install -y software-properties-common
28# Add PPA containing codetree28# Add PPA containing codetree
29apt-add-repository --yes ppa:mojo-maintainers29add-apt-repository --yes ppa:mojo-maintainers
30# Add PPA containing other dependencies30# Add PPA containing other dependencies
31apt-add-repository --yes ppa:ubuntuone/isitdeployable-dependencies31add-apt-repository --yes ppa:ubuntuone/isitdeployable-dependencies
32apt-get update32apt-get update
33# Install apt dependencies33# Install apt dependencies
34cat dependencies.txt dependencies-devel.txt | xargs apt-get install -y --no-install-recommends34cat dependencies.txt dependencies-devel.txt | xargs apt-get install -y --no-install-recommends

Subscribers

People subscribed via source and target branches