From 7dcc4af8277744eb275a5f5cf43316688460f5e7 Mon Sep 17 00:00:00 2001 From: Ben Noordhuis Date: Wed, 29 Jun 2016 21:11:14 +0200 Subject: [PATCH] build: drop unconditional openssl dep from cctest Don't link in openssl when building `./configure --without-inspector`, it's only used by the inspector cctests. Ditto libuv and http_parser. Fixes unnecessarily building openssl when `--shared-openssl` is also passed to configure. Fixes: https://github.com/nodejs/node/issues/7478 PR-URL: https://github.com/nodejs/node/pull/7486 Reviewed-By: Colin Ihrig Reviewed-By: James M Snell --- node.gyp | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/node.gyp b/node.gyp index 51fe580af7d..a1d5e54ffbe 100644 --- a/node.gyp +++ b/node.gyp @@ -749,12 +749,7 @@ { 'target_name': 'cctest', 'type': 'executable', - 'dependencies': [ - 'deps/openssl/openssl.gyp:openssl', - 'deps/http_parser/http_parser.gyp:http_parser', - 'deps/gtest/gtest.gyp:gtest', - 'deps/uv/uv.gyp:libuv', - ], + 'dependencies': [ 'deps/gtest/gtest.gyp:gtest' ], 'include_dirs': [ 'src', 'deps/v8/include'