Skip to content

Commit

Permalink
Add back node js installation in chip-build image (#23667)
Browse files Browse the repository at this point in the history
* Add back node js installation in chip-build image

* Add terminating newline to dockerfile

* Restyle
  • Loading branch information
andy31415 authored and pull[bot] committed Sep 25, 2023
1 parent f20f929 commit 4d4de50
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 1 deletion.
20 changes: 20 additions & 0 deletions integrations/docker/images/chip-build/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -148,3 +148,23 @@ RUN set -x \
&& rm zap \
&& ln -s /opt/zap-${ZAP_VERSION}/zap-cli /usr/bin/ \
&& : # last line

# NodeJS: install a newer version than what apt-get would read
# This installs the latest LTS version of nodejs
#
# NodeJS is required by github actions, we use Wandalen/[email protected]
# and that seems to use the built-in node installation in the image
#
# This is not a CHIP dependency directly, but used by CI
ENV CHIP_NODE_VERSION=v16.13.2
RUN set -x \
&& mkdir node_js \
&& cd node_js \
&& wget https://nodejs.org/dist/$CHIP_NODE_VERSION/node-$CHIP_NODE_VERSION-linux-x64.tar.xz \
&& tar xfvJ node-$CHIP_NODE_VERSION-linux-x64.tar.xz \
&& mv node-$CHIP_NODE_VERSION-linux-x64 /opt/ \
&& ln -s /opt/node-$CHIP_NODE_VERSION-linux-x64 /opt/node \
&& ln -s /opt/node/bin/* /usr/bin \
&& cd .. \
&& rm -rf node_js \
&& : # last line
2 changes: 1 addition & 1 deletion integrations/docker/images/chip-build/version
Original file line number Diff line number Diff line change
@@ -1 +1 @@
0.6.10 Version bump reason: Fix zap link (link zap-cli into /usr/bin instead of zap)
0.6.11 Version bump reason: install nodejs into chip-build image, since github js actions seem to need it.

0 comments on commit 4d4de50

Please sign in to comment.