Skip to content

(v7.x backport) src: guard bundled_ca/openssl_ca with HAVE_OPENSSL

Rodrigo Muino Tomonari requested to merge github/fork/danbev/v7x-guard-use_bundled_ca into v7.x-staging

Currently, the following warning will be reported when configuring without-ssl:

../src/node.cc:3653:8: warning: unused variable 'use_bundled_ca' [-Wunused-variable] bool use_bundled_ca = false; ^ ../src/node.cc:3654:8: warning: unused variable 'use_openssl_ca' [-Wunused-variable] bool use_openssl_ca = false; ^

I missed this when working on commit 8a7db9d4 ("src: add --use-bundled-ca --use-openssl-ca check").

Refs: https://github.com/nodejs/node/pull/12087 PR-URL: https://github.com/nodejs/node/pull/12302 Reviewed-By: Colin Ihrig cjihrig@gmail.com Reviewed-By: Richard Lau riclau@uk.ibm.com Reviewed-By: Anna Henningsen anna@addaleax.net

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

Merge request reports