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

Improve Fortran with CMake #99

Merged
merged 11 commits into from
Jun 4, 2023

Conversation

gruenich
Copy link
Contributor

@gruenich gruenich commented Jun 4, 2023

Improve handling of Fortran interface when building with CMake.

  • Removed examples without provided Fortran code
  • Removed main files from superlu_fortran library
  • Added building provided Fortran examples
  • Improved handling of xSDK varialbes for CBLAS and Fortran

The Fortran code is not provided for f_p?drive,
fpd?drive3d, f_5x5
Honor enable_internal_blaslib if set by the user, do not
always override with xSDK's default value.
Handle XSDK_ENABLE_Fortran similar to XSDK_ENABLE_Fortran.
Both main Fortran files where part of the library.
Remove use of enable_* variables
It does not serve any purpose
Use can define to build examples with ALL
@xiaoyeli xiaoyeli merged commit 5d9fba5 into xiaoyeli:master Jun 4, 2023
1 check passed
@gruenich gruenich deleted the feature/improve-fortran branch June 5, 2023 20:24
@prj-
Copy link
Contributor

prj- commented Jun 7, 2023

Either this MR or #98 broke configuration in PETSc, yet again, see, e.g., https://gitlab.com/petsc/petsc/-/jobs/4430894576. That's unbelievable...

@xiaoyeli
Copy link
Owner

xiaoyeli commented Jun 7, 2023

@gruenich
Can you please take a look at this error reported from PETSc?
Thanks.

@gruenich
Copy link
Contributor Author

gruenich commented Jun 8, 2023

I opened an issue to track PETSc' build problem: #101

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.

None yet

3 participants