Skip to content

Cmake log output for found dependent libraries OpenSSL and Boost, Add vcpkg.json [DEX-431] #1320

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.

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ihsandemir
Copy link
Collaborator

@ihsandemir ihsandemir commented Jul 14, 2025

Added cmake log output for found Boost library and OpenSSL. This added log will print the found OpenSSL and Boost versions to be used while building the project. It is written in configure stage of cmake.

Added the vcpkg.json for building the project using vcpkg. This helps building the project in terms of dealing with its dependencies. When you do cmake .. -DCMAKE_TOOLCHAIN_FILE=/Users/ihsan/Desktop/work/src/vcpkg/scripts/buildsy or you do vcpkg install it installs the needed Boost and OpenSSL dependencies if missing and uses the vcpkg tool chain to compile the cpp client source code.

Added the vcpkg.json for building the project using vcpkg.
@ihsandemir ihsandemir added this to the 5.5.0 milestone Jul 14, 2025
@ihsandemir ihsandemir requested review from yuce and JackPGreen July 14, 2025 12:24
@ihsandemir ihsandemir self-assigned this Jul 14, 2025
@ihsandemir ihsandemir changed the title Cmake log output for found dependent libraries OpenSSL and Boost, Add vcpkg.json Cmake log output for found dependent libraries OpenSSL and Boost, Add vcpkg.json [DEX-431] Jul 14, 2025
@JackPGreen
Copy link
Contributor

@ihsandemir can you add some more context around what this is doing and why, please?

@ihsandemir
Copy link
Collaborator Author

@ihsandemir can you add some more context around what this is doing and why, please?

Just added. Let me know if that is enough.

@JackPGreen
Copy link
Contributor

@ihsandemir can you add some more context around what this is doing and why, please?

Just added. Let me know if that is enough.

Thanks. Is there a particular reason we need to log the version?

@ihsandemir
Copy link
Collaborator Author

@ihsandemir can you add some more context around what this is doing and why, please?

Just added. Let me know if that is enough.

Thanks. Is there a particular reason we need to log the version?

That is for trouble shooting and making sure the location for the used library was as I expected.

JackPGreen
JackPGreen previously approved these changes Jul 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants