search

eBay / NuRaft

C++ implementation of Raft core logic as a replication library
Apache License 2.0
996 stars 236 forks source link

Clean up CMakeLists #450

Closed JosiahWI closed 1 year ago

JosiahWI commented 1 year ago

Hello, I'm a contributor to Apache Trafficserver and I'm part of an effort to move our build to CMake. One of our plugins depends on NuRaft, and I'm investigating how we should detect it in our build. It doesn't look like you export your targets, so will we have to write our own Find script? I am willing to PR the necessary changes to your CMakeLists.txt to export targets, to make this cleaner in the future. At the same time, I'm willing to do some other clean up; in particular, I am curious why you wrote your own logic to detect OpenSSL when find_package(OpenSSL) exists.

JosiahWI commented 1 year ago

Looks like #358 already addresses that latter concern. I would be happy to pick up that PR if the original author is no longer working on it.

greensky00 commented 1 year ago

Hi @JosiahWI please go ahead and make a PR about exporting target. Let me review it.

Regarding Open SSL -- we don't want to break existing building script with pre-configured library path. If you can take over the PR and address the things I mentioned in the comment, that will be great. Thanks.