scalapack icon indicating copy to clipboard operation
scalapack copied to clipboard

Allow build without building tests

Open derpda opened this issue 4 years ago • 4 comments

Currently there seems to be no way to avoid building all tests when building with cmake. This takes considerable time, and allowing users to only compile the actual library would be easy by including an option like

option(SCALAPACK_BUILD_TESTS "Build all tests of the ScaLAPACK library" ON) and later writing

if (${SCALAPACK_BUILD_TESTS})
  add_subdirectory(TESTING)
endif()

If this is acceptable, I can make a small PR for this.

derpda avatar Dec 20 '20 15:12 derpda

Sounds good to me. A question I am asking myself is whether the testing should be enabled by default (and then deactivated by a flag) or whether the testing should be disabled by default (and then enabled by a flag). I see that you are choosing the alter (so testing disabled by default). This sounds good to me. I do not have an opinion on this. But yes, being able to control testing (or not) would be great.

langou avatar Dec 20 '20 16:12 langou

I think many libraries build their tests by default, so I would probably it at that as well. Its easy enough for users to disable it once the cmake option exists. I'll make a simple PR now!

derpda avatar Dec 20 '20 19:12 derpda

@langou Let me know what you think of the PR I made!

derpda avatar Dec 22 '20 11:12 derpda

Closed with PR #30

langou avatar Dec 26 '20 19:12 langou