arch icon indicating copy to clipboard operation
arch copied to clipboard

feat: use simpler env vars for win32

Open CanadaHonk opened this issue 2 years ago • 4 comments

Just use environment variables PROCESSOR_ARCHITEW6432/PROCESSOR_ARCHITECTURE on win32 instead of checking paths. Closes #26.

Bonus when running as x86: probably much faster and works with no IO access. Also ~10% smaller filesize.

CanadaHonk avatar Nov 22 '23 19:11 CanadaHonk

Agreed, this is a lot simpler and probably faster. @feross do you see something wrong with this approach?

XhmikosR avatar Feb 12 '24 08:02 XhmikosR

Fixed formatting to be like the rest of the repo.

CanadaHonk avatar Feb 13 '24 22:02 CanadaHonk

Updated to return 'arm64' on such systems now too.

CanadaHonk avatar Feb 13 '24 22:02 CanadaHonk

Removed now unused imports in index.js.

CanadaHonk avatar Feb 13 '24 22:02 CanadaHonk