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 netcdf-fortran to MacOS homebrew build #211

Merged
merged 2 commits into from
Oct 11, 2022

Conversation

marshallward
Copy link
Member

The latest homebrew netcdf package appears to no longer include netCDF Fortran headers and tools. This patch installs the new netcdf-fortran package which adds them back.

The latest homebrew `netcdf` package appears to no longer include
netCDF Fortran headers and tools.  This patch installs the new
`netcdf-fortran` package which adds them back.
@codecov
Copy link

codecov bot commented Oct 4, 2022

Codecov Report

Merging #211 (19f0127) into dev/gfdl (9fcdce9) will not change coverage.
The diff coverage is n/a.

❗ Current head 19f0127 differs from pull request most recent head 4b3af59. Consider uploading reports for the commit 4b3af59 to get more accurate results

@@            Coverage Diff            @@
##           dev/gfdl     #211   +/-   ##
=========================================
  Coverage     37.20%   37.20%           
=========================================
  Files           262      262           
  Lines         72871    72871           
  Branches      13619    13619           
=========================================
  Hits          27112    27112           
  Misses        40743    40743           
  Partials       5016     5016           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@Hallberg-NOAA Hallberg-NOAA left a comment

Choose a reason for hiding this comment

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

This seems like a reasonable addition, and it should fix the problems with the MacOS version of the TC testing.

@Hallberg-NOAA Hallberg-NOAA merged commit d214ad8 into NOAA-GFDL:dev/gfdl Oct 11, 2022
@marshallward marshallward deleted the gh_macos_nc_fortran branch November 8, 2022 23:03
adcroft pushed a commit to adcroft/MOM6 that referenced this pull request Dec 7, 2022
Merge with latest main (2022 feb17)
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