Add armv5 to list of compatible machine for nodejs_6

imyller / meta-nodejs

OpenEmbedded layer for latest Node.js releases

MIT License

79 stars 87 forks source link

Closed nghiant2710 closed 8 years ago

nghiant2710 commented 8 years ago

We can confirm that node.js 6 work fine with few ARMv5 configurations, not all of them as my colleague mentioned, ref: https://github.com/nodejs/node/blob/master/deps/v8/src/base/atomicops_internals_arm_gcc.h#L159-L161

imyller commented 8 years ago

Thank you for confirming ARMv5 compatibility. Merging.

nghiant2710 commented 8 years ago

You're welcome! Thanks for the fast response :+1:

imyller commented 8 years ago

As a side note, I looked up the Node.js v4 LTS and Node.js v5 sources and they have the same ARMv5 archs supported.

I'm going to enable ARMv5 compiling also for those release lines.

edit: double oops. ARMv5 was never disabled for those ;) Never mind.