Skip to content

Commit

Permalink
Use Ubuntu 20.04 for coverage
Browse files Browse the repository at this point in the history
  • Loading branch information
2-towns committed Jan 9, 2025
1 parent 251d286 commit b5848da
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .github/actions/nimbus-build-system/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -85,8 +85,8 @@ runs:
pacman -U --noconfirm https://repo.msys2.org/mingw/ucrt64/mingw-w64-ucrt-x86_64-gcc-14.2.0-2-any.pkg.tar.zst https://repo.msys2.org/mingw/ucrt64/mingw-w64-ucrt-x86_64-gcc-libs-14.2.0-2-any.pkg.tar.zst
- name: Install gcc 14 on Linux
# Should be removed when ubuntu-latest is 26.04
if : ${{ inputs.os == 'linux' }}
# We don't want to install gcc 14 for coverage (Ubuntu 20.04)
if : ${{ inputs.os == 'linux' && !inputs.coverage }}
shell: ${{ inputs.shell }} {0}
run: |
# Add GCC-14 to alternatives
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/ci-reusable.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ jobs:
os: ${{ matrix.os }}
shell: ${{ matrix.shell }}
nim_version: ${{ matrix.nim_version }}
coverage: false

## Part 1 Tests ##
- name: Unit tests
Expand Down
7 changes: 6 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,11 @@ jobs:
cache_nonce: ${{ needs.matrix.outputs.cache_nonce }}

coverage:
runs-on: ubuntu-latest
# Force to stick to ubuntu 20.04 for coverage because

This comment has been minimized.

Copy link
@2-towns

2-towns Jan 9, 2025

Author Contributor

As mentioned in the comment, ubuntu-latest contains version 2 of lcov, which is breaking the coverage. I tried to run with the flags mentioned, --keep-going, but it didn't work. I am not sure if it is because of lcov alone, or the combination with gcov and ubuntu-24. But we should look into this during the next weeks/months because ubuntu-20.04 will soon be deprecated and then unsupported. @veaceslavdoina

# lcov was updated to 2.x version in ubuntu-latest
# and cause a lot of issues.
# See https://github.com/linux-test-project/lcov/issues/238
runs-on: ubuntu-20.04
steps:
- name: Checkout sources
uses: actions/checkout@v4
Expand All @@ -61,6 +65,7 @@ jobs:
with:
os: linux
nim_version: ${{ env.nim_version }}
coverage: true

- name: Generate coverage data
run: |
Expand Down

0 comments on commit b5848da

Please sign in to comment.