Try again with testing modules on Clang 17

tcbrindle · Jan 25, 2024 · 6497ff5 · 6497ff5
1 parent 7db7611
commit 6497ff5
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/.github/workflows/macos.yml b/.github/workflows/macos.yml
@@ -18,16 +18,16 @@ jobs:
         cxx: [g++-13,
               /usr/local/opt/llvm@16/bin/clang++,
               /usr/local/opt/llvm@17/bin/clang++]
+        module_tests: [Off]
+
         include:
           - cxx: g++-13
             install: |
               brew update
               brew install gcc@13 ninja
-            module_tests: Off
           - cxx: /usr/local/opt/llvm@16/bin/clang++
             install: |
               brew install llvm@16 ninja
-            module_tests: Off
           - cxx: /usr/local/opt/llvm@17/bin/clang++
             install: |
               brew install llvm@17 ninja
@@ -51,7 +51,7 @@ jobs:
         cmake -GNinja -DCMAKE_BUILD_TYPE=${{matrix.build_type}} -DCMAKE_CXX_COMPILER=${{matrix.cxx}} \
                -DFLUX_BUILD_BENCHMARKS=${{matrix.build_type == 'Release'}} \
                -DFLUX_BUILD_MODULE=${{matrix.cxx != 'g++-13'}} \
-               -DFLUX_BUILD_TESTS_USING_MODULE={{matrix.module_tests}} \
+               -DFLUX_BUILD_TESTS_USING_MODULE=${{matrix.module_tests}} \
                $GITHUB_WORKSPACE
 
     - name: Build