<p>In <a href="https://github.com/mlpack/mlpack/pull/720#discussion_r70496696">CMakeLists.txt</a>:</p>
<pre style='color:#555'>> @@ -74,6 +74,34 @@ if("${CMAKE_CXX_COMPILER_ID}" STREQUAL "Clang")
> set(CMAKE_MODULE_LINKER_FLAGS "${CMAKE_MODULE_LINKER_FLAGS} -lm")
> endif()
>
> +# Setup build for test coverage
> +if(BUILD_WITH_COVERAGE)
> + # Currently coverage only works with GNU g++
> + if ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "GNU")
> + # Find gcov and lcov
> + find_program(GCOV gcov)
> + find_program(LCOV lcov)
> +
> + if(NOT GCOV)
> + message(FATAL_ERROR "gcov not found! Aborting...")
> + endif()
> +
> + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} --coverage -fno-inline -fno-inline-small-functions -fno-default-inline -fprofile-arcs -fkeep-inline-functions")
> + message(WARNING "Adding debug options for coverage")
</pre>
<p>I think this should just be a <code>STATUS</code> message instead of a warning.</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/mlpack/mlpack/pull/720/files/ff02cb500867c83c848d69c3f85c5004b1d9fb4a#r70496696">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe/AJ4bFI49HZq6jgVOYgqVJAKTXOj54UwZks5qU99PgaJpZM4JJw9X">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AJ4bFHubfgY_NABYJ5BkyF5LUuGsPyf1ks5qU99PgaJpZM4JJw9X.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
<link itemprop="url" href="https://github.com/mlpack/mlpack/pull/720/files/ff02cb500867c83c848d69c3f85c5004b1d9fb4a#r70496696"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>