Skip to content

v8: include Android trap handler sources - #65774

Open
luoqianlin wants to merge 1 commit into
nodejs:mainfrom
luoqianlin:android-v8-trap-handler-sources
Open

v8: include Android trap handler sources#65774
luoqianlin wants to merge 1 commit into
nodejs:mainfrom
luoqianlin:android-v8-trap-handler-sources

Conversation

@luoqianlin

@luoqianlin luoqianlin commented Sep 4, 2026

Copy link
Copy Markdown

Allow the V8 POSIX trap-handler sources needed by Android cross-builds to be
compiled without enabling the Android trap handler.

Android remains explicitly unsupported for trap-handler activation because its
signal and crash-reporting behavior requires a separate security review. This
change adds the required POSIX source files, including the host simulator
helper needed by arm64-on-x64 configurations. It does not enable trap-handler
activation on Android, and the existing V8 Android safety guard remains
unchanged.

NDK r29 with Android API 30 and arm64 is the current validation baseline.

Refs: #65771

Validation:

  • git diff --check
  • Android arm64/API 30 GYP generation includes the required POSIX source files.
  • V8_TRAP_HANDLER_SUPPORTED remains false for Android native and
    arm64-on-x64 simulator configurations.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/gyp
  • @nodejs/security-wg
  • @nodejs/v8-update

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. v8 engine Issues and PRs related to the V8 dependency. labels Sep 4, 2026
@MikeMcC399 MikeMcC399 added the android Issues and PRs related to the Android platform. label Sep 4, 2026
@luoqianlin
luoqianlin force-pushed the android-v8-trap-handler-sources branch from 50ba292 to 3bd87a6 Compare September 4, 2026 12:35
@luoqianlin
luoqianlin requested a review from a team as a code owner September 4, 2026 12:35
@luoqianlin
luoqianlin force-pushed the android-v8-trap-handler-sources branch from 3bd87a6 to d9f1c2d Compare September 4, 2026 12:39
Signed-off-by: luoqianlin <qianlinluo@foxmail.com>
@luoqianlin
luoqianlin force-pushed the android-v8-trap-handler-sources branch from d9f1c2d to 98da35b Compare September 4, 2026 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

android Issues and PRs related to the Android platform. needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. v8 engine Issues and PRs related to the V8 dependency.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants