Merge pull request #53 from AlaricChan/pipeline-testing

[RAC-6701] Fix NPM5 build dependency issue
RackHD · Apr 11, 2018 · ea09728 · ea09728
2 parents 629044b + 04701ec
commit ea09728
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -9,9 +9,10 @@ COPY . /RackHD/on-syslog/
 WORKDIR /RackHD/on-syslog
 
 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 514/udp
 CMD [ "node", "/RackHD/on-syslog/index.js" ]