Merge pull request metacpan#2246 from metacpan/oalders/Dockerfile

Don't install npm in Dockerfile

Author: ssoriche

Dockerfile

@@ -1,22 +1,24 @@
 FROM metacpan/metacpan-base:latest
 
-RUN curl -sL https://deb.nodesource.com/setup_10.x | bash \
+RUN apt install -f && curl -sL https://deb.nodesource.com/setup_10.x | bash \
+    && curl -sL https://dl.yarnpkg.com/debian/pubkey.gpg | apt-key add - \
+    && echo "deb https://dl.yarnpkg.com/debian/ stable main" | tee /etc/apt/sources.list.d/yarn.list \
     && apt-get update \
-    && apt-get install -y nodejs npm \
-    && npm install -g yarn
+    && apt-get install -y nodejs yarn
 
 ENV PERL_MM_USE_DEFAULT=1 PERL_CARTON_PATH=/carton
 
 COPY . /metacpan-web/
 WORKDIR /metacpan-web
 
+RUN yarn install
+
 RUN cpanm --notest App::cpm \
     && cpm install -g Carton \
     && useradd -m metacpan-web -g users \
     && mkdir /carton \
     && cpm install -L /carton \
-    && rm -fr /root/.cpanm /root/.perl-cpm /tmp/* \
-    && yarn install
+    && rm -fr /root/.cpanm /root/.perl-cpm /tmp/*
 
 RUN chown -R metacpan-web:users /metacpan-web /carton