yetus-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From a.@apache.org
Subject [yetus] branch master updated: YETUS-855. npm does not work in docker image
Date Sat, 20 Apr 2019 18:12:59 GMT
This is an automated email from the ASF dual-hosted git repository.

aw pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/yetus.git


The following commit(s) were added to refs/heads/master by this push:
     new 90fd5f9  YETUS-855. npm does not work in docker image
90fd5f9 is described below

commit 90fd5f952e9d184bbc94cc252d600b6b7fecc5d4
Author: Allen Wittenauer <aw@apache.org>
AuthorDate: Wed Apr 17 09:43:39 2019 -0700

    YETUS-855. npm does not work in docker image
    
    Signed-off-by: Allen Wittenauer <aw@apache.org>
---
 precommit/src/main/shell/test-patch-docker/Dockerfile | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/precommit/src/main/shell/test-patch-docker/Dockerfile b/precommit/src/main/shell/test-patch-docker/Dockerfile
index e85b187..c563dfd 100644
--- a/precommit/src/main/shell/test-patch-docker/Dockerfile
+++ b/precommit/src/main/shell/test-patch-docker/Dockerfile
@@ -241,12 +241,10 @@ RUN curl -L -s -S \
 # Install npm and JSHint
 ###
 # hadolint ignore=DL3008,DL3016
-RUN apt-get -q update \
-    && apt-get -q install --no-install-recommends -y nodejs npm \
+RUN curl -sL https://deb.nodesource.com/setup_11.x | bash - \
+    && apt-get -q install --no-install-recommends -y nodejs \
     && apt-get clean \
     && rm -rf /var/lib/apt/lists/* && \
-    ln -s /usr/bin/nodejs /usr/bin/node && \
-    npm install -g npm@latest && \
     npm install -g jshint
 
 ####


Mime
View raw message