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 requirements update (macOS, GeoClaw netCDF) #354

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

scivision
Copy link
Contributor

@scivision scivision commented Jul 22, 2024

ci: macos: update GCC version.

GeoClaw now requires netCDF.

Simplify CMake presets.

GCC-11 is being removed from GitHub Actions macOS runners in August 2024.

actions/runner-images#10213

GCC-11 is being removed from GitHub Actions macOS runners in a few weeks

actions/runner-images#10213
@scivision scivision changed the title Ci macOS CI requirements update (macOS, GeoClaw netCDF) Jul 22, 2024
@donnaaboise
Copy link
Member

donnaaboise commented Jul 23, 2024

I would prefer if GeoClaw didn't require NETCDF. We rarely use it and so do not require it.

@scottaiton
Copy link
Member

Netcdf is not a required library at all. Things should be able to work without it.

@scottaiton
Copy link
Member

The problem is probably related to geoclaw needing preprocessing for the fortran source files


"configurePresets": [
{
"name": "ci",
"binaryDir": "${sourceDir}/build",
"generator": "Ninja",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please stop removing Ninja as the ci generator. We need it for Geoclaw since it makes heavy use of f90 modules

@scottaiton scottaiton changed the base branch from develop-3d to develop September 20, 2024 15:08
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.

3 participants