-
Notifications
You must be signed in to change notification settings - Fork 263
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
Error: Unsupported target versions: Node.js > v17 #710
Comments
@cclauss I think node-pre-gyp can be viewed as having no maintenance due to #657. I was never associated with the package, just made PRs as needed, and since that issue I ended up using patch-package to maintain updates to the abi_crosswalk.json file there, it's not ideal but AFAICT it's the only path forward for users right now. At 6.5 million weekly downloads and >1000 dependents (across |
Fixed in #711 but perhaps not yet released. https://github.com/mapbox/node-pre-gyp/tags |
* Fixing issue 647 - Updated abi_crosswalk.json to include node 18 (current). - Added test coverage for cases of unsupported/unknown targets. * run.util.js: Test building with msvs 2022 * Update appveyor.yml * Update appveyor.yml * Update appveyor.yml * Update appveyor.yml * Update appveyor.yml * Update appveyor.yml * Update appveyor.yml * Update appveyor.yml * Update appveyor.yml * #710 * update abi_crosswalk.json * Update appveyor.yml for node 20 support * test: show command * Revert "test: show command" This reverts commit 7804f5c. * add line ending to abi_crosswalk.json fix github warning error * add node 21 to appveyor.yml Co-authored-by: Christian Clauss <cclauss@me.com> * Update appveyor.yml Co-authored-by: Christian Clauss <cclauss@me.com> * npm run update-crosswalk Co-authored-by: Christian Clauss <cclauss@me.com> * npm run update-crosswalk for node 22 * change node 21 to node 22 in appveyor.yml --------- Co-authored-by: Ron Ilan <github@ronilan.com> Co-authored-by: Christian Clauss <cclauss@me.com> Co-authored-by: Ron Ilan <ronilan@users.noreply.github.com>
WORKAROUND: To support newer versions of Node.js > v17
abi_crosswalk.json
must be updated to cover all currently supported versions of Node.js.node-pre-gyp/lib/util/abi_crosswalk.json
Line 2598 in a74f5e3
Proof:
The testing on this repo fails on all currently supported versions of Node.js but passes on end-of-life versions.
@springmeyer @ewanharris, @mapsam
The text was updated successfully, but these errors were encountered: