remove commented out line so hpp11 tests should fail now

KhronosGroup · Oct 25, 2024 · 1f0c59d · 1f0c59d
1 parent 8bc6df5
commit 1f0c59d
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 12 deletions.
diff --git a/.github/workflows/presubmit.yml b/.github/workflows/presubmit.yml
@@ -39,17 +39,6 @@ jobs:
       - name: Check generated headers
         run: git diff --exit-code
 
-  build_with_tests:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - name: Build
-        run: |
-          mkdir build
-          cd build
-          cmake -DCMAKE_INSTALL_PREFIX=install -DSPIRV_HEADERS_ENABLE_TESTS=ON ..
-          cmake --build . --target install
-
   test_cmake_min_required:
     runs-on: ubuntu-latest
     steps:

diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
@@ -30,7 +30,7 @@ target_link_libraries(spirv_headers_simple_test PRIVATE SPIRV-Headers::SPIRV-Hea
 add_test(NAME spirv_headers_simple_test COMMAND spirv_headers_simple_test)
 
 add_executable(spirv_headers_simple_test_cpp11 example11.cpp)
-#target_compile_definitions(spirv_headers_simple_test_cpp11 PRIVATE SPV_ENABLE_UTILITY_CODE)
+target_compile_definitions(spirv_headers_simple_test_cpp11 PRIVATE SPV_ENABLE_UTILITY_CODE)
 target_link_libraries(spirv_headers_simple_test_cpp11 PRIVATE SPIRV-Headers::SPIRV-Headers)
 set_target_properties(spirv_headers_simple_test_cpp11 PROPERTIES CXX_STANDARD 11)
 add_test(NAME spirv_headers_simple_test_cpp11 COMMAND spirv_headers_simple_test_cpp11)