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

Upgrade to Julia 1.3+, use JLL package to provide FFTW #122

Merged
merged 1 commit into from
Nov 27, 2019

Conversation

staticfloat
Copy link
Collaborator

This is the first package where I've tried to mix JLL packages with more
custom installation procedures, but it seems to more-or-less work.

@staticfloat
Copy link
Collaborator Author

Appveyor 1.3 x86 failure will be fixed in rc5

This is the first package where I've tried to mix JLL packages with more
custom installation procedures, but it seems to more-or-less work.
@ararslan
Copy link
Member

Lookin' good! Would it be possible to use a BinaryBuilder-provided MKL upon request instead of using Conda?

@ararslan ararslan merged commit 82c835d into master Nov 27, 2019
@ararslan ararslan deleted the sf/jll_packages branch November 27, 2019 04:07
@staticfloat
Copy link
Collaborator Author

Lookin' good! Would it be possible to use a BinaryBuilder-provided MKL upon request instead of using Conda?

https://github.com/JuliaBinaryWrappers/MKL_jll.jl/releases/tag/MKL-v2019.0.117%2B0

@ararslan
Copy link
Member

Yeah, I know it exists, I'm just not sure how one would restructure the provider choice since it's actually loading a package rather than just picking up an existing library.

@staticfloat
Copy link
Collaborator Author

I think just like how we do this:

https://github.com/JuliaMath/FFTW.jl/pull/122/files#diff-ed74277d7f2e98f12cf51e6b8be90705R44-R48

We could generate a few lines to load in MKL_jll.

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.

2 participants