Cleanup GitHub action script #1421
Conversation
|
CI Vulkan-Loader build queued with queue ID 105049. |
|
CI Vulkan-Loader build # 2424 running. |
|
CI Vulkan-Loader build # 2424 passed. |
| strategy: | ||
| matrix: | ||
| cc: [ gcc, clang ] | ||
| cxx: [ g++, clang++ ] | ||
| compiler: [ {cc: gcc, cxx: g++}, {cc: clang, cxx: clang++} ] |
There was a problem hiding this comment.
Specifying the compilers this way avoids the need to use exclude
| CC: ${{matrix.cc}} | ||
| CXX: ${{matrix.cxx}} | ||
|
|
||
| - name: Build the loader |
There was a problem hiding this comment.
Removed names when I felt they were redundant.
| - run: ctest --output-on-failure --test-dir build/ | ||
| - run: cmake --install build --prefix build/install | ||
|
|
||
| mac: | ||
| runs-on: macos-12 | ||
|
|
||
| runs-on: macos-11 |
There was a problem hiding this comment.
macos-11 is fixed with CMake 3.28.1
| - name: Setup ccache | ||
| uses: hendrikmuhs/[email protected] | ||
| with: | ||
| key: mingw-ccache |
There was a problem hiding this comment.
Removed ccache usage since the key was getting duplicated and therefor getting poor utilization. The loader doesn't really need it anyway.
| cmake -S. -B build ` | ||
| -D BUILD_TESTS=ON ` |
There was a problem hiding this comment.
Broke up windows commands via ` which is what powershell accepts
| cc: [ gcc, clang ] | ||
| cxx: [ g++, clang++ ] |
There was a problem hiding this comment.
Compilers weren't getting used at all.
|
CI Vulkan-Loader build queued with queue ID 105063. |
|
CI Vulkan-Loader build # 2425 running. |
|
CI Vulkan-Loader build # 2425 passed. |
|
CI Vulkan-Loader build queued with queue ID 105078. |
|
CI Vulkan-Loader build # 2426 running. |
|
CI Vulkan-Loader build # 2426 passed. |
No description provided.