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

CI: Activate "enable-fortran" in our CI tests #1214

Closed
holke opened this issue Aug 13, 2024 · 3 comments · Fixed by #1255
Closed

CI: Activate "enable-fortran" in our CI tests #1214

holke opened this issue Aug 13, 2024 · 3 comments · Fixed by #1255
Labels
CI Continuous Integration CMake Is about the CMake build system Fortran workload: medium Would take a week or less

Comments

@holke
Copy link
Collaborator

holke commented Aug 13, 2024

Feature request

Is your feature request related to a problem? Please describe.
We are currently adding a fortran interface. With #1182 we now have a configure option "--enable-fortran" and a CMake option should follow.
These are currently not tested in our CI, which also excludes all possible future Fortran related test cases.

Describe the solution or feature you'd like
Test the "enable-fortran" option in our CI.
I would suggest adding it to all test cases by default.

Should we also consider testing one version with "disable-fortran"?

Describe alternatives you've considered
Not testing is not an option.

Estimated priority
Which of these is most applicable (remove the others):

"Priority: medium" Should be solved within half a year

Additional context

@holke holke added CI Continuous Integration workload: medium Would take a week or less Fortran labels Aug 13, 2024
@holke holke mentioned this issue Aug 13, 2024
14 tasks
@Davknapp
Copy link
Collaborator

I think @sandro-elsweijer is already on it

@holke
Copy link
Collaborator Author

holke commented Aug 13, 2024

Yes, havent seen that before. I guess #1204 will adress this.

@Davknapp Davknapp linked a pull request Aug 13, 2024 that will close this issue
14 tasks
@sandro-elsweijer
Copy link
Collaborator

@holke I already implemented it for the CMake build system, but there the cmake flag is not yet implemented. I will not implement it for the autotools CI.

@holke holke mentioned this issue Aug 16, 2024
14 tasks
@sandro-elsweijer sandro-elsweijer added the CMake Is about the CMake build system label Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration CMake Is about the CMake build system Fortran workload: medium Would take a week or less
Projects
None yet
3 participants