Skip to content
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

nodejs: Bump to 16.11.0 #7710

Merged
merged 1 commit into from
Oct 9, 2021
Merged

nodejs: Bump to 16.11.0 #7710

merged 1 commit into from
Oct 9, 2021

Conversation

thunder-coding
Copy link
Member

@thunder-coding thunder-coding commented Oct 9, 2021

Other changes done:

  • Removed no-android-log.patch since Node.js uses GYP and not GN to
    build itself. This patch must be having no effect at all
  • Added patch to force V8_TRAP_HANDLER_SUPPORTED to always false which fixes builds for x86_64 which started reproducing even on aarch64
  • Removed x86_64 as blacklisted arch

Since we have got how to fix builds for nodejs for x86_64 let's port this fix to nodejs-lts too 🤞

Manual Tests to be done

  • Check functioning of npm. npm has been updated to v8 in Node.js 16.11.0

@thunder-coding thunder-coding changed the title nodejs: Bump to 16.10.0 nodejs: Bump to 16.11.0 Oct 9, 2021
@thunder-coding
Copy link
Member Author

Oops, in my commit message I wrote 16.10.0 instead of 16.11.0 😛. Wait for CI to pass. Will rebase then after doing some manual testing

@thunder-coding
Copy link
Member Author

The earlier x86_64 only undefined reference to v8::internal::trap_handler::TryHa ndleSignal(int, siginfo_t*, void*) seem to be also now present on aarch64. This seems to be a 64 bit problem now. Both archs should be requiring a similar fix.

Other changes done:

- Removed `no-android-log.patch` since Node.js uses GYP and not GN to
  build itself. This patch must be having no effect at all
- Added patch to force `V8_TRAP_HANDLER_SUPPORTED` to always false which
  fixes builds for x86_64 which started reproducing even on aarch64
- Removed `x86_64` as blacklisted arch
@thunder-coding
Copy link
Member Author

Ready to merge. The previous CI passed and the debs also seem to be well ✅

@landfillbaby landfillbaby merged commit 154bd59 into termux:master Oct 9, 2021
@thunder-coding thunder-coding deleted the nodejs-v16.11 branch October 10, 2021 01:41
metayan pushed a commit to metayan/termux-packages that referenced this pull request Nov 28, 2021
Other changes done:

- Removed `no-android-log.patch` since Node.js uses GYP and not GN to
  build itself. This patch must be having no effect at all
- Added patch to force `V8_TRAP_HANDLER_SUPPORTED` to always false which
  fixes builds for x86_64 which started reproducing even on aarch64
- Removed `x86_64` as blacklisted arch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants