From 39a4daa20b8a86f5d3a3ab426e7a2fea186232c9 Mon Sep 17 00:00:00 2001
From: Kalin Kiesling <krkiesling@gmail.com>
Date: Thu, 24 Mar 2022 15:07:12 -0500
Subject: [PATCH] don't compile faddeeva separately

---
 CMakeLists.txt | 25 +++++--------------------
 1 file changed, 5 insertions(+), 20 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index ba8a79efa..aa4499043 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -258,18 +258,6 @@ if("${isSystemDir}" STREQUAL "-1")
   set(CMAKE_INSTALL_RPATH "${CMAKE_INSTALL_FULL_LIBDIR}")
 endif()
 
-#===============================================================================
-# faddeeva library
-#===============================================================================
-
-add_library(faddeeva STATIC vendor/faddeeva/Faddeeva.cc)
-target_include_directories(faddeeva
-  PUBLIC
-    $<INSTALL_INTERFACE:include/faddeeva>
-    $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/vendor/faddeeva>
-)
-target_compile_options(faddeeva PRIVATE ${cxxflags})
-
 #===============================================================================
 # libopenmc
 #===============================================================================
@@ -372,7 +360,8 @@ list(APPEND libopenmc_SOURCES
 
 # Add bundled external dependencies
 list(APPEND libopenmc_SOURCES
-  src/external/quartic_solver.cpp)
+  src/external/quartic_solver.cpp
+  src/external/Faddeeva.cc)
 
 # For Visual Studio compilers
 if(MSVC)
@@ -426,7 +415,7 @@ endif()
 # target_link_libraries treats any arguments starting with - but not -l as
 # linker flags. Thus, we can pass both linker flags and libraries together.
 target_link_libraries(libopenmc ${ldflags} ${HDF5_LIBRARIES} ${HDF5_HL_LIBRARIES}
-                      faddeeva xtensor gsl::gsl-lite-v1 fmt::fmt)
+                      xtensor gsl::gsl-lite-v1 fmt::fmt)
 
 if(TARGET pugixml::pugixml)
   target_link_libraries(libopenmc pugixml::pugixml)
@@ -460,7 +449,7 @@ target_link_libraries(openmc libopenmc)
 # Ensure C++14 standard is used. Starting with CMake 3.8, another way this could
 # be done is using the cxx_std_14 compiler feature.
 set_target_properties(
-    openmc libopenmc faddeeva
+    openmc libopenmc
     PROPERTIES CXX_STANDARD 14 CXX_EXTENSIONS OFF)
 
 #===============================================================================
@@ -481,7 +470,7 @@ configure_file(cmake/OpenMCConfig.cmake.in "${CMAKE_BINARY_DIR}${CMAKE_FILES_DIR
 configure_file(cmake/OpenMCConfigVersion.cmake.in "${CMAKE_BINARY_DIR}${CMAKE_FILES_DIRECTORY}/OpenMCConfigVersion.cmake" @ONLY)
 
 set(INSTALL_CONFIGDIR ${CMAKE_INSTALL_LIBDIR}/cmake/OpenMC)
-install(TARGETS openmc libopenmc faddeeva
+install(TARGETS openmc libopenmc
   EXPORT openmc-targets
   RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
   LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
@@ -501,7 +490,3 @@ install(FILES man/man1/openmc.1 DESTINATION ${CMAKE_INSTALL_MANDIR}/man1)
 install(FILES LICENSE DESTINATION "${CMAKE_INSTALL_DOCDIR}" RENAME copyright)
 install(DIRECTORY include/ DESTINATION ${CMAKE_INSTALL_INCLUDEDIR})
 install(FILES "${CMAKE_BINARY_DIR}/include/openmc/version.h" DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/openmc)
-
-# Copy headers for vendored dependencies (note that all except faddeeva are handled
-# separately since they are managed by CMake)
-install(DIRECTORY vendor/faddeeva DESTINATION ${CMAKE_INSTALL_INCLUDEDIR})