We should get rid of the GLOB here
|
file(GLOB SOURCE_FILES "*.cpp") |
Reason is, that this will not rerun if files change. For that we would need to add CONFIGURE_DEPENDS. But even better would be if we inject the source names directly. This allows CMake to accurately track which files have changed and only update/compile/install based on that.
CONFIGURE_DEPENDS does also solve this but forces CMake to reconfigure the whole directory every single time
We should get rid of the
GLOBherepykokkos/pykokkos/core/CMakeLists.txt
Line 21 in 9addcb0
Reason is, that this will not rerun if files change. For that we would need to add
CONFIGURE_DEPENDS. But even better would be if we inject the source names directly. This allows CMake to accurately track which files have changed and only update/compile/install based on that.CONFIGURE_DEPENDSdoes also solve this but forces CMake to reconfigure the whole directory every single time