Skip to content

src: add HAVE_OPENSSL directive to openssl_config

Rodrigo Muino Tomonari requested to merge github/fork/danbev/add-have_openssl-directive into v7.x-staging

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 anna@addaleax.net Reviewed-By: James M Snell jasnell@gmail.com Reviewed-By: Colin Ihrig cjihrig@gmail.com Reviewed-By: Ben Noordhuis info@bnoordhuis.nl Reviewed-By: Sam Roberts vieuxtech@gmail.com

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

src

Merge request reports