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

Node v20 warning for non-Docker install #179

Open
kb2ma opened this issue Sep 21, 2024 · 1 comment
Open

Node v20 warning for non-Docker install #179

kb2ma opened this issue Sep 21, 2024 · 1 comment

Comments

@kb2ma
Copy link
Contributor

kb2ma commented Sep 21, 2024

When running npm install with node lts/iron (v20.17.0), the install reports that the drivelist package does not support that node version, like below. I see jetson-flash package.json specifies only node>=16. So, I dropped the node version down to lts/hydrogen (v18.20.4) to remove the warning.

[2024-09-23 -- The statement below from the original post of this issue is wrong. Drivelist up to v11.1.1 supports node >=16 < 19. Drivelist v11.2.0 supports node >=16, and drivelist v12.0.0 supports node >=18 -- so it still supports node 18. So the issue is that my jetson-flash build used drivelist v11.1.0. The problem is more about package resolution from jetson-flash -- at least when I tested. Let me recreate the test.]


The drivelist package dropped support for node v18 in its v12.0.0. Time to update nodelist to latest (v12.0.2) and update jetson-flash node version requirement to >=20 as well?

$ npm install
npm warn EBADENGINE Unsupported engine {
npm warn EBADENGINE   package: '[email protected]',
npm warn EBADENGINE   required: { node: '>=16 < 19' },
npm warn EBADENGINE   current: { node: 'v20.17.0', npm: '10.8.2' }
npm warn EBADENGINE }
@kb2ma
Copy link
Contributor Author

kb2ma commented Sep 24, 2024

I can confirm that use of node v20 generates the warning on current master. The chain of package dependencies is:

  • jetson-flash
  • resin-cli-visuals
  • etcher-sdk
  • drivelist

Next step is to experiment with advancing the drivelist version in the etcher-sdk dependencies.

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

1 participant