build: do not copy v8-inspector* headers ar part of install

These headers are exposed from V8 for embedder and should not be used by native addons. Fixes #22415 PR-URL: #22586 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
nodejs · Aug 31, 2018 · 4b47d29 · 4b47d29
1 parent 21e8ce2
commit 4b47d29
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/tools/install.py b/tools/install.py
@@ -157,6 +157,14 @@ def files(action):
   headers(action)
 
 def headers(action):
+  def ignore_inspector_headers(files, dest):
+    inspector_headers = [
+      'deps/v8/include/v8-inspector.h',
+      'deps/v8/include/v8-inspector-protocol.h'
+    ]
+    files = filter(lambda name: name not in inspector_headers, files)
+    action(files, dest)
+
   action([
     'common.gypi',
     'config.gypi',
@@ -172,7 +180,7 @@ def headers(action):
   if sys.platform.startswith('aix'):
     action(['out/Release/node.exp'], 'include/node/')
 
-  subdir_files('deps/v8/include', 'include/node/', action)
+  subdir_files('deps/v8/include', 'include/node/', ignore_inspector_headers)
 
   if 'false' == variables.get('node_shared_libuv'):
     subdir_files('deps/uv/include', 'include/node/', action)