Skip to content

Commit 1bd6a62

Browse files
bnoordhuisFishrock123
authored andcommitted
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: #7478 PR-URL: #7486 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 6151544 commit 1bd6a62

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

node.gyp

+1-6
Original file line numberDiff line numberDiff line change
@@ -753,12 +753,7 @@
753753
{
754754
'target_name': 'cctest',
755755
'type': 'executable',
756-
'dependencies': [
757-
'deps/openssl/openssl.gyp:openssl',
758-
'deps/http_parser/http_parser.gyp:http_parser',
759-
'deps/gtest/gtest.gyp:gtest',
760-
'deps/uv/uv.gyp:libuv',
761-
],
756+
'dependencies': [ 'deps/gtest/gtest.gyp:gtest' ],
762757
'include_dirs': [
763758
'src',
764759
'deps/v8/include'

0 commit comments

Comments
 (0)