-
Notifications
You must be signed in to change notification settings - Fork 55
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
Pkg.add("FFTW") shouldn't install MKL #133
Comments
I definitely agree, but this is a difficult problem to address at the moment, since there is no support in the package manager for conditional dependencies. |
Given that seems to be still some time away, perhaps reverting to the previous state would be best? Even with a good connection the download still takes quite a bit of time (during which it is unclear what's happening) |
The previous state was having MKL managed by Conda, which introduces a Python dependency. It sort of seems like there are no good options at the moment... |
I know very little about these things (and am eternally grateful to the poor souls that have to manage this) so sorry if I'm missing something, but isn't depending on python way better than depending on MKL? It's free software, it's more lightweight, it's shared by more packages, it runs on all sane platforms, and it's installed on most machines already. |
It was using Python to install MKL |
Not by default though? It's definitely the first time I've noticed |
Previously it only installed Anaconda Python etcetera if FFTW.jl was configured to use MKL. (It installed Conda.jl unconditionally, but Conda.jl installs Anaconda lazily.) Now, because Pkg doesn't support optional dependencies, we must unconditionally download MKL_jll. |
So why not switch back to the old system until Pkg supports optional dependencies? |
Please revert this. You are making my life with a bad network and AMD processors very hard. |
The artifact system already supports "lazy artifacts" that are only downloaded when the artifact is actually used. |
With latest release of In the context of |
Thank you Mosè, that's excellent! |
Since 8a9197a,
]add FFTW
installs MKL. I'm not a free software absolutist, but I don't think adding such a core package (that used to be part of base) should download a non-free binary dependency. A much more practical concern is that MKL is a whopping 130mb, and my work connection sucks...The text was updated successfully, but these errors were encountered: