diff options
author | Viktor Szakats <commit@vsz.me> | 2021-10-25 19:10:55 +0000 |
---|---|---|
committer | Viktor Szakats <commit@vsz.me> | 2021-10-25 19:10:55 +0000 |
commit | 13bbf81c59fd2c5c9ba552ced4eec8dbcf7028d9 (patch) | |
tree | baf24f58158957db1bdce351c1c227a273a3a624 /src | |
parent | 16a6b02df29494df05cc4de003c5c94d14c37688 (diff) | |
download | curl-13bbf81c59fd2c5c9ba552ced4eec8dbcf7028d9.tar.gz |
Makefile.m32: fix to not require OpenSSL with -libssh2 or -rtmp options
Previously, -libssh2/-rtmp options assumed that OpenSSL is also enabled
(and then failed with an error when not finding expected OpenSSL headers),
but this isn't necessarly true, e.g. when building both libssh2 and curl
against Schannel. This patch makes sure to only enable the OpenSSL backend
with -libssh2/-rtmp, when there was no SSL option explicitly selected.
- Re-implement the logic as a single block of script.
- Also fix an indentation while there.
Assisted-by: Jay Satiro
Closes #7895
Diffstat (limited to 'src')
-rw-r--r-- | src/Makefile.m32 | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/src/Makefile.m32 b/src/Makefile.m32 index 380d264e7..d769bad75 100644 --- a/src/Makefile.m32 +++ b/src/Makefile.m32 @@ -187,12 +187,10 @@ SYNC = 1 endif ifeq ($(findstring -rtmp,$(CFG)),-rtmp) RTMP = 1 -SSL = 1 ZLIB = 1 endif ifeq ($(findstring -ssh2,$(CFG)),-ssh2) SSH2 = 1 -SSL = 1 ZLIB = 1 endif ifeq ($(findstring -ssl,$(CFG)),-ssl) @@ -242,6 +240,13 @@ ifeq ($(findstring -unicode,$(CFG)),-unicode) UNICODE = 1 endif +# SSH2 and RTMP require an SSL library; assume OpenSSL if none specified +ifneq ($(SSH2)$(RTMP),) + ifeq ($(SSL)$(WINSSL),) + SSL = 1 + endif +endif + INCLUDES = -I. -I../include -I../lib ifdef SSL ifdef WINSSL @@ -309,7 +314,7 @@ ifdef SSL endif endif ifneq "$(wildcard $(OPENSSL_INCLUDE)/openssl/opensslv.h)" "$(OPENSSL_INCLUDE)/openssl/opensslv.h" - $(error Invalid path to OpenSSL package: $(OPENSSL_PATH)) + $(error Invalid path to OpenSSL package: $(OPENSSL_PATH)) endif ifndef OPENSSL_LIBPATH OPENSSL_LIBS = -lssl -lcrypto |