Skip to content

Commit 46bd32e

Browse files
danbevjasnell
authored andcommitted
build: fix openssl link error on windows
This commit attempts to fix an issue when building on windows using the following command line options: .\vcbuild.bat dll debug x64 vc2015 This will result in the following options passed to configure: configure --debug --shared --dest-cpu=x64 --tag= This commit excludes the dependency to openssl if node is configured with --shared. Also, FP_API to the categories to export in mkssldef when generating the module definition (openssl.def) allowing the build to compile and link successfully. Fixes: #12952 PR-URL: #13078 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Refael Ackermann <[email protected]>
1 parent 6e4394f commit 46bd32e

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

node.gyp

+2-2
Original file line numberDiff line numberDiff line change
@@ -277,7 +277,7 @@
277277
# Categories to export.
278278
'-CAES,BF,BIO,DES,DH,DSA,EC,ECDH,ECDSA,ENGINE,EVP,HMAC,MD4,MD5,'
279279
'NEXTPROTONEG,PSK,RC2,RC4,RSA,SHA,SHA0,SHA1,SHA256,SHA512,SOCK,'
280-
'STDIO,TLSEXT',
280+
'STDIO,TLSEXT,FP_API',
281281
# Defines.
282282
'-DWIN32',
283283
# Symbols to filter from the export list.
@@ -671,7 +671,7 @@
671671
'deps/zlib/zlib.gyp:zlib',
672672
]
673673
}],
674-
[ 'node_shared_openssl=="false"', {
674+
[ 'node_shared_openssl=="false" and node_shared=="false"', {
675675
'dependencies': [
676676
'deps/openssl/openssl.gyp:openssl'
677677
]

0 commit comments

Comments
 (0)