Skip to content

[fix][client-cpp] Fix static compilation that includes openssl - #17613

Closed
lhotari wants to merge 1 commit into
apache:masterfrom
lhotari:lh-fix-static-compilation-with-openssl
Closed

[fix][client-cpp] Fix static compilation that includes openssl#17613
lhotari wants to merge 1 commit into
apache:masterfrom
lhotari:lh-fix-static-compilation-with-openssl

Conversation

@lhotari

@lhotari lhotari commented Sep 13, 2022

Copy link
Copy Markdown
Member

Motivation

Master branch is broken at the moment.

../lib/.libs/libcurl.so: undefined reference to `SSL_CTX_set_keylog_callback@OPENSSL_1_1_1'
../lib/.libs/libcurl.so: undefined reference to `SSL_CTX_set_ciphersuites@OPENSSL_1_1_1'
collect2: error: ld returned 1 exit status
make[2]: *** [curl] Error 1
Makefile:839: recipe for target 'curl' failed
make[2]: Leaving directory '/curl-7.61.0/src'
make[1]: *** [all-recursive] Error 1
Makefile:1878: recipe for target 'all-recursive' failed
make[1]: Leaving directory '/curl-7.61.0/src'
Makefile:927: recipe for target 'all-recursive' failed
make: *** [all-recursive] Error 1

Modifications

Fix the static compilation that intends to include a locally built version of openssl. The linking configuration was invalid.

Documentation

  • doc-required
    (Your PR needs to update docs and you will update later)

  • doc-not-needed
    (Please explain why)

  • doc
    (Your PR contains doc changes)

  • doc-complete
    (Docs have been already added)

@lhotari lhotari added this to the 2.12.0 milestone Sep 13, 2022
@lhotari lhotari self-assigned this Sep 13, 2022
@lhotari

lhotari commented Sep 13, 2022

Copy link
Copy Markdown
Member Author

closing since @BewareMyPower will provide another PR with an improved solution

@lhotari lhotari closed this Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant