build: add checks for openssl configure options

Currently it is possible to configure using --without-ssl and --shared-openssl/--openssl-no-asm/openssl-fips without an error occuring. The commit add check for these combinations: $ ./configure --without-ssl --shared-openssl Error: --without-ssl is incompatible with --shared-openssl $ ./configure --without-ssl --openssl-no-asm Error: --without-ssl is incompatible with --openssl-no-asm $ ./configure --without-ssl --openssl-fips=dummy Error: --without-ssl is incompatible with --openssl-fips PR-URL: #12175 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Richard Lau <[email protected]>
nodejs · Apr 19, 2017 · 986ef6f · 986ef6f
1 parent c2c467e
commit 986ef6f
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/configure b/configure
@@ -965,6 +965,15 @@ def configure_openssl(o):
 
 
   if options.without_ssl:
+    def without_ssl_error(option):
+      print('Error: --without-ssl is incompatible with %s' % option)
+      exit(1)
+    if options.shared_openssl:
+      without_ssl_error('--shared-openssl')
+    if options.openssl_no_asm:
+      without_ssl_error('--openssl-no-asm')
+    if options.openssl_fips:
+      without_ssl_error('--openssl-fips')
     return
   configure_library('openssl', o)