fixing clang on windows tests?

NCAR · Mar 25, 2024 · db3d546 · db3d546
1 parent 1f82a3e
commit db3d546
Showing 1 changed file with 12 additions and 7 deletions.
diff --git a/python/CMakeLists.txt b/python/CMakeLists.txt
@@ -3,18 +3,23 @@ pybind11_add_module(musica_python wrapper.cpp)
 set_target_properties(musica_python PROPERTIES OUTPUT_NAME musica)
 
 target_include_directories(musica_python
-    PUBLIC  
-      $<BUILD_INTERFACE:${CMAKE_SOURCE_DIR}/lib/micm/include>
-      $<INSTALL_INTERFACE:${MUSICA_INSTALL_INCLUDE_DIR}>
-  )
+  PUBLIC  
+    $<BUILD_INTERFACE:${CMAKE_SOURCE_DIR}/lib/micm/include>
+    $<INSTALL_INTERFACE:${MUSICA_INSTALL_INCLUDE_DIR}>
+)
 
 target_link_libraries(musica_python PUBLIC musica::musica)
 
-message(STATUS "thing: $<CONFIG}>")
+
 if(WIN32)
-    set(PYTHON_MODULE_PATH "${CMAKE_CURRENT_BINARY_DIR}/$<CONFIG>")
+  # clang on windows doesn't need the config directory
+  if(CMAKE_CXX_COMPILER_ID MATCHES "Clang")
+      set(PYTHON_MODULE_PATH "${CMAKE_CURRENT_BINARY_DIR}")
+  else()
+      set(PYTHON_MODULE_PATH "${CMAKE_CURRENT_BINARY_DIR}/$<CONFIG>")
+  endif()
 else()
-    set(PYTHON_MODULE_PATH "${CMAKE_CURRENT_BINARY_DIR}")
+  set(PYTHON_MODULE_PATH "${CMAKE_CURRENT_BINARY_DIR}")
 endif()
 
 add_test(NAME musica_python_test