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
1=== modified file 'Dockerfile'
2--- Dockerfile 2018-01-30 15:33:52 +0000
3+++ Dockerfile 2018-02-07 14:18:42 +0000
4@@ -8,11 +8,12 @@
5 WORKDIR /usr/src/app
6
7 # install system dependencies (debs)
8+RUN apt-get update \
9+ && apt-get install -y software-properties-common \
10+ && add-apt-repository --yes --update ppa:ubuntuone/isitdeployable-dependencies \
11+ && apt-get upgrade -y
12 COPY dependencies.txt.docker ./
13 RUN apt-get update \
14- && apt-get install -y software-properties-common \
15- && apt-add-repository --yes ppa:ubuntuone/isitdeployable-dependencies \
16- && apt-get update && apt-get upgrade -y \
17 && cat dependencies.txt.docker | xargs apt-get install -y --no-install-recommends \
18 && apt-get autoremove -y \
19 && apt-get clean \
20
21=== modified file 'setup-container'
22--- setup-container 2016-09-27 14:31:46 +0000
23+++ setup-container 2018-02-07 14:18:42 +0000
24@@ -26,9 +26,9 @@
25 # Install software-properties-common for easy addition of PPA
26 apt-get install -y software-properties-common
27 # Add PPA containing codetree
28-apt-add-repository --yes ppa:mojo-maintainers
29+add-apt-repository --yes ppa:mojo-maintainers
30 # Add PPA containing other dependencies
31-apt-add-repository --yes ppa:ubuntuone/isitdeployable-dependencies
32+add-apt-repository --yes ppa:ubuntuone/isitdeployable-dependencies
33 apt-get update
34 # Install apt dependencies
35 cat dependencies.txt dependencies-devel.txt | xargs apt-get install -y --no-install-recommends

Subscribers

People subscribed via source and target branches