-
Notifications
You must be signed in to change notification settings - Fork 66
Adjust CI to test for CMake versions #123
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
Conversation
87565fc
to
39d0ba9
Compare
Signed-off-by: Cristian Le <[email protected]>
Based on the failure in https://github.com/Reference-ScaLAPACK/scalapack/actions/runs/14168865703/job/39688007257. I would suggest bumping the minimum to I also think it is high time to drop the Makefile support based on the current status of the CI there. |
Note: do not merge this, because I still have a |
Hi @LecrisUT, Okey doc, let me know when this is ready for merge. I would be fine to push the cmake version requirement to 3.26. Julien. |
Signed-off-by: Cristian Le <[email protected]>
Signed-off-by: Cristian Le <[email protected]>
Signed-off-by: Cristian Le <[email protected]>
Thanks, the discussion on the minimum CMake was the only blocker. What to do with the Makefile builds will also be appreciated, but it can be deferred. |
This breaks something else, BTW.
|
Could you go into detail on what configure flags you used? Generally it is no longer recommended to use the But yes, there are plenty of things to be addressed and I will try to do them gradually with CI coverage along the way. |
Here is my configuration line/log. Let me know if you need something else.
|
If I further do
|
Ok, I've found the issue EXECUTE_PROCESS(COMMAND ${CMAKE_COMMAND}
"${PROJECT_SOURCE_DIR}/BLACS/INSTALL"
-G ${CMAKE_GENERATOR} ${BUILD_TYPE}
WORKING_DIRECTORY ${PROJECT_SOURCE_DIR}/BLACS/INSTALL/
RESULT_VARIABLE RESVAR OUTPUT_VARIABLE LOG1_OUT ERROR_VARIABLE LOG1_ERR
) It is pre-existing failure, but thanks for pointing it out. Could you split this in a separate issue and I will address it in an upcoming PR which will refactor most of these. |
It's working on |
Just reference my comment. The part there needs to be migrated to |
The reasoning for each CMake version to test:
latest
/latest-rc
: make sure previous policies are not dropped yet