-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Reintroduce SparseArrays in the system image #46278
Conversation
I don't think this needs backport. It's in the sysimage in 1.8 |
I expected this. Is this the plan for all stdlibs that were being removed? |
We still hope to keep |
I can't quite tell what is causing the failing darwin x64 test. Doesn't feel related to this PR. |
@SobhanMP Heads up. |
This reverts commit 237c8d3.
This reverts commit 237c8d3.
Why does it need to be removed? |
So we can bump the package w/o bumping Julia or needing to backport (or at least make the process easier). |
Reverting #44247 in order to avoid the impact on startup time for several packages. The impact is about 2 seconds, but it affects several packages that do not always need SparseArrays, but are forced to have that dependency due to lack of conditional dependencies, which we hope to have in 1.10.
We do not need to reintroduce
SuiteSparse
separately since it was reintegrated into SparseArrays.jl in order to maintain backwards compatibility.cc @Wimmerer