[SuiteSparse] Update to 7.12.1 - only changes SSGraphBLAS#13124
[SuiteSparse] Update to 7.12.1 - only changes SSGraphBLAS#13124
Conversation
This update only changes GraphBLAS and the common library, but it isn't worth us doing a full bump for the common library right now.
|
Ok, turns out because we are building using the SuiteSparse version of the Cmake file, we do need SuiteSparse to be around at build time. I will need to examine the artifacts to see if they are in fact separated from SS though, but I think they should be. Alternately, we could just switch this recipe to use the up-upstream SuiteSparse:GraphBLAS repo (https://github.com/DrTimothyAldenDavis/GraphBLAS) instead of the SuiteSparse bundled one. |
|
There is no dependency but it just seemed easier to use the unified install script. There's no issue with using the GraphBLAS specific repo. Neither GraphBLAS nor LAGraph use the SuiteSparse_config malloc get/set functions (they use their own internal one). |
|
I would suggest we switch GraphBLAS to the separate upstream repo and decouple it altogether since it sounds straightforward. |
|
Would it be better to merge this, and then focus on separation of the GraphBLAS build in a separatePR? |
This update only changes GraphBLAS and the common library, but it isn't worth us doing a full bump for the common library right now.
GraphBLAS also doesn't actually depend on SuiteSparse, so we can break the dependency chain on it and allow GraphBLAS to be installed on older Julia versions independent of SuiteSparse.