Skip to content

Commit

Permalink
Be explicit with OS settings. Also try to workaround Homebrew/homebre…
Browse files Browse the repository at this point in the history
  • Loading branch information
asl committed Apr 30, 2024
1 parent 115859c commit 7a554ef
Showing 1 changed file with 15 additions and 6 deletions.
21 changes: 15 additions & 6 deletions .github/workflows/release.yml
Expand Up @@ -115,7 +115,11 @@ jobs:
build-mac:
strategy:
matrix:
os: [macos-13, macos-14]
include:
- os: macos-13
arch: x86_64
- os: macos-14
arch: arm64

name: '🍎 Build SPAdes for macOS'
runs-on: ${{ matrix.os }}
Expand All @@ -139,7 +143,7 @@ jobs:

- name: '⚙️ Install dependencies'
run: >
brew install llvm libomp
brew install llvm libomp bzip2
- name: '⚙️ Configure CMake'
run: >
Expand All @@ -149,6 +153,7 @@ jobs:
-DSPADES_USE_NCBISDK=ON -DSPADES_ENABLE_PROJECTS=all
-DCMAKE_C_COMPILER=$(brew --prefix llvm)/bin/clang
-DCMAKE_CXX_COMPILER=$(brew --prefix llvm)/bin/clang++
-DCMAKE_EXE_LINKER_FLAGS=-L$(brew --prefix llvm)/lib/c++
- name: '🚧 Build'
run: >
Expand All @@ -160,18 +165,22 @@ jobs:
- name: '📦 Package'
uses: actions/upload-artifact@v4
with:
name: build-artifacts-mac-${{ matrix.os }}
name: build-artifacts-mac-${{ matrix.arch }}
path: |
${{env.BUILD_DIR}}/${{env.PKG_MAC}}.tar.gz
spades-1k-checks-mac:
name: '🚬 E. coli 1k smoke checks'
strategy:
matrix:
os: [macos-13, macos-14]
include:
- os: macos-13
arch: x86_64
- os: macos-14
arch: arm64

runs-on: ${{ matrix.os }}

needs: build-mac

steps:
Expand All @@ -184,7 +193,7 @@ jobs:
- name: '📦 Download package'
uses: actions/download-artifact@v4
with:
name: build-artifacts-mac-${{ matrix.os }}
name: build-artifacts-mac-${{ matrix.arch }}
path: ${{env.INSTALL_DIR}}

- name: '📦 Unpack package'
Expand Down

0 comments on commit 7a554ef

Please sign in to comment.