From 5cff00e5ea893afbd7169d5c046936c5065336b4 Mon Sep 17 00:00:00 2001
From: Arseny Kapoulkine <arseny.kapoulkine@gmail.com>
Date: Sun, 27 Jun 2021 19:20:27 -0700
Subject: [PATCH] Fix coverage build to include CXX define

Also it seems like we don't need to restrict coverage build to
Linux/clang?
---
 .github/workflows/build.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 56b9554..2d22954 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -26,8 +26,8 @@ jobs:
         make test cxxstd=c++98 defines=${{matrix.defines}} config=debug -j2
         make test defines=${{matrix.defines}} config=sanitize -j2
     - name: make coverage
-      if: ${{matrix.os == 'ubuntu' && matrix.compiler == 'clang++'}}
       run: |
+        export CXX=${{matrix.compiler}}
         make test defines=${{matrix.defines}} config=coverage -j2
         bash <(curl -s https://codecov.io/bash) -f pugixml.cpp.gcov -X search -t ${{secrets.CODECOV_TOKEN}} -B ${{github.ref}}