Summary:
Find folly using its installed CMake configuration file, and delete the custom
FindFolly.cmake file that was being used previously. The FindFolly.cmake file
did not list all dependent libraries necessary to link statically against
folly. Linking against folly as a shared library is not recommended, as
folly provides no binary compatibility guarantees between changes. folly's
current CMake-based build only builds it as a static library by default.
This also changes the build process to build folly with CMake rather than its
deprecated autotools-based build when RSOCKET_INSTALL_DEPS or INSTALL_FOLLY is
set. Note that the build now happens as part of the cmake step, since folly
must be installed before we call find_package() to find folly.
Test Plan:
Tested building rsocket-cpp on an Ubuntu 18.04 host. Confirmed it worked both
with an existing folly installation, as well as when using INSTALL_FOLLY to
have rsocket download and build folly as part of its build steps.
Summary: Find folly using its installed CMake configuration file, and delete the custom FindFolly.cmake file that was being used previously. The FindFolly.cmake file did not list all dependent libraries necessary to link statically against folly. Linking against folly as a shared library is not recommended, as folly provides no binary compatibility guarantees between changes. folly's current CMake-based build only builds it as a static library by default.
This also changes the build process to build folly with CMake rather than its deprecated autotools-based build when RSOCKET_INSTALL_DEPS or INSTALL_FOLLY is set. Note that the build now happens as part of the
cmake
step, since folly must be installed before we callfind_package()
to find folly.Test Plan: Tested building rsocket-cpp on an Ubuntu 18.04 host. Confirmed it worked both with an existing folly installation, as well as when using
INSTALL_FOLLY
to have rsocket download and build folly as part of its build steps.