Skip to content

Commit

Permalink
Remove non-existant dir from CMake include paths (#1892)
Browse files Browse the repository at this point in the history
ode/ in the binary dir doesn't exist as a header directory
(because the ODE component has no auto-generated headers for ETI).
The unit tests might be there, but contain no headers that need to be
found.

This fixes errors when building against a Trilinos installation, where
CMake looks for this directory that doesn't exist.
  • Loading branch information
brian-kelley authored Jul 3, 2023
1 parent 25dea4f commit 19b30d8
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion ode/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -11,5 +11,8 @@ ENDIF()


# Adding unit-tests
KOKKOSKERNELS_INCLUDE_DIRECTORIES(${CMAKE_CURRENT_BINARY_DIR}/ode)
# Note BMK: Since ODE has no auto-generated ETI files, this directory does not exist in a build without unit tests.
# This causes configure errors when building an app against a Trilinos install, and the unit test build dir doesn't contain any headers that need to be found.
# But uncomment the next line if ETI headers are ever added.
# KOKKOSKERNELS_INCLUDE_DIRECTORIES(${CMAKE_CURRENT_BINARY_DIR}/ode)
KOKKOSKERNELS_INCLUDE_DIRECTORIES(REQUIRED_DURING_INSTALLATION_TESTING ${CMAKE_CURRENT_SOURCE_DIR}/ode)

0 comments on commit 19b30d8

Please sign in to comment.