New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update to SUNDIALS v7 #3835
Update to SUNDIALS v7 #3835
Conversation
@balos1 I am able to compile and run with both sundials 6 and 7 using gnu make. But there are still some issues with cmake.
Is version 7.0.0 considered incompatible with 6.0.0 in the sundials' cmake config file? If we want to support both v7 and v6, how should we handle this? |
Note that if I simply set the minimum version to 7.0.0 with core as optional, it works. |
@WeiqunZhang Unfortunately it looks like we setup the CMake version file for SUNDIALS with the find_package(SUNDIALS CONFIG REQUIRED
COMPONENTS ${SUNDIALS_COMPONENTS}
OPTIONAL_COMPONENTS core)
set(SUNDIALS_MINIMUM_VERSION 6.0.0)
if (SUNDIALS_VERSION VERSION_LESS ${SUNDIALS_MINIMUM_VERSION})
message(FATAL_ERROR "SUNDIALS ${SUNDIALS_MINIMUM_VERSION} or newer is required. Found version ${SUNDIALS_VERSION}.")
endif()
|
Co-authored-by: Axel Huebl <axel.huebl@plasma.ninja>
Co-authored-by: Axel Huebl <axel.huebl@plasma.ninja>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, looks good!
This is the rebirth of #3756. But it's compatible with SUNDIALS 6.