[BUG] Libcurl/OSSL issues: Had to bypass libcurl library SDK was using to get it to work

[zuluecho9]

Logging

Describe the bug
My attempt to run cmake -DBUILD_GSTREAMER_PLUGIN=ON .. ran into problems...

Claude summary of problem: "The SDK is building libcurl 7.74.0 (from December 2020), which is too old to properly support OpenSSL 3.0+ (released September 2021). The linker errors happen because curl 7.74.0 detects OpenSSL 3.6.0 headers and tries to use the new functions, but it doesn't have the proper linking configuration for OpenSSL 3.0+."

This led to quite a few other errors. The solution Claude arrived at was to set env variable OPENSSL_ROOT_DIR to run the local OpenSSL instead of one in package

To Reproduce
Steps to reproduce the behavior:
Running cmake -DBUILD_GSTREAMER_PLUGIN=ON .. on a mac via terminal

Platform (please complete the following information):

• Mac

Additional context
Add any other context about the problem here.

[sirknightj]

HI @zuluecho9, I didn't see any logs shared in the issue. If you could share those, that would help us to determine the issue.

For MacOS, we recommend building with the -DBUILD_DEPENDENCIES=OFF flag: https://github.com/awslabs/amazon-kinesis-video-streams-producer-sdk-cpp/blob/v3.5.0/.github/workflows/samples.yml#L128

Please remember to perform a clean build when changing the CMake flags. Delete the build, open-source, dependency folders if they exist.

However, do note I was able to build the latest (3.5.0) on my Mac with dependencies on (default) without any issues. Do note that you must be using CMake 3.x version. CMake 4 versions are not currently supported.

cmake --version
cmake version 3.30.0

cmake .. -DBUILD_GSTREAMER_PLUGIN=ON

11%] Creating directories for 'libkvspic-download'
[ 22%] Performing download step (git clone) for 'libkvspic-download'
Cloning into 'kvspic-src'...
HEAD is now at c98c2a2 Release 1.2.0 (#267)
[ 33%] Performing update step for 'libkvspic-download'
-- Already at requested tag: v1.2.0
[ 44%] No patch step for 'libkvspic-download'
[ 55%] No configure step for 'libkvspic-download'
[ 66%] No build step for 'libkvspic-download'
[ 77%] No install step for 'libkvspic-download'
[ 88%] No test step for 'libkvspic-download'
[100%] Completed 'libkvspic-download'
[100%] Built target libkvspic-download
-- Using system default stack size
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Success
-- Found Threads: TRUE
-- Found OpenSSL: /Users/me/Downloads/test-99/amazon-kinesis-video-streams-producer-sdk-cpp/open-source/local/lib/libcrypto.dylib (found version "1.1.1t")
-- Checking for module 'libcurl'
--   Found libcurl, version 7.74.0-DEV
-- Bitness 64 bits
-- Found OpenSSL: /Users/me/Downloads/test-99/amazon-kinesis-video-streams-producer-sdk-cpp/open-source/local/lib/libcrypto.dylib (found suitable version "1.1.1t", minimum required is "1")
-- Found ZLIB: /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX26.0.sdk/usr/lib/libz.tbd (found suitable version "1.2.12", minimum required is "1")
found log4cplus.
-- Found Log4cplus: /Users/me/Downloads/test-99/amazon-kinesis-video-streams-producer-sdk-cpp/open-source/local/lib/liblog4cplus.dylib
-- Configuring done (333.5s)
-- Generating done (0.1s)
-- Build files have been written to: /Users/me/Downloads/test-99/amazon-kinesis-video-streams-producer-sdk-cpp/build

make -j

[zuluecho9]

Thank you. I didn't know to use cmake version 3 so I suspect that is it. Will try soon.

[zuluecho9]

I was curious if there was somewhere in docs it said that Mac required cmake < 4.0. Mainly just curious if I missed something obvious or if that might benefit from calling out more.

[sirknightj]

We haven't added it to the documentation yet.

From my notes here: awslabs/amazon-kinesis-video-streams-producer-c#479 the curl version and the libwebsockets version currently used both require cmake 3 to build. It should apply to all platforms, not just mac.

If you would like to make a contribution to update the readme, feel free. Otherwise we'll update it in the upcoming days.