Skip to content

Commit

Permalink
Merge branch 'develop' into feature/mri-adaptivity
Browse files Browse the repository at this point in the history
  • Loading branch information
gardner48 authored Oct 1, 2024
2 parents 96926e7 + 25ea7af commit adccf33
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 1 deletion.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,8 @@ Fixed loading the default IMEX-MRI method if `ARKodeSetOrder` is used to specify
a third or fourth order method. Previously, the default second order method
was loaded in both cases.

Fixed a CMake configuration issue related to aliasing an `ALIAS` target when
using `ENABLE_KLU=ON` in combination with a static-only build of SuiteSparse.

### Deprecation Notices

Expand Down
9 changes: 8 additions & 1 deletion cmake/tpl/FindKLU.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,14 @@ if(NOT

if(TARGET SuiteSparse::KLU)
if(NOT TARGET SUNDIALS::KLU)
add_library(SUNDIALS::KLU ALIAS SuiteSparse::KLU)
# For static-only builds of SuiteSparse, SuiteSparse::KLU will itself be
# an ALIAS target which can't be aliased.
get_target_property(klu_aliased_target SuiteSparse::KLU ALIASED_TARGET)
if(klu_aliased_target)
add_library(SUNDIALS::KLU ALIAS ${klu_aliased_target})
else()
add_library(SUNDIALS::KLU ALIAS SuiteSparse::KLU)
endif()
set(KLU_SUITESPARSE_TARGET ON)
mark_as_advanced(KLU_SUITESPARSE_TARGET)
endif()
Expand Down
2 changes: 2 additions & 0 deletions doc/shared/RecentChanges.rst
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,8 @@ Fixed loading the default IMEX-MRI method if :c:func:`ARKodeSetOrder` is used to
specify a third or fourth order method. Previously, the default second order method
was loaded in both cases.

Fixed a CMake configuration issue related to aliasing an ``ALIAS`` target when
using ``ENABLE_KLU=ON`` in combination with a static-only build of SuiteSparse.

**Deprecation Notices**

Expand Down

0 comments on commit adccf33

Please sign in to comment.