Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace deprecated script and install Node directly from nodesource #1927

Closed
wants to merge 1 commit into from

Conversation

CodeHex
Copy link

@CodeHex CodeHex commented Sep 14, 2023

Installing Node from nodesource scripts has been very recently deprecated (and sleeps the build for 60s). This replaces the script with installing the signed debian repo directly using the instructions provided here.

Fixes #1926

@CodeHex CodeHex changed the title Replace deprecated script and install Node directly for nodesource Replace deprecated script and install Node directly from nodesource Sep 14, 2023
@CodeHex
Copy link
Author

CodeHex commented Sep 14, 2023

When I tested this, I had to use the changes in PR #1900, as I'm working on an M1 Mac. It got through the install steps fine, but someone might want to double check it on a different machine

@biggianteye
Copy link
Contributor

When I tested this, I had to use the changes in PR #1900, as I'm working on an M1 Mac.

I'm glad you used them. Good to get some independent confirmation that they work.

@CodeHex
Copy link
Author

CodeHex commented Sep 15, 2023

When I tested this, I had to use the changes in PR #1900, as I'm working on an M1 Mac.

I'm glad you used them. Good to get some independent confirmation that they work.

👍 Yeah thanks, I did get a couple of test failures but running the website seems fine

RUN curl -fsSL https://deb.nodesource.com/setup_20.x | bash -
# Instructions for installing node from nodesource can be found here:
# https://github.com/nodesource/distributions
RUN set -ex; \
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is set -ex; needed?

@matyikriszta
Copy link
Contributor

Looks like the install scripts have been undeprecated? https://github.com/nodesource/distributions now suggest we use the script we have in the current Dockerfile.

The docs at https://github.com/nodesource/distributions/blob/master/README.md#faq and https://github.com/nodesource/distributions/wiki/Repository-Manual-Installation#-simplified-installation-using-scripts suggest that the script at https://deb.nodesource.com/setup_20.x is supported and recommended.

Reading the source of https://deb.nodesource.com/setup_20.x it no longer contains any sleep behaviour.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecated Node install script when building Docker image
4 participants