@stdlib/os-arch returns architecture of node, not of the cpu.

[ArmelChesnais]

Description

pulling arch when running in an x64 build of Node on an arm64 CPU (mac m1) will return x64, not arm64.

This was discovered while trying to determine the CPU arch of an electron app running an x64 version of the app on an m1 mac (arm64).

Related Issues

No response

Questions

No.

Demo

No response

Reproduction

No response

Expected Results

arch value should be `arm64` when running in a NodeJS x64 build running on an arm64 CPU

Actual Results

arch value returns the NodeJS build arch, not the CPU arch.

Version

No response

Environments

Node.js

Browser Version

No response

Node.js / npm Version

16.15.0

Platform

No response

Checklist

• Read and understood the Code of Conduct.
• Searched for existing issues and pull requests.

[stdlib-bot]

🎉 Welcome! 🎉

And thank you for opening your first issue! We will get back to you shortly. 🏃 💨

[kgryte]

@ArmelChesnais Thanks for filing this issue. @stdlib/os-arch returns the value of process.arch, which is supplied by Node.js. So if the architecture is not correct, this would be due to an upstream issue in Node.js core.

[kgryte]

...or in this case, this could be due to how the electron app was built.

In short, I don't think the issue lies with stdlib, I'm afraid.

[ArmelChesnais]

it may be a good idea to update the documentation in that case, as it is then currently incorrect:
Operating system CPU architecture.
Should instead indicate that it reports the architecture that NodeJS was built for, not that of the system itself is.
NodeJS build architecture Target
Or along those lines.

As per NodeJS documentation:
The operating system CPU architecture for which the Node.js binary was compiled.

Which is incidentally being differentiated in upcoming Node versions through the os module offering os.arch() and os.machine(), which indicate the difference between the NodeJS build and the actual CPU architecture.

[kgryte]

Thanks for the suggestion. Agreed. We should update the description.

[kgryte]

The package description was updated in 7d6544c and will be part of the next release.

[kgryte]

This has been resolved: https://github.com/stdlib-js/os-arch/tree/f7d812a2dcc40bf68b37d2e7ebd58e7817abf62a. Closing...