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

Update rclnodejs dependency to 0.27.5 for newer Node.js LTS's #26

Open
mrjogo opened this issue Dec 19, 2024 · 1 comment
Open

Update rclnodejs dependency to 0.27.5 for newer Node.js LTS's #26

mrjogo opened this issue Dec 19, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@mrjogo
Copy link

mrjogo commented Dec 19, 2024

Description

rclnodejs-cli depends on rclnodejs ^0.21.4, which only works on Node.js versions below 20.0.0. There have been two newer LTS versions since then (20 and 22), and the latest supported one (18) is EOL in April 2025. rclnodejs 0.27.5 has support for Node.js version 22.

  • Library Version: 0.3.1 (as an aside, it seems like the version got transposed and it should be 0.1.3)
  • ROS Version: Jazzy
  • Platform / OS: Ubuntu 22.04

Steps To Reproduce

$ nvm use --lts
Now using node v22.12.0 (npm v10.9.0)
$ npx rclnodejs-cli -h
Need to install the following packages:
[email protected]
Ok to proceed? (y) y

Expected Behavior

rclnodejs-cli installs and prints the help string.

Actual Behavior

It does not install with following output:

npm warn EBADENGINE Unsupported engine {
npm warn EBADENGINE   package: '[email protected]',
npm warn EBADENGINE   required: { node: '>= 10.23.1 <20.0.0' },
npm warn EBADENGINE   current: { node: 'v22.12.0', npm: '10.9.0' }
npm warn EBADENGINE }
npm warn deprecated [email protected]: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm warn deprecated [email protected]: Glob versions prior to v9 are no longer supported
npm warn deprecated [email protected]: this library is no longer supported
npm warn deprecated [email protected]: This package is no longer supported.
npm warn deprecated [email protected]: This package is no longer supported.
npm warn deprecated [email protected]: Rimraf versions prior to v4 are no longer supported
npm warn deprecated [email protected]: request has been deprecated, see https://github.com/request/request/issues/3142
npm warn deprecated [email protected]: This package is no longer supported.
npm warn deprecated [email protected]: See https://aka.ms/type-testing-tools
npm warn deprecated [email protected]: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
@mrjogo mrjogo added the bug Something isn't working label Dec 19, 2024
@mrjogo
Copy link
Author

mrjogo commented Dec 19, 2024

As a workaround, the following can be added to a project's package.json:

For npm:

"overrides": {
  "rclnodejs-cli": {
    "rclnodejs": "^0.27.5"
  }
}

For yarn:

"resolutions": {
  "rclnodejs": "^0.27.5"
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant