Skip to content

src: allow inspector without v8 platform

Rodrigo Muino Tomonari requested to merge github/fork/codebytere/no-error-inspector into master

Related to https://github.com/nodejs/node/pull/30029.

Embedders sometimes want to take control of V8 platform initialization themselves, so we wouldn't want to experience an error if NODE_USE_V8_PLATFORM was false necessarily. Instead, we can also gate it with HAVE_INSPECTOR to allow embedders to define that themselves and use Inspector without NODE_USE_V8_PLATFORM.

We currently patch this in ourselves, so this would help us further reduce patch surface. Additionally, if there are thoughts on a more robust way to handle this case I would welcome them!

cc @joyeecheung

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines

Merge request reports