-
Notifications
You must be signed in to change notification settings - Fork 27
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
CMake integration #10
Comments
Sure. If you're willing to make a nuget/conan package, that would be great too. Try to get it working in both CI systems with CMake too. |
I'm not a conan expert, although I'm using it for cpprestsdk. One of the problems that I've found out, is there's no "findCpprestsdk.cmake" file, so we can't do a |
So, will you build it as part of the cmake build? |
Seems like v2.10.0 will support
find_package(cpprestsdk REQUIRED)
target_link_libraries(my_executable PRIVATE cpprestsdk::cpprest) I've forked this repo and will work on it 😄 |
P.S. updated the submodules |
As far as I see, no breaking changes. I'll update my remote with this. I'll propose a PR for this issue within some days. |
How is it going? I see there's even a fork of your fork |
Hi, I'm really sorry for not doing the MR sooner. I found a problem with CppReskSDK: their FindXXX.cmake is not working properly when you use Conan; they fixed it on 2.10, but 2.10 is not yet available on conan packages by bincrafters (it's on testing branch). As a result, I was waiting for it to go stable in order to propose the MR. |
ok. Let's wait |
Hi, would you accept a PR with CMake integration? I would like to use this as a third party library, but it is hard to integrate this repo as a submodule.
The text was updated successfully, but these errors were encountered: