From 77dea6aace7abfb3f4e108b0cc690baa2144d23a Mon Sep 17 00:00:00 2001
From: Patrick Niklaus <patrick@mapbox.com>
Date: Thu, 1 Feb 2018 12:54:22 +0000
Subject: [PATCH] Include node headers as system headers

This will avoid problems were GCC/clang might trigger useless warnings induced by including the `v8` header.
---
 NodeJS.cmake | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/NodeJS.cmake b/NodeJS.cmake
index 676e4e0..7cad0bc 100644
--- a/NodeJS.cmake
+++ b/NodeJS.cmake
@@ -600,7 +600,7 @@ function(add_nodejs_module NAME)
         PUBLIC ${NODEJS_DEFINITIONS}
     )
     # This properly defines includes for the module
-    target_include_directories(${NAME} PUBLIC ${NODEJS_INCLUDE_DIRS})
+    target_include_directories(${NAME} SYSTEM PUBLIC ${NODEJS_INCLUDE_DIRS})
 
     # Add link flags to the module
     target_link_libraries(${NAME} ${NODEJS_LIBRARIES})