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 LAPACK option and support building only LAPACKE without requiring fortran #273

Closed
1 of 4 tasks
jcfr opened this issue Aug 13, 2018 · 4 comments · Fixed by #834
Closed
1 of 4 tasks

Add LAPACK option and support building only LAPACKE without requiring fortran #273

jcfr opened this issue Aug 13, 2018 · 4 comments · Fixed by #834

Comments

@jcfr
Copy link
Contributor

jcfr commented Aug 13, 2018

@langou : @phcerdan and I will start working on the following

  • Do not require Fortran. See

    project(LAPACK Fortran C)

  • Add option LAPACK enabled by default (similar to the other option named LAPACKE and CBLAS). Enable Fortran only with LAPACK is on. In a nutshell, enable_language(Fortran) will be called only if LAPACK is ON

  • Exclude this code if CMAKE_CONFIGURATION_TYPES is defined

  • Update appveyor.yml to test building with LAPACK:BOOL=OFF and LAPACKE:BOOL=ON using generators -G "Visual Studio 14 2015 Win64 (will add VS 2017 later)

@julielangou
Copy link
Contributor

Will this fix #267?

@jcfr
Copy link
Contributor Author

jcfr commented Aug 13, 2018

Will this fix #267?

yes

@jcfr
Copy link
Contributor Author

jcfr commented Aug 13, 2018

@phcerdan Let's fix #267 in with a dedicated PR.

@phcerdan
Copy link
Contributor

@phcerdan Let's fix #267 in with a dedicated PR.

Opened #274 for this, please have a look!

a-andreyev added a commit to a-andreyev/lapack that referenced this issue May 8, 2020
a-andreyev added a commit to a-andreyev/lapack that referenced this issue May 8, 2020
a-andreyev added a commit to a-andreyev/lapack that referenced this issue May 8, 2020
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 a pull request may close this issue.

3 participants