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: nodejs#12175 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Richard Lau <[email protected]>
italoacasas · Apr 10, 2017 · a703bde · a703bde
1 parent b495b6a
commit a703bde
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/configure b/configure
@@ -977,6 +977,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)