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

--enable-lapack flag problems #55

Closed
azrael417 opened this issue Sep 15, 2016 · 6 comments
Closed

--enable-lapack flag problems #55

azrael417 opened this issue Sep 15, 2016 · 6 comments

Comments

@azrael417
Copy link

It seems that the configure flag --enable-lapack just checks if liblapack is there. However, it would be good if it also checks if MKL is there and continue if this is the case. Maybe by test-compiling a small file which makes a lapack call with and without trailing underscores and if one of it succeeds, just continue.

@aportelli
Copy link
Collaborator

Hi. I am not so keen on going down the road of testing platform specific libraries like the MKL. Especially that the MKL can be configured linked in numerous different ways. However what I can do (and have already done with other libraries) is to add an option --with-lapack-name= to allow to change the name of lapack (e.g. to mkl_core). I'll look into that.

@aportelli aportelli added this to the 0.6.0 milestone Oct 3, 2016
@aportelli aportelli self-assigned this Oct 3, 2016
@paboyle
Copy link
Owner

paboyle commented Oct 11, 2016

We also need to be able to distinguish between MKL fftw wrappers and fftw itself in the build selection.

@aportelli
Copy link
Collaborator

Hi Peter, yes I will set all that clean when I will have my workstation with all the Intel software

@aportelli
Copy link
Collaborator

Hi Thorsten,

I have been trying to change all kind of things in the build system to address automatically some of the issues you have been filling. Could you please help me check that everything is ok?
Could you please check out our release candidate branch release/v.0.6.0 and follow the instructions in the updated README file and let me know if Grid configures and compiles fine without you needing to tweak any extra environment variables?

@aportelli
Copy link
Collaborator

Hi again Thorsten.

We are trying to close issues related to v0.6 in order to release. Could you please confirm that this issue is solved for you?

@azrael417
Copy link
Author

Hi Antonin, I will come back to that today, sorry for the delay.

Am 04.11.2016 um 06:56 schrieb Antonin Portelli notifications@github.com:

Hi again Thorsten.

We are trying to close issues related to v0.6 in order to release. Could you please confirm that this issue is solved for you?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants