On 08/23/2015 04:41 PM, Michael Scott wrote: > Okay no problem, I've changed the patch to use > SelectLibraryConfigurations instead. I've also changed the last bit of > the patch, to set the "IMPORTED_CONFIGURATIONS" target property as well > when there's debug and release variants, to be consistent with other > Find modules. Let me know if this shouldn't be done in this situation.
Please add each of the _DEBUG and _RELEASE configurations if and only if the corresponding variables are set. Grep for mention of IMPORTED_CONFIGURATIONS in other Modules/Find*.cmake files for examples. Thanks, -Brad P.S. Your mailer is breaking threads. -- 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