Skip to content

Change CMakeLists.txt defaults in order to be more user-friendly.

Nick Thompson requested to merge (removed):master into master

This is not an uncontroversial commit, and I wouldn't be at all offended if this was closed.

However, we were discussing how to make VTKm more friendly to new users. Basically, all the defaults in the CMakeLists.txt are nice for developers. So when I clone this library and build it, first I have to rebuild it in release. Then I have to enable CUDA. Then I have to tell it to not build shared libs, since CUDA and shared libs are incompatible. Then I have to wait for the tests to build, even though I might not care about running the unit tests.

This fixes all these issues, from a user perspective. From a developer perspective it's probably painful, but nonetheless I think this will remove the first initial hurdles to start using VTK-m.

Merge request reports

Loading