From e1d8899c28997613505d288d22bfb95470d606a1 Mon Sep 17 00:00:00 2001 From: Daniel Bevenius Date: Tue, 28 Feb 2017 20:04:12 +0100 Subject: [PATCH] src: add HAVE_OPENSSL directive to openssl_config Currently when building with the following configuration options: $ ./configure --without-ssl && make The following link error is reported: Undefined symbols for architecture x86_64: "node::openssl_config", referenced from: node::Init(int*, char const**, int*, char const***) in node.o ld: symbol(s) not found for architecture x86_64 clang: error: linker command failed with exit code 1 (use -v to see invocation) Adding an HAVE_OPENSSL directive around this code allows the build to pass. PR-URL: https://github.com/nodejs/node/pull/11618 Reviewed-By: Anna Henningsen Reviewed-By: James M Snell Reviewed-By: Colin Ihrig Reviewed-By: Ben Noordhuis Reviewed-By: Sam Roberts --- src/node.cc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/node.cc b/src/node.cc index 57415bba418..ec78339d89a 100644 --- a/src/node.cc +++ b/src/node.cc @@ -4233,8 +4233,10 @@ void Init(int* argc, if (config_warning_file.empty()) SafeGetenv("NODE_REDIRECT_WARNINGS", &config_warning_file); +#if HAVE_OPENSSL if (openssl_config.empty()) SafeGetenv("OPENSSL_CONF", &openssl_config); +#endif // Parse a few arguments which are specific to Node. int v8_argc;