[mlpack-git] [mlpack/mlpack] Added test coverage framework (#720)

[sumedhghaisas]

> +    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")
> +    # Remove optimizations for better line coverage
> +    set(DEBUG ON)
> +
> +    if(LCOV)
> +      configure_file(CMake/mlpack_coverage.in mlpack_coverage @ONLY)
> +      add_custom_target(mlpack_coverage DEPENDS mlpack_test COMMAND ${PROJECT_BINARY_DIR}/mlpack_coverage)
> +    else()
> +      message(WARNING "'lcov' not found, local coverage report is disabled. Install 'lcov' and rerun cmake to generate local coverage report.")

Maybe the user wants to use MLPACK with some other library to test coverage? So BUILD_WITH_COVERAGE just compiles mlpack with coverage functionality. Would it be better to separate the two? I mean create another target which will generate 'mlpack_coverage' . In that case there would be no confusion.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/720/files/ff02cb500867c83c848d69c3f85c5004b1d9fb4a#r70869929
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160714/6a1a3aa0/attachment.html>