Merge pull request #88 from AlaricChan/pipeline-testing

[RAC-6701] Fix NPM5 build dependency issue
RackHD · Apr 11, 2018 · 787b034 · 787b034
2 parents 6b6eee3 + 85e3eb7
commit 787b034
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -9,9 +9,10 @@ COPY . /RackHD/on-dhcp-proxy/
 WORKDIR /RackHD/on-dhcp-proxy
 
 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 68/udp 4011
 CMD [ "node", "/RackHD/on-dhcp-proxy/index.js" ]