Skip to content
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

Open
promgamer opened this issue Dec 14, 2017 · 9 comments
Open

CMake integration #10

promgamer opened this issue Dec 14, 2017 · 9 comments

Comments

@promgamer
Copy link

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.

@d-led
Copy link
Owner

d-led commented Dec 14, 2017

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.

@promgamer
Copy link
Author

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 find_package(cpprestsdk) easily (I think it will fail if we get cpprestsdk via conan)

@d-led
Copy link
Owner

d-led commented Dec 15, 2017

So, will you build it as part of the cmake build?

@promgamer
Copy link
Author

Seems like v2.10.0 will support find_package. From the release notes:

  • Added CMake config-file support to installation. This should be consumed by doing:
find_package(cpprestsdk REQUIRED)
target_link_libraries(my_executable PRIVATE cpprestsdk::cpprest)

I've forked this repo and will work on it 😄

@d-led
Copy link
Owner

d-led commented Dec 17, 2017

P.S. updated the submodules git submodule update --remote, in case you want to merge upstream

@promgamer
Copy link
Author

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.

@d-led
Copy link
Owner

d-led commented Feb 26, 2018

How is it going? I see there's even a fork of your fork

@promgamer
Copy link
Author

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.

@d-led
Copy link
Owner

d-led commented Feb 27, 2018

ok. Let's wait

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants