Skip to content

Commit

Permalink
Removes FORCE from SET(KokkosKernels_ENABLE_ALL_COMPONENTS OFF) when …
Browse files Browse the repository at this point in the history
…a component is enabled

This fixes the following issue:

1. User sets KokkosKernernels_ENABLE_ALL_COMPONENTS to on
2. This enables each component
3. Invoke cmake again
4. Since components are enabled, KokkosKernels_ENABLE_ALL_COMPONENTS would be set to off
5. Things that rely on KokkosKernels_ENABLE_ALL_COMPONENTS are no longer build (e.g. tests)

By removing FORCE, we no longer override KokkosKernels_ENABLE_ALL_COMPONENTS if it is on
  • Loading branch information
cwpearson committed Feb 21, 2023
1 parent 6d73c14 commit 30616de
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cmake/kokkoskernels_components.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ KOKKOSKERNELS_ADD_OPTION(
# off.
IF (KokkosKernels_ENABLE_COMPONENT_BATCHED OR KokkosKernels_ENABLE_COMPONENT_BLAS
OR KokkosKernels_ENABLE_COMPONENT_GRAPH OR KokkosKernels_ENABLE_COMPONENT_SPARSE)
SET(KokkosKernels_ENABLE_ALL_COMPONENTS OFF CACHE BOOL "" FORCE)
SET(KokkosKernels_ENABLE_ALL_COMPONENTS OFF CACHE BOOL "")
ENDIF()

# Graph depends on everything else because it depends
Expand Down

0 comments on commit 30616de

Please sign in to comment.