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

Nodesource script is deprecated #1846

Closed
1 task done
blm768 opened this issue Aug 31, 2023 · 6 comments
Closed
1 task done

Nodesource script is deprecated #1846

blm768 opened this issue Aug 31, 2023 · 6 comments
Milestone

Comments

@blm768
Copy link
Contributor

blm768 commented Aug 31, 2023

Verification

  • I searched for similar issues (including closed issues) and found none was relevant.

Introduce the issue

The dev container pulls in https://deb.nodesource.com/setup_18.x. Running that script produces a deprecation notice:/

================================================================================
▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓
================================================================================

                           SCRIPT DEPRECATION WARNING                    

  
  This script, located at https://deb.nodesource.com/setup_X, used to
  install Node.js is deprecated now and will eventually be made inactive.

  Please visit the NodeSource distributions Github and follow the
  instructions to migrate your repo.
  https://github.com/nodesource/distributions

  The NodeSource Node.js Linux distributions GitHub repository contains
  information about which versions of Node.js and which Linux distributions
  are supported and how to install it.
  https://github.com/nodesource/distributions


                          SCRIPT DEPRECATION WARNING

================================================================================
▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓
================================================================================

I don't know if this is related, but on my machine, the dev container is somehow missing npm. I'm not sure how the rest of the dev container can build if this step is failing, so that's pretty weird. My system might just be special, though. Running the script manually in the container does let me install npm.

Preferred solution

No response

Version

v1.13.0

@blm768
Copy link
Contributor Author

blm768 commented Sep 1, 2023

Ah, I was looking at the wrong Dockerfile. That'd explain why the dev container didn't have NPM; it just wasn't installing it.

@FintasticMan
Copy link
Member

Yeah, the devcontainer is very outdated

@blm768
Copy link
Contributor Author

blm768 commented Sep 3, 2023

While it was up to date, did it have reasonable usability? I ask because I'm working on NixOS, which makes it kind of clunky to install all the dependencies via apt. There are various things I can try with the main Dockerfile and/or systemd-machined if the dev container was abandoned due to usability issues.

@FintasticMan
Copy link
Member

FintasticMan commented Sep 3, 2023

The Dockerfile in the docker directory is up to date, and is used for the CI builds. There is a PR that merges the devcontainer with the main Dockerfile, see #1587. I've never used the devcontainer myself, so I can't attest to how well it worked, but I think the reason it was left untouched was just because none of the core developers use it to keep it up to date.

@blm768
Copy link
Contributor Author

blm768 commented Sep 3, 2023

Ooh, nice! I'll see if I can get things to build on top of that PR locally. If it looks like the PR's going to stall out, I could try to stack some fixes on it to address the outstanding issues.

@JF002
Copy link
Collaborator

JF002 commented Dec 10, 2023

Fixed in #1849.

@JF002 JF002 closed this as completed Dec 10, 2023
@JF002 JF002 added this to the 1.14.0 milestone Dec 10, 2023
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

No branches or pull requests

3 participants