Be more explicit about JUCE7 and JUCE8 tests

sudara · Jun 9, 2024 · ca84412 · ca84412
1 parent f7315b1
commit ca84412
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -34,7 +34,7 @@ jobs:
       fail-fast: false # show errors for each platform vs. cancel build
       matrix:
         os: [ macos-11, macos-12, macos-13, macos-14, windows-2022, windows-2019, ubuntu-latest ]
-        juce: [ 7, 8 ]
+        juce: [ JUCE7, JUCE8 ]
 
     steps:
 
@@ -76,7 +76,7 @@ jobs:
         # D_DISABLE_CONSTEXPR_MUTEX_CONSTRUCTOR is a temporary workaround for a Visual Studio+GA snafu
         # check if resolved here: https://github.com/sudara/pamplejuce/issues/103
         run: | 
-          if [ ${{ matrix.juce }} -eq 8 ]; then
+          if [ ${{ matrix.juce }} -eq JUCE8 ]; then
             cmake -B ${{ env.BUILD_DIR }} -G Ninja -DCMAKE_BUILD_TYPE=${{ env.BUILD_TYPE}} -DCMAKE_C_COMPILER_LAUNCHER=sccache -DCMAKE_CXX_COMPILER_LAUNCHER=sccache . -DCMAKE_SYSTEM_VERSION=10.0.22621.0 -DCMAKE_CXX_FLAGS=" /D_DISABLE_CONSTEXPR_MUTEX_CONSTRUCTOR -DJUCE8=ON"
           else
             cmake -B ${{ env.BUILD_DIR }} -G Ninja -DCMAKE_BUILD_TYPE=${{ env.BUILD_TYPE}} -DCMAKE_C_COMPILER_LAUNCHER=sccache -DCMAKE_CXX_COMPILER_LAUNCHER=sccache . -DCMAKE_SYSTEM_VERSION=10.0.22621.0 -DCMAKE_CXX_FLAGS=" /D_DISABLE_CONSTEXPR_MUTEX_CONSTRUCTOR"
@@ -85,7 +85,7 @@ jobs:
       - name: Configure
         if: runner.os != 'Windows'
         run: |
-          if [ ${{ matrix.juce }} -eq 8 ]; then
+          if [ ${{ matrix.juce }} -eq JUCE8 ]; then
             cmake -B ${{ env.BUILD_DIR }} -G Ninja -DCMAKE_BUILD_TYPE=${{ env.BUILD_TYPE}} -DCMAKE_C_COMPILER_LAUNCHER=sccache -DCMAKE_CXX_COMPILER_LAUNCHER=sccache . -DJUCE8=ON
           else 
             cmake -B ${{ env.BUILD_DIR }} -G Ninja -DCMAKE_BUILD_TYPE=${{ env.BUILD_TYPE}} -DCMAKE_C_COMPILER_LAUNCHER=sccache -DCMAKE_CXX_COMPILER_LAUNCHER=sccache .