Skip to content

Commit

Permalink
Build the tidy plugin on non-windows
Browse files Browse the repository at this point in the history
  • Loading branch information
danakj committed Mar 11, 2023
1 parent 8f40b08 commit 4476dcf
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 0 deletions.
6 changes: 6 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -138,6 +138,11 @@ jobs:
set(HAS_LLVM OFF)
endif()
set(CLANG_TIDY OFF)
if (NOT "${{ runner.os }}" STREQUAL "Windows")
set(CLANG_TIDY ON)
endif()
if ("${{ runner.os }}" STREQUAL "Windows" AND NOT "x${{ matrix.config.environment_script }}" STREQUAL "x")
execute_process(
COMMAND "${{ matrix.config.environment_script }}" && set
Expand All @@ -163,6 +168,7 @@ jobs:
-D CMAKE_MAKE_PROGRAM=${ninja_program}
-D SUBSPACE_BUILD_CIR=${HAS_LLVM}
-D SUBSPACE_BUILD_SUBDOC=${HAS_LLVM}
-D SUBSPACE_BUILD_TIDY=${CLANG_TIDY}
RESULT_VARIABLE result
)
if (NOT result EQUAL 0)
Expand Down
6 changes: 6 additions & 0 deletions .github/workflows/try.yml
Original file line number Diff line number Diff line change
Expand Up @@ -138,6 +138,11 @@ jobs:
set(HAS_LLVM OFF)
endif()
set(CLANG_TIDY OFF)
if (NOT "${{ runner.os }}" STREQUAL "Windows")
set(CLANG_TIDY ON)
endif()
if ("${{ runner.os }}" STREQUAL "Windows" AND NOT "x${{ matrix.config.environment_script }}" STREQUAL "x")
execute_process(
COMMAND "${{ matrix.config.environment_script }}" && set
Expand All @@ -162,6 +167,7 @@ jobs:
-D CMAKE_MAKE_PROGRAM=${ninja_program}
-D SUBSPACE_BUILD_CIR=${HAS_LLVM}
-D SUBSPACE_BUILD_SUBDOC=${HAS_LLVM}
-D SUBSPACE_BUILD_TIDY=${CLANG_TIDY}
RESULT_VARIABLE result
)
if (NOT result EQUAL 0)
Expand Down

0 comments on commit 4476dcf

Please sign in to comment.