From 373ffc5bad3d24f6ca3f1fcbd0e1a3d5bb7dcef4 Mon Sep 17 00:00:00 2001 From: Ben Noordhuis Date: Sat, 21 May 2016 11:14:05 +0200 Subject: [PATCH] installer: don't install node_internals.h MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit As the name suggests, it's for internal use only, so don't install it. Including it in an add-on doesn't work because the file depends on other header files that are not installed. Adding it to the install list appears to have been an oversight in commit 32478acf ("build: unix install node and dep library headers"). PR-URL: https://github.com/nodejs/node/pull/6913 Reviewed-By: Colin Ihrig Reviewed-By: James M Snell Reviewed-By: Johan Bergström Reviewed-By: Robert Jefe Lindstaedt --- tools/install.py | 1 - 1 file changed, 1 deletion(-) diff --git a/tools/install.py b/tools/install.py index 88791eab10..b1997d4852 100755 --- a/tools/install.py +++ b/tools/install.py @@ -150,7 +150,6 @@ def headers(action): 'config.gypi', 'src/node.h', 'src/node_buffer.h', - 'src/node_internals.h', 'src/node_object_wrap.h', 'src/node_version.h', ], 'include/node/')