Skip to content
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

Add CXSPARSE component to SuiteSparse #4709

Merged
merged 2 commits into from
Apr 5, 2022

Conversation

kulakovri
Copy link
Contributor

No description provided.

@giordano
Copy link
Member

giordano commented Apr 5, 2022

CC: @ViralBShah

@vchuravy
Copy link
Member

vchuravy commented Apr 5, 2022

@giordano remind me is this Suitesparse_jll shipped with Julia or can we use it directly? @kulakovri has a package that he wants to build that depends on this.

@giordano
Copy link
Member

giordano commented Apr 5, 2022

This is the stdlib: https://github.com/JuliaLang/julia/tree/dacf9d65aff4668b8fff25957d9aaa2cf03868c8/stdlib/SuiteSparse_jll. In order to use it you'll also have to update the package in Julia and wait for the next minor version (1.9 at this point)

@vchuravy
Copy link
Member

vchuravy commented Apr 5, 2022

sigh that's what I feared. Is one of the other Suitespares things usable?

Or can we at least use this to build a package?

@vchuravy vchuravy merged commit a4d30a3 into JuliaPackaging:master Apr 5, 2022
@ViralBShah
Copy link
Member

ViralBShah commented Apr 5, 2022

Also, in the past - people were unhappy when I added all the SuiteSparse libraries to the build (this was related to the Metis dependency) complaining that it was bloating the Julia distribution. Anyways, we hope to move Sparse out of system image and stdlib for 1.9: JuliaLang/julia#44247 (comment)

This will allow us to build SuiteSparse with all its bells and whistles, and hopefully even the GPU solvers!

In the meanwhile, my suggestion would be to revert this and instead create a separate BB recipe for SuiteSparseExtras, sort of like how we have SuiteSparseGraphBLAS.

vchuravy added a commit that referenced this pull request Apr 5, 2022
vchuravy added a commit that referenced this pull request Apr 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants