-
Notifications
You must be signed in to change notification settings - Fork 1
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
chore: update nodejs mobile to v18 #194
Conversation
// We use NDK >= 24 due to NodeJS Mobile React Native | ||
ndkVersion = "24.0.8215888" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this was strictly necessary since we don't really rely on the ndk for nodejs-mobile, but maybe better to keep here (i didn't try without this change)
return `${name}-${version}-node-${NODE_ABI}-${target}.tar.gz`; | ||
function getNodeJsMobileNodeVersions() { | ||
const nodeVersionFilePath = new URL( | ||
'android/libnode/include/node/node_version.h', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this corresponds to the header file that the NodeJS project places several version-related constants in
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything (that was working before) seems to be working!
This removes Node 16 support, given that (1) it is deprecated (2) it [is no longer used in CoMapeo][0]. [0]: digidem/comapeo-mobile#194
This removes Node 16 support, given that (1) it is deprecated (2) it [is no longer used in CoMapeo][0]. [0]: digidem/comapeo-mobile#194
This removes Node 16 support, given that (1) it is deprecated (2) it [is no longer used in CoMapeo][0]. [0]: digidem/comapeo-mobile#194
This removes Node 16 support, given that (1) it is deprecated (2) it [is no longer used in CoMapeo][0]. [0]: digidem/comapeo-mobile#194
This removes Node 16 support, given that (1) it is deprecated (2) it [is no longer used in CoMapeo][0]. [0]: digidem/comapeo-mobile#194
This removes Node 16 support, given that (1) it is deprecated (2) it [is no longer used in CoMapeo][0]. [0]: digidem/comapeo-mobile#194
This removes Node 16 support, given that (1) it is deprecated (2) it [is no longer used in CoMapeo][0]. [0]: digidem/comapeo-mobile#194
This removes Node 16 support, given that (1) it is deprecated (2) it [is no longer used in CoMapeo][0]. [0]: digidem/comapeo-mobile#194
Towards #171
Confirmation log via adb: