Skip to content
This repository has been archived by the owner on Jun 4, 2023. It is now read-only.

Commit

Permalink
docker: don't reinstall all node modules, just link
Browse files Browse the repository at this point in the history
Using `npm install` in the dist directory was useful if you plan to get
ride of the non-dist stuff. However, we don't do that, so we can just
use a symbolic link. We don't do that because there is no space to get
back because of how Docker use layers.
  • Loading branch information
vincentbernat committed Apr 20, 2015
1 parent 6e626bb commit 949f32a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ WORKDIR /dashkiosk
COPY . /dashkiosk
RUN npm install
RUN grunt --branding=exoscale
RUN cd dist ; npm install --production
RUN cd dist ; ln -s ../node_modules

# We use SQLite by default. If you want to keep the database between
# runs, don't forget to provide a volume for /database.
Expand Down

0 comments on commit 949f32a

Please sign in to comment.