Skip to content

Commit

Permalink
ci: fix compiler setup
Browse files Browse the repository at this point in the history
`os_name` in include section needs to be explicit (putting it at the end doesn't
apply to the extra jobs).

Move macOS test to extra job instead of doing all gcc/clang (which isn't setup
for mac anyway).

Also adds name to build-yosys task.
  • Loading branch information
KrystalDelusion committed Feb 6, 2024
1 parent dea490b commit 157a94a
Showing 1 changed file with 11 additions and 4 deletions.
15 changes: 11 additions & 4 deletions .github/workflows/test-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ on:

jobs:
build-yosys:
name: Build Yosys
runs-on: ${{ matrix.os }}
env:
CC: ${{ matrix.compiler }}
Expand All @@ -19,7 +20,6 @@ jobs:
matrix:
os:
- ubuntu-20.04
- macos-13
compiler:
- 'clang-12'
- 'gcc-11'
Expand All @@ -29,20 +29,27 @@ jobs:
- 'c++17'
- 'c++20'
include:
# Add os_name
- os: ubuntu-20.04
os_name: focal
# Build for testing
- os: ubuntu-20.04
os_name: focal
compiler: 'clang'
cpp_std: 'c++11'
# macOS build
- os: macos-13
compiler: 'clang'
cpp_std: 'c++11'
# Limited testing for older compilers
- os: ubuntu-20.04
os_name: focal
compiler: 'clang-11'
cpp_std: 'c++11'
- os: ubuntu-20.04
os_name: focal
compiler: 'gcc-10'
cpp_std: 'c++11'
# Add os_name
- os: ubuntu-20.04
os_name: focal
fail-fast: false
steps:
- name: Install Linux Dependencies
Expand Down

0 comments on commit 157a94a

Please sign in to comment.