Skip to content

Commit b21c540

Browse files
committed
cmake: add openssl to the private deps list when it's the TLS implementation
We might want OpenSSL to be the implementation for SHA-1 and/or TLS. If we only want it for TLS (e.g. we're building with the collision-detecting SHA-1 implementation) then we did not indicate this to the systems including us a static library. Add OpenSSL to the list also during the TLS decision to make sure we say we should link to it if we use it for TLS.
1 parent b85548e commit b21c540

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

src/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -184,6 +184,7 @@ IF (USE_HTTPS)
184184
LIST(APPEND LIBGIT2_INCLUDES ${OPENSSL_INCLUDE_DIR})
185185
LIST(APPEND LIBGIT2_LIBS ${OPENSSL_LIBRARIES})
186186
LIST(APPEND LIBGIT2_PC_LIBS ${OPENSSL_LDFLAGS})
187+
LIST(APPEND LIBGIT2_PC_REQUIRES "openssl")
187188
ELSEIF (HTTPS_BACKEND STREQUAL "WinHTTP")
188189
# WinHTTP setup was handled in the WinHTTP-specific block above
189190
ELSE()

0 commit comments

Comments
 (0)