From 19b30d858afba1b6a5af3e905af6ba02a1d77e85 Mon Sep 17 00:00:00 2001 From: brian-kelley Date: Mon, 3 Jul 2023 11:22:00 -0600 Subject: [PATCH] Remove non-existant dir from CMake include paths (#1892) 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. --- ode/CMakeLists.txt | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/ode/CMakeLists.txt b/ode/CMakeLists.txt index 9d92dc07ba..b9cf089445 100644 --- a/ode/CMakeLists.txt +++ b/ode/CMakeLists.txt @@ -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)