On 08/16/2016 04:44 PM, Alexander Lamaison wrote: > I've attached a new patch without the changes to the singular-named variables.
Thanks. > + if(WIN32 AND NOT CYGWIN) > + set_target_properties(OpenSSL::Crypto PROPERTIES > + INTERFACE_LINK_LIBRARIES crypt32) > + else() > + set_target_properties(OpenSSL::Crypto PROPERTIES > + INTERFACE_LINK_LIBRARIES "${CMAKE_DL_LIBS}") > + endif() The addition of crypt32/dl is done with this logic in several places. Instead please use a _OPENSSL_LIBRARY_DEPENDS variable to hold the value and then reference it in the other locations so that the conditions for setting it do not have to be duplicated. Thanks, -Brad -- Powered by www.kitware.com Please keep messages on-topic and check the CMake FAQ at: http://www.cmake.org/Wiki/CMake_FAQ Kitware offers various services to support the CMake community. For more information on each offering, please visit: CMake Support: http://cmake.org/cmake/help/support.html CMake Consulting: http://cmake.org/cmake/help/consulting.html CMake Training Courses: http://cmake.org/cmake/help/training.html Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Follow this link to subscribe/unsubscribe: http://public.kitware.com/mailman/listinfo/cmake-developers