Skip to content

Add openssl include path for sockets_posix target #1947

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jul 18, 2025
Merged

Add openssl include path for sockets_posix target #1947

merged 1 commit into from
Jul 18, 2025

Conversation

@gmtt
Copy link
Contributor

@gmtt gmtt commented Jul 15, 2025

Issue #, if available:
When compile device sdk as shared library, compiler complaining openssl/ssl.h header file not found, in the cmakelists.txt, it has find_package(OpenSSL), but the OPENSSL_INCLUDE_DIR is not used anywhere, it only linked OPENSSL_LIBRARIES

The cmake configure and compile command I used:

cmake -DCMAKE_TOOLCHAIN_FILE=$IOTMI_TOOLCHAIN_PATH/toolchain.cmake \                                                                                                                                            
  -DCMAKE_INSTALL_PREFIX=$SYSROOT_PATH/usr/local \
  -DCMAKE_PREFIX_PATH=$SYSROOT_PATH/usr/local \
  -DCMAKE_BUILD_TYPE=Release \
  -DBUILD_DEMOS=OFF \
  -DBUILD_CLONE_SUBMODULES=OFF \
  -DDOWNLOAD_CERTS=OFF \
  -DINSTALL_TO_SYSTEM=ON \
  -DBUILD_SHARED_LIBS=ON \
  ..

cmake --build . --target install

Description of changes:
Add OPENSSL_INCLUDE_DIR to include directories for target sockets_posix

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

@xuelix xuelix self-assigned this Jul 18, 2025
@xuelix xuelix merged commit 3d170a0 into aws:main Jul 18, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xuelix xuelix xuelix approved these changes

+2 more reviewers

@thomas-roos thomas-roos thomas-roos approved these changes

@aggarg aggarg aggarg approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@xuelix xuelix

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@gmtt @thomas-roos @aggarg @xuelix