On 12/16/2016 04:35 AM, Sylvain Joubert wrote: > The issue is that CMake appends its own "detect_leak=1" > > I see two possible fixes here: > 1- Since "detect_leaks=1" is the default, we can simply not append > it to the options and make sure the user options are last
It looks like this was added in commit 44726714322ca0b75628e234229f4583a480d7ec when support for asan was first added. Bill, do you recall why? > If my analysis is correct I can make a MR. Which fixes would you prefer? A MR would be fine, 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
