Skip to content

[package] cpprestsdk/2.10.19: Enable transitive dependencies #27492

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
cosminpolifronie opened this issue May 21, 2025 · 4 comments
Open

[package] cpprestsdk/2.10.19: Enable transitive dependencies #27492

cosminpolifronie opened this issue May 21, 2025 · 4 comments
Labels
bug Something isn't working

Comments

@cosminpolifronie
Copy link

Description

When using this package, you have to manually include headers for boost and OpenSSL. This is probably because requirements do not have transitive_headers=True

Package and Environment Details

  • Package Name/Version: cpprestsdk/2.10.19
  • Operating System+version: Windows 11
  • Compiler+version: MSVC 194
  • Docker image: -
  • Conan version: conan 2.16.1
  • Python version: Python 3.13.3

Conan profile

[settings]
os=Windows
arch=x86
build_type=Debug
compiler=msvc
compiler.cppstd=14
compiler.runtime=static
compiler.runtime_type=Debug
compiler.version=194
os=Windows

Steps to reproduce

Use the package in a dummy application, without manually including headers for Boost and OpenSSL.

Logs

Build just fails, not able to find boost/asio/ssl.hpp

@cosminpolifronie cosminpolifronie added the bug Something isn't working label May 21, 2025
@AbrilRBS
Copy link
Member

Hi @cosminpolifronie thanks a lot for the report. I can see that we need to transitively show boost headers for this case, but I can not find why openssl would be needed. If you add only boost, what error de you get for openssl? Thanks

@cosminpolifronie
Copy link
Author

Hello, thanks a lot for the quick response.

Error for OpenSSL:
include/boost/asio/ssl/detail/openssl_types.hpp(23,10): error C1083: Cannot open include file: 'openssl/conf.h': No such file or directory

@cosminpolifronie
Copy link
Author

While we are at it, we also probably need transitive_libs=True for OpenSSL and ZLIB.

@cosminpolifronie
Copy link
Author

Never mind, I have rechecked and they are not needed (transitive_libs).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants