cgal icon indicating copy to clipboard operation
cgal copied to clipboard

CMake - Add one CMakeLists per package

Open janetournois opened this issue 1 year ago • 3 comments

Summary of Changes

To be able to test a full package (tests, examples, and if they exist benchmark and demo) at once, for example using ctest (or simply compiling in visual studio), I suggest to add one CMakeLists per package, at the root of the package.

This PR introduces two such CMakeLists, which I think are very handy.

janetournois avatar Jun 07 '24 08:06 janetournois

@lrineau for "benchmark" I suggest 808fa60

janetournois avatar Jun 10 '24 14:06 janetournois

Instead of manually creating cmake scripts, can't we create custom targets in case we have CGAL_ENABLE_TESTING=ON?

sloriot avatar Sep 17 '24 08:09 sloriot

I don't like much that option, because it forces to use the top-level CMakeLists, while the solution I suggested is "package-centered"

janetournois avatar Sep 19 '24 07:09 janetournois