X Tutup
Skip to content

Fixed #3588 to use darwin-x64 for all node < 16.0.0#3709

Open
Rahulbeniwal26119 wants to merge 1 commit intonvm-sh:masterfrom
Rahulbeniwal26119:fallback-to-x64-for-darwin-node-versions-less-than-16
Open

Fixed #3588 to use darwin-x64 for all node < 16.0.0#3709
Rahulbeniwal26119 wants to merge 1 commit intonvm-sh:masterfrom
Rahulbeniwal26119:fallback-to-x64-for-darwin-node-versions-less-than-16

Conversation

@Rahulbeniwal26119
Copy link
Contributor

@Rahulbeniwal26119 Rahulbeniwal26119 commented Nov 16, 2025

This PR addresses the issue raised by #3588

  • For Node Version < 16.0.0 there are no amd64 binaries
  • Due to Resotta 2, x64 binaries can be run on new Macs
  • Existing logic was having an issue for node versions like 14.18.0, it looked for darwin-amd64 binaries
  • This PR updates the existing logic to make a stricter check for all versions less than 16.0.0
  • Added a few unit test cases to ensure that changes are behaving as expected.
> bash -c "source ./nvm.sh && nvm install 14.18.0"
Downloading and installing node v14.18.0...
Downloading https://nodejs.org/dist/v14.18.0/node-v14.18.0-darwin-x64.tar.xz...
######################################################################### 100.0%
Computing checksum with sha256sum
Checksums matched!
Now using node v14.18.0 (npm v6.14.15)
> uname -a 
Darwin Mac.lan 24.6.0 Darwin Kernel Version 24.6.0

@ljharb
Copy link
Member

ljharb commented Jan 29, 2026

I'm still a bit unclear. You're saying that node 14.18.0 doesn't run on mac arm64, despite the existence of https://nodejs.org/dist/v14.18.0/node-v14.18.0-darwin-x64.tar.gz ?

@Rahulbeniwal26119 Rahulbeniwal26119 force-pushed the fallback-to-x64-for-darwin-node-versions-less-than-16 branch from 3385907 to 433cd44 Compare February 20, 2026 06:48
@Rahulbeniwal26119
Copy link
Contributor Author

I'm still a bit unclear. You're saying that node 14.18.0 doesn't run on mac arm64, despite the existence of https://nodejs.org/dist/v14.18.0/node-v14.18.0-darwin-x64.tar.gz ?

This issue for for arm based macs as less than <16 we don't have arm based tar only darwin-x64 are there so in those when we try to install v14 then expected file based on architecture (darwin-arm64) don't exists.

So this pr make changes to consider arch as x64 for version less than 16 for darwin.

image

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.

2 participants

X Tutup