Merge pull request #79 from AlaricChan/pipeline-testing

[RAC-6701] Fix NPM5 build dependency issue
RackHD · Apr 11, 2018 · 4a222ec · 4a222ec
2 parents 92d2038 + e76248a
commit 4a222ec
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -9,9 +9,10 @@ COPY . /RackHD/on-tftp/
 WORKDIR /RackHD/on-tftp
 
 RUN mkdir -p ./node_modules \
+  && npm install --ignore-scripts --production \
+  && rm -r ./node_modules/on-core ./node_modules/di \
   && ln -s /RackHD/on-core ./node_modules/on-core \
-  && ln -s /RackHD/on-core/node_modules/di ./node_modules/di \
-  && npm install --ignore-scripts --production
+  && ln -s /RackHD/on-core/node_modules/di ./node_modules/di
 
 EXPOSE 69/udp
 VOLUME /RackHD/on-tftp/static/tftp