Skip to content

Commit

Permalink
Remove CI test for macOS
Browse files Browse the repository at this point in the history
  • Loading branch information
TomasTorsvik committed Nov 25, 2024
1 parent d49fe2a commit 3c0cba8
Showing 1 changed file with 13 additions and 9 deletions.
22 changes: 13 additions & 9 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,14 +11,18 @@ jobs:
strategy:
fail-fast: false
matrix:
os: [ubuntu-latest, macos-latest]
os: [ubuntu-latest] # Testing with macos-latest currently not working
mpi: [true, false]
openmp: ['enabled', 'disabled']
ecosys: [false] # ecosys==true fails with gcc, disable for now but leave placeholder
exclude:
# Do not build on macOS with MPI as that is having some dependency issues
- os: macos-latest
mpi: true
# # Do not build on macOS with MPI as that is having some dependency issues
# - os: macos-latest
# mpi: true
# # Do not build on macOS with OpenMP. This cause an error as discussed in PR
# # https://github.com/NorESMhub/BLOM/pull/317#issuecomment-2191552330
# - os: macos-latest
# openmp: 'enabled'
# Run test (fuk95 executed with mpi=false) fails with ecosys
- ecosys: true
mpi: false
Expand All @@ -29,11 +33,11 @@ jobs:
sudo apt install -y libnetcdff-dev mpi-default-dev ninja-build
if: runner.os == 'Linux'

- name: Install dependencies - macOS
run: brew install netcdf-fortran open-mpi ninja
env:
HOMEBREW_NO_INSTALL_CLEANUP: 1
if: runner.os == 'macOS'
# - name: Install dependencies - macOS
# run: brew install netcdf-fortran open-mpi ninja
# env:
# HOMEBREW_NO_INSTALL_CLEANUP: 1
# if: runner.os == 'macOS'

- name: Setup Python for newer version of Meson
uses: actions/setup-python@v5
Expand Down

0 comments on commit 3c0cba8

Please sign in to comment.